Drop the unreferenced pre-redesign screenshot - #171
Merged
Conversation
static/images/site-screenshot.png has never been referenced: no component, no post, no README entry, no manifest. `git log -S"site-screenshot"` finds the string in no text file in the repo's history, so it arrived with the template's sample images and was never wired up. It is not a version record worth relocating either. It captures the same page state now kept as static/versioning/v0.2.png -- same header, headline, store badges and phone, differing only in capture scale. Moving it to static/versioning/ would file a second copy of v0.2 under a name the vX.Y.png convention has no slot for. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
static/images/site-screenshot.png(2774x1686, 163 KB) is referenced nowhere and duplicates a record we already keep, so this removes it.It is genuinely unreferenced
Checked
src/,static/,README.md,frontmatter.json,static/favicons/site.webmanifest,static/_redirects,scripts/,functions/, and blog post frontmatter:/images/path referenced anywhere in the repo yields 32 paths; this file is not among them.import.meta.globcalls target.mdroutes.git log -S"site-screenshot" --allreturns zero commits: the string has never appeared in a tracked text file in this repo's history. It arrived ind603511 image samplesalongsidesample-image.png(deleted back ince21ba8) as one of the template's sample images, and was never wired up.The grep method was validated against
site-preview.png, which correctly returns its 4 real references.Why delete rather than move to
static/versioning/The first instinct was to preserve it as a historical record, but it is not one. It is not the v0.1-era site (v0.1 shows an "Early Access" button and three phones). It captures the same page state as
static/versioning/v0.2.png, added in #170 — same header, headline, store badges, wave, single phone, and "rewarding journey" section, differing only in capture scale (v0.2 is ~4% more zoomed).So filing it under
static/versioning/would mean a second copy of v0.2, at a resolution that breaks the directory's uniform 1941x1180 sizing, under a name thevX.Y.pngconvention has no slot for. The version is already recorded; this copy adds nothing.Note
The file is currently served publicly at
/images/site-screenshot.png. Nothing on the site ever linked it, but an external deep link would now 404.Verification
npx vite buildpasses; build output containssite-preview.pngand nosite-screenshot.png.🤖 Generated with Claude Code