Skip to content

Conversation

@christymelucci
Copy link
Contributor

@christymelucci christymelucci commented Jan 10, 2022

Replace placeholder thumbs for all current news items
Closes #544

@christymelucci christymelucci changed the title cmelucci/news item images cmelucci/news-item-images Jan 10, 2022
@netlify
Copy link

netlify bot commented Jan 10, 2022

✔️ Deploy Preview for cmu-delphi-main ready!

🔨 Explore the source changes: b81f32e

🔍 Inspect the deploy log: https://app.netlify.com/sites/cmu-delphi-main/deploys/61de08cce1b05700081df971

😎 Browse the preview: https://deploy-preview-545--cmu-delphi-main.netlify.app

@christymelucci christymelucci changed the title cmelucci/news-item-images Update news page: Replace placeholder images Jan 11, 2022
@christymelucci christymelucci changed the title Update news page: Replace placeholder images Update news page: Replace placeholder image Jan 11, 2022
File make be a bit large
@christymelucci
Copy link
Contributor Author

christymelucci commented Jan 11, 2022

I added a high-res images to the PNAS news item, but it still appears as low res on dev.
Can we release "as is", then address image specs, etc. to solve over the next week, then update the News images for the next release?
@ryantibs
@krivard

@christymelucci christymelucci marked this pull request as ready for review January 11, 2022 23:02
@ryantibs
Copy link
Member

Sure, I'm fine to proceed as is for efficiency and revisiting after that.

Copy link
Contributor

@krivard krivard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. It's a bit blurry, but I don't see any compression artifacts, so I'm satisfied

@krivard krivard merged commit aed35f0 into dev Jan 12, 2022
@krivard krivard deleted the cmelucci/news-item-images branch January 12, 2022 14:35
@krivard krivard mentioned this pull request Jan 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace placeholder image for current news items

4 participants