[Docs] README: refresh the hero screenshot for the Tempered theme - #139
Open
vedanshujain wants to merge 1 commit into
Open
[Docs] README: refresh the hero screenshot for the Tempered theme#139vedanshujain wants to merge 1 commit into
vedanshujain wants to merge 1 commit into
Conversation
The committed hero still showed the pre-theme storefront (default link styling, bordered cards). Replace it with the current design: the home page's display headline and tagline beside the inventory tape, with live SKU / price / stock rows served by the commerce service. Shot from the deployed reference site at 1280x600, light theme. Framing note: this is the HOME page rather than the product grid the old image used. The grid renders each product's description, and the seeded descriptions still carry copy we do not want as the project's first impression — one of them markets an availability guarantee we deliberately avoid claiming, and another explains our own CMS internals to shoppers. The home page shows the theme's character without that copy. When the storefront copy pass lands, the grid becomes a fair candidate again. The alt text and caption are updated to match: the image no longer shows titles and descriptions, and it is no longer a local run.
|
Caution The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased. |
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
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.
Summary
The committed README hero still showed the pre-theme storefront — default blue link
styling, bordered cards. This replaces it with the current Tempered design and updates the
alt text and caption to match.
New shot: the home page's display headline and tagline beside the inventory tape, with live
SKU / price / stock rows served by the commerce service. 1280×600, light theme, taken from
the deployed reference site.
Framing disclosure
This is the home page, whereas the old image used the product grid. That is a
deliberate change, not a convenience.
The grid renders each product's description, and the seeded descriptions are not something
we want as the project's first impression:
Both are already scheduled to be rewritten in the storefront copy pass. The home page shows
the theme's character — type scale, inventory tape, mono numerals — without carrying that
copy into the README. When the copy pass lands, the grid is a fair candidate again.
Worth noting the old image had the same problem: that copy is visible in the hero on
maintoday. This PR removes it from the README as a side effect.Alt text / caption
Both were describing the old image and are now inaccurate, so both changed:
headline plus a SKU/price/stock table.
the caption now says the quick start gives you the same thing locally, which is the claim
that is actually true.
Verification
./docs/storefront.pngunchanged.🤖 Generated with Claude Code
https://claude.ai/code/session_01HoB4yKtw1hgN5sEvDfupdt