Repository navigation
Adopt frame-inspired design discipline - #47
Merged
Merged
Conversation
- Add a WCAG contrast gate for the shipped OKLCH palette: parses the light and dark custom-property blocks in src/styles.css, resolves semantic alias chains, and converts OKLCH to sRGB with self-contained color math. All measured ratios pass (ink >= 14.4:1, muted >= 6.9:1, danger >= 4.7:1). - Reconcile DESIGN.md frontmatter and color/shadow prose with the shipped shadcn-neutral palette (dark values canonical, light noted inline). - Introduce a two-tier motion vocabulary: 90ms --interaction-motion for small-control hover/press tints, 180ms --state-motion for layout and elevation. Both collapse under prefers-reduced-motion. - Render structural separators at 0.5px on 2dppx+ displays via a --hairline token, restored to 1px under prefers-contrast: more. - Reinforce the selected model row with a 2px leading bar (::before, so the focus ring survives) beside the existing background and check signals. - Add an Active Edge Light: a 0.5px inset top highlight on the focused composer and open popovers/menus only, documented in DESIGN.md. Fixes the undefined --shadow-popover reference on the capsule install dialog. - Restore focus to the composer (or rail header/reopen control) when the focused element inside the shell is removed by a canvas update, without stealing focus that legitimately moved elsewhere; covered by jsdom tests. - Fix preexisting lint debt: unused imports in workspace-controller and a descending-specificity warning for .capsule-surface. Claude-Session: https://claude.ai/code/session_01VNmFXmRMNV98Hyd3xogUuK
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.
Selective adoption from the 66HEX/frame design evaluation (ideas only - frame is GPL-3.0, no code was viewed or copied; all values chosen independently). Wholesale adoption was rejected: frame's signature aesthetics conflict with named DESIGN.md rules.
--interaction-motion: 90msfor hover/press tints,--state-motion: 180msstays for layout/elevation--hairlinetoken, 0.5px on Retina, restored to 1px under prefers-contrast: more; structural separators only.capsule-install-dialogreferenced an undefined--shadow-popover(shadow silently absent); also clears lint debt that was red on mainEvidence: lint/typecheck clean, 930 unit tests, full Playwright suite 94/94 on this branch, bundle gates pass (workspace CSS at 19,595/20,480 gzip - decoded headroom is thin at ~0.5 KiB, noted for future CSS work).
🤖 Generated with Claude Code
https://claude.ai/code/session_01VNmFXmRMNV98Hyd3xogUuK