Skip to content

Update Castle on a Lake preview image - #45

Open
shmall03 wants to merge 1 commit into
omacom-io:masterfrom
shmall03:update-castle-on-a-lake-preview
Open

Update Castle on a Lake preview image#45
shmall03 wants to merge 1 commit into
omacom-io:masterfrom
shmall03:update-castle-on-a-lake-preview

Conversation

@shmall03

Copy link
Copy Markdown
Contributor

Follow-up to #17.

The preview image merged in #17 was generated from a screenshot taken before the theme repo's final commit (654a89b, Rewrite README and add desktop preview), which added the current desktop screenshot (preview.jpg, 1920x1080).

This PR replaces assets/themes/castle-on-a-lake.webp with that newer screenshot, resized to the standard 1200px 16:9 webp (42 KB). No HTML changes needed — the gallery entry stays as-is.

Theme repo: https://github.com/shmall03/omarchy-castle-on-a-lake-theme

Replace the theme gallery preview with the newer desktop screenshot
from the theme repo (shmall03/omarchy-castle-on-a-lake-theme@654a89b),
resized to the standard 1200px 16:9 webp. Follow-up to omacom-io#17.
@omarchybot

Copy link
Copy Markdown
Collaborator

Reviewed. Nothing blocking, and the provenance in your description checks out exactly.

Running the conversion the page documents — magick preview.jpg -strip -resize '1200>' -quality 80 — on preview.jpg from shmall03/omarchy-castle-on-a-lake-theme at 654a89b reproduces the committed file byte for byte: SHA-256 cf9873ca3a5a77f49c2553d6da39c54392d406f5e8304266e57dbd42eb368cc9, 1200x675, 42,850 bytes. Same dimensions as the image it replaces and as 92 of the 124 previews on the page, so nothing in the grid moves.

It is also the better image on its own terms. The old one was the bare wallpaper; this is an actual desktop with the bar and a terminal reading "OMARCHY QUATTRO / CASTLE ON A LAKE", which is what the rest of the gallery shows. Same theme — same castle-and-lake painting behind it.

The commit touches that one file and nothing else, and no HTML change is needed. Merges cleanly alongside #41 through #47.

Tests: none run, and there are none to run — omarchy-site has no test suite or link checker, and its only workflows are GitHub's own Copilot reviewer, CodeQL and Pages deploy.

Second opinion from codex at xhigh: it found no defect either. It reached the same byte size and SHA-256 for the new file independently and confirmed the diff contains nothing but that one blob; its independence is not currently guaranteed. The byte-for-byte provenance check above is mine — codex had no network access to fetch the source image.

Waiting on the maintainer. Not merged and not approved; that call is theirs.

This was referenced Aug 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants