One red on the logo page: the mark's #D03C4A everywhere - #205
Merged
Conversation
The logo page showed several reds next to the one it names. The mascot brand copies had been shifted towards #D03C4A but did not land on it — the sheep's flat fill sat at #C93A48, the sloth's ring a crimson mean of #BB3C44 — and the Colours table listed a second, darker red (#A83232) that existed only as the hover shade of links and buttons. - Recolour the three mascots from their originals so that the flat red of each drawing lands on #D03C4A exactly; shading and highlights are darker and lighter shades of the same red, outlines and the sloth's cream fur are untouched (the hue window is narrow enough now to leave the fur out, which the earlier pass did not quite manage). - Collapse the brand tokens to the one red: --vp-c-brand-2 and the button hover tokens now equal --vp-c-brand-1, and the link hover colour equals the link colour in both themes. Hover feedback stays the underline a link already gains. - The Colours section lists one red and white, and the mascot table rows carry the measured source reds (#A62219 for the sheep, ≈#F5011D for the sloth). Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KoQ7aC6p8nvEkjNho6RDRR
There was a problem hiding this comment.
🟡 Changes recommended
docs/resources/logo.md now claims the site uses only #D03C4A, but the dark theme still uses a lifted link red (#e8707c) in CSS, making the palette description inaccurate.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
This PR standardizes the documentation site’s brand palette so the logo page (and the site theme tokens it references) no longer implies multiple “brand reds”, aligning hover/secondary brand tokens with the primary brand red #D03C4A.
Changes:
- Updated VitePress/theme brand tokens so hover/secondary brand variables also resolve to
#D03C4A. - Updated
resources/logo.mdcopy and the Colours table to describe a single brand red and corrected the recorded source reds for mascots.
File summaries
| File | Description |
|---|---|
| docs/resources/logo.md | Updates mascot-red descriptions and simplifies the documented palette to a single brand red. |
| docs/.vitepress/theme/style.css | Sets all brand tokens (including hover) to #d03c4a and aligns link-hover token behavior accordingly. |
Review details
- Files reviewed: 2/8 changed files
- Comments generated: 1
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Comment on lines
+176
to
+178
| One red and white. The mark, every mascot above and this site use that red | ||
| and nothing else. There is no second shade for interface states: a link under | ||
| a cursor gains an underline, not a darker red, and a button keeps its colour. |
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.
The logo page showed several reds next to the one it names. This makes
#D03C4A— the red oflogo.png— the only red on it.What was off, measured on the files in the repository
#D03C4Aexactly.#C93A48, the dinosaur averaged#C23C49, the sloth a blotchy crimson around#BB3C44. The earlier recolouring had only landed near the target.#A83232, which existed only as the hover shade of links and buttons in the site's CSS.Changes
#D03C4Aexactly; shading and highlights become darker and lighter shades of the same red. Outlines and the sloth's cream fur are untouched — the hue window is narrower than in the first pass, which is what keeps the fur cream instead of pink.docs/.vitepress/theme/style.css: every brand token is the one red, hover tokens included (--vp-c-brand-2, the button hover pair, and the link hover colour in both themes). Links still show their hover through the underline they already gained; buttons keep their colour under the cursor. The dark theme keeps its lifted link red#e8707cfor contrast, with the hover now the same tone.docs/resources/logo.md: Colours lists one red and white. The mascots' measured source reds are corrected (sheep#A62219instead of#AB2319, sloth ≈#F5011D).Verified
npm run checkpasses (check:samplesskipped without a samples checkout, as documented in AGENTS.md). The built page was checked by screenshot.Note: the hover change is site-wide, not only on this page — otherwise the colour table would have had to keep naming two reds.
🤖 Generated with Claude Code
https://claude.ai/code/session_01KoQ7aC6p8nvEkjNho6RDRR
Generated by Claude Code