diff options
| author | Craig Jennings <c@cjennings.net> | 2026-06-09 17:55:44 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-06-09 17:55:44 -0500 |
| commit | 4d6b04b9a4ccccca0b5b36afea3d9fab6594ce72 (patch) | |
| tree | 2fb79277efbb3859afc0e4c63d5c81759d66cb59 /tests | |
| parent | 3587d251f5ba82b57a4f8c4184d2b07cd3c66d69 (diff) | |
| download | dotemacs-4d6b04b9a4ccccca0b5b36afea3d9fab6594ce72.tar.gz dotemacs-4d6b04b9a4ccccca0b5b36afea3d9fab6594ce72.zip | |
docs(theme-studio): fold Codex review into palette-ramps spec
Resolved both open decisions. The contrast target is WCAG AA by default, with AAA selectable and APCA shown as a diagnostic only. The v1 foreground set is the distinct syntax-assignment hexes plus the default foreground, with locked background-only roles excluded.
Pinned what the review flagged as underspecified: a closed five-face covered set (region, hl-line, highlight, lazy-highlight, isearch), ramp defaults and palette-insertion rules (n/stepL/chromaEase, naming, collisions, clamp display), and explicit-state structured-error contracts for ramp, fgSetFor, floor, and lMax. Package and non-overlay UI cells stay single-pair in v1.
Closed the v1 face set rather than keeping the review's open-ended "any face the buffer renders text over" clause, since an open set reintroduces the gap the foreground-set decision exists to close. Kept throwing for genuine programmer error while returning structured results for bad user input, matching colormath.js.
The spec is implementation-ready. The six implementation tasks get created once Craig confirms the go. Review file consumed and deleted.
Diffstat (limited to 'tests')
0 files changed, 0 insertions, 0 deletions