-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Content] Hurricanes and Salt Marshes Story #487
Conversation
✅ Deploy Preview for visex ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@acblackford the only thing that I see is image size, the two images are >1 MB, is that already compressed? If it could be smaller <500kb, that would be ideal. The smaller the better. Otherwise, this looks good to go and I'll give a check mark depending on the image question |
@aboydnw As far as I am aware Trent compressed the images before he graduated and stopped working with us. He was the lead on this story so I do not have a clear answer. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, since it is only two images I think not worth the time to further compress.
Why are you creating this Pull Request?