De-Imgur example images#4570
Conversation
|
I agree with the idea here. Some thoughts:
|
|
Without a doubt, now if everyone will agree on the quality, that's a different question i can't answer. I'd ask you all again to review https://homebrewery.naturalcrit.com/share/WOd9MWxphMwo
now, this is for the covers, if everyone agrees i'll start searching for assets for everything else. This should not block the current PR, let's de-imgur the images and later swap them as we go. |
|
These are all really nice. Good finds. Swapping the images should definitely block this though. We don't want to make a snippet that points to temporary images we intend to delete later. Otherwise we get users with broken links in their brews, and we get complaints that our example images have stopped working. |
|
I'd use the assets used in this brew: https://homebrewery.naturalcrit.com/share/VIkdYqx9_6LE you may download the assets from my imagekit URLs, or i can provide them. Word of advice: we probably want to downscale some of these, at least to the max quality output of the browser render process, 300DPI, which for a letter sized page is 2551 px × 3295 px, some of them are larger than that. |
|
Should I incorporate these things with this PR or is a separate one preferable? |
|
Put them in this one, no need for fuss with this one, should have done it months ago |
|
Alright, I'll go ahead and merge this. We'll just need to swap in the public domain art before deploying the next live version. |

Description
This relocates the handful of imgur based images to /assets.
There may be value in one or more of these snippets using a full URL to make sure it is clear to users who are not yet familiar with Markdown how to construct the image markup.