Docs: moodboard-identity glow-up for the home page (draft) - #201
Draft
testtest126 wants to merge 1 commit into
Draft
Docs: moodboard-identity glow-up for the home page (draft)#201testtest126 wants to merge 1 commit into
testtest126 wants to merge 1 commit into
Conversation
docs/index.html invented its own green/buff palette instead of the project's actual identity (brass #8A6216 light / gold #E7BA5C dark, cream #F6EFE2 / walnut #171109 surfaces — see docs/design/moodboard/MOODBOARD.md in the LULL repo, canonical source for both projects' visual identity). - Both theme paths (the `prefers-color-scheme` media query, which is what real visitors get with no toggle on this page, and the `data-theme` attribute override) now read the same brass/gold/ cream/walnut tokens. - The hero is a deliberately dark, walnut-toned surface in both page themes (same idea as the original "deliberately green in both themes" comment) — so it always pairs with gold, the dark-surface half of the accent, never brass. Introduced named --hero-* tokens in place of the hardcoded hex the hero rules used before. - Dark mode's accent-on-fill label color is #1F1F1F, not white — the app's own contrast rule (`BrandCTALabelColor`): white on gold fails WCAG. Verified all new text/background pairs are >=4.5:1 (most 5.5-15:1); decorative dividers stay at the original's low contrast, same as before. - Removed the unused --green*/--buff* tokens the previous palette left dead in :root (never read via var(), the hero rules hardcoded their own hex instead). - No copy, stats, layout, or structure changed — colors only. Scope: home page only (docs/index.html), per the task. architecture/ and the-fleet/ each run their own larger multi-skin theme engine with a much bigger token set that doesn't extend cleanly from this simple palette, and the-fleet/index.html is also mid-flight in open PR #200 (viewport-meta fix) — left both untouched to avoid destabilizing the former and colliding with the latter.
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
docs/index.html"How MateMate Works" home page (served attesttest126.github.io/chess/) was styling itself with an invented green/buff palette that has nothing to do with the project's actual visual identity. This recolors it to the real MateMate identity from the canonical moodboard (docs/design/moodboard/MOODBOARD.mdin the LULL repo, shared reference for both projects):#8A6216(light) / gold#E7BA5C(dark) — never default iOS blue, never an invented hue.#F6EFE2/ card#FFFDF8(light); walnut#171109/ card#251C11(dark).--hero-*tokens in place of the hardcoded hex the hero rules used before.#1F1F1F, not white — this is the app's ownBrandCTALabelColorrule (white on gold fails WCAG). All new text/background pairs verified ≥4.5:1 (most are 5.5–15:1); decorative dividers stay at the original's low contrast (~1.3:1), unchanged from before — they're non-text visual separators, not functional UI boundaries.--green*/--buff*tokens the old palette left dead in:root— never actually read viavar(), since the hero rules hardcoded their own hex instead.Scope
Home page only (
docs/index.html), per the task.architecture/andthe-fleet/each run their own larger multi-skin "theme engine" with a much bigger token set (cultural skins: arabic/indian/andalus/terminal/codex) that doesn't extend cleanly from this page's simple palette — recoloring either risked moving their existing baseline in ways I couldn't fully verify.the-fleet/index.htmlis also mid-flight in open PR #200 (viewport-meta fix), so I left it untouched entirely to avoid any collision.Constraints held
Iowan Old Style/Palatino/Georgia serif + system sans, both already in place, unchanged) — no external fonts/scripts/CDNs/trackers added.prefers-color-schemekept, with both a deliberate light and a designed warm dark (not a naive invert) — verified both the real media-query path (matchMedia('(prefers-color-scheme: dark)')) and thedata-themeattribute override resolve to the same tokens.:focus-visibleoutline uses--accent) is 4.78:1 (light) / 10.33:1 (dark), well past the 3:1 UI-component threshold.:focus-visible{outline:2px solid var(--accent)}, just recolored).Verification
Served
docs/locally and captured after-screenshots via a CDP-driven headless Chrome (Emulation.setDeviceMetricsOverride+setEmulatedMedia, full-page capture) — plain--window-sizeon this Chrome build silently floors narrow widths to 500px, so I drove the DevTools protocol directly instead of trusting the CLI flag for the mobile shot. Confirmed:prefers-color-scheme: light): hero + cards + facts + footer all render in the new brass/cream palette.prefers-color-scheme: dark, verified via real media-query emulation, not just the manual attribute): full warm walnut/gold treatment, not an inverted light theme.@media (max-width:760px)rules unchanged and still correct at this width.Screenshots saved locally (not attached here — reported as absolute paths to Yakiv directly, since gh CLI has no way to embed local images in a PR body/comment).
🤖 Generated with Claude Code