Skip to content

De-Imgur example images#4570

Merged
calculuschild merged 7 commits intonaturalcrit:masterfrom
dbolack-ab:moveSnippetImages
Feb 23, 2026
Merged

De-Imgur example images#4570
calculuschild merged 7 commits intonaturalcrit:masterfrom
dbolack-ab:moveSnippetImages

Conversation

@dbolack-ab
Copy link
Copy Markdown
Collaborator

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.

@5e-Cleric 5e-Cleric moved this from Backlog to Waiting for Calc's First Review in @calculuschild's backlog Jan 3, 2026
@calculuschild
Copy link
Copy Markdown
Member

calculuschild commented Jan 24, 2026

I agree with the idea here. Some thoughts:

  • Yes, a full URL is probably a good idea for these examples. It might actually be best to just make them all a full URL for consistency.
  • We should rename the files while we are at it, instead of 4a8179462cfdf39054a418efd4cb743e.jpg
  • This is probably also the time to transition away from the AI-generated images. I made those on my own laptop with Stable Diffusion + Photoshop back before AI-gen became a more critical issue, but I think the common consensus between the devs now is we don't want that to be part of our brand. Do you guys think we can find some good "actually free" artwork to sub in here that still looks ok for the theme, and won't come with any licensing/hosting restrictions? @G-Ambatte @Gazook89 @5e-Cleric @dbolack-ab ?

@5e-Cleric
Copy link
Copy Markdown
Member

5e-Cleric commented Jan 24, 2026

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
attaching a preview as well just in case:

image

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.

@calculuschild
Copy link
Copy Markdown
Member

calculuschild commented Jan 24, 2026

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.

@5e-Cleric
Copy link
Copy Markdown
Member

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.

@dbolack-ab
Copy link
Copy Markdown
Collaborator Author

Should I incorporate these things with this PR or is a separate one preferable?

@5e-Cleric
Copy link
Copy Markdown
Member

Put them in this one, no need for fuss with this one, should have done it months ago

@5e-Cleric 5e-Cleric added 🔍 R3 - Reviewed - Awaiting Fixes 🔧 PR is okayed but needs fixes before merging Approved Has been discussed and an approach is agreed upon labels Feb 3, 2026
@dbolack-ab dbolack-ab added 🔍 R4 - Fixed! Awaiting re-review⭐ PR review comments addressed and removed 🔍 R3 - Reviewed - Awaiting Fixes 🔧 PR is okayed but needs fixes before merging labels Feb 18, 2026
@calculuschild calculuschild temporarily deployed to homebrewery-pr-4570 February 23, 2026 01:23 Inactive
@calculuschild
Copy link
Copy Markdown
Member

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.

@calculuschild calculuschild merged commit 191cdcf into naturalcrit:master Feb 23, 2026
1 of 2 checks passed
@github-project-automation github-project-automation Bot moved this from Waiting for Calc's First Review to Done in @calculuschild's backlog Feb 23, 2026
@calculuschild calculuschild added 🔍 R5 - Approved for merge 💯 PR has all feedback resolved and ready to merge and removed 🔍 R4 - Fixed! Awaiting re-review⭐ PR review comments addressed labels Feb 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved Has been discussed and an approach is agreed upon 🔍 R5 - Approved for merge 💯 PR has all feedback resolved and ready to merge

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants