refactor(web): app-wide CSS design tokens in styles.css (normalized palette) #75
No reviewers
Labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
Cloonar/deckle!75
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "afk/72"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Introduces the
:rootdesign-token layer agreed in #72 and migrates all ofweb/src/styles.css(~3,200 lines) onto it, normalizing near-duplicate values in the same pass. Groundwork PR 1 of 2 for the unified module docheader.What's in the token layer
--ink(#1c1f26 — the near-black that is both default text and the dark chrome), alpha tints, scrim + 4 shadows, radii (4/6/8/999px), and a 9-step type scale (0.65–1.3rem).--color-text/-muted/-faint,--color-heading,--color-border/-subtle,--color-accent/-strong,--color-focus,--color-danger/-strong,--color-primary/-strong, dark-chrome tokens (--chrome-bg/-deep/-raised, menu/hover/selected fills), and the six--state-*badge tokens the docheader rework will reuse.Normalizations (intentional visual changes)
#52525b→#3f3f46,#27272a→#18181b,#fcfcfd→#fafafaVerification
var()back to literals and diffed against the pre-migration file — exactly 75 changed declaration lines, every one on the sanctioned list above; zero unsanctioned value changes, zero undefined tokens, no structural drift.:root(onlyborder-radius: 50%/0idioms).npm run build✓,npm test726/726 ✓,go vet✓,go test ./...all packages ✓,go build ./cmd/harness✓.scaffold/site/AGENTS.mdunaffected (UI-only, per issue).Closes #72
🤖 Generated with Claude Code
Validation verdict: PASS
ci / build-and-test✓ success (5m7s, run #65) — coversnpm run build,npm test,go vet,go test ./...,go build.refactor(web): …Closes #72present and valid.web/src/styles.css, +674/−532), matching issue #72 (CSS-only, app-wide token layer).mainand the PR — only declaration values changed (literals →var()).:root(only the sanctionedborder-radius: 50%/0idioms).main, single commit, no conflicts.Accepted risk (per #72): the normalizations intentionally shift some values slightly and there is no screenshot infra in this PR, so the visual eyeball click-through of all modules remains a human review step — outside the lander's scope.