diff options
| author | Craig Jennings <c@cjennings.net> | 2026-06-10 00:12:32 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-06-10 00:12:32 -0500 |
| commit | 4f4ceb3360e56ea2b1fb6f303c645a0b2205483c (patch) | |
| tree | 31fbe8d4f3cfb91d85bcf8fecc80271fdbd7aa22 /docs | |
| parent | 4211f625534d87a263d2d6b3dbe5ff5bc4093720 (diff) | |
| download | dotemacs-4f4ceb3360e56ea2b1fb6f303c645a0b2205483c.tar.gz dotemacs-4f4ceb3360e56ea2b1fb6f303c645a0b2205483c.zip | |
chore(todo): mark color-families spec Ready, break out build tasks
Craig confirmed the spec Ready. The six phases become tracked tasks under the color-families parent — family model and sort cores, strip rendering, the count control, the ground strip plus ramp-panel retirement, and the warnings/seeding/export/README close-out — all :solo:. Three manual aesthetic checks (hue grouping matches the eye, regenerate-replace reads as deliberate, removed-step references read as "(gone)") go under the Manual testing parent.
Diffstat (limited to 'docs')
| -rw-r--r-- | docs/theme-studio-color-families-spec.org | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/docs/theme-studio-color-families-spec.org b/docs/theme-studio-color-families-spec.org index a844d551..ce3b7a9f 100644 --- a/docs/theme-studio-color-families-spec.org +++ b/docs/theme-studio-color-families-spec.org @@ -3,7 +3,7 @@ #+DATE: 2026-06-09 * Metadata -| Status | draft — review incorporated (Codex, 2026-06-09); pivoted to hex grouping | +| Status | Ready (Craig confirmed 2026-06-10); review incorporated, hex grouping | | Owner | Craig | | Reviewer | Codex | | Related | [[file:../todo.org][todo.org: theme-studio color families]] | @@ -196,3 +196,7 @@ Only modified and rejected review recommendations are listed; everything else fr - What changed or was recommended: folded the Codex review and Craig's three clarifications. Resolved both open decisions (flat persistence; remove the standalone ramp panel). Pivoted the whole model from name-derived families to hex-derived families over a still-flat, still-editable palette — which designs out the name-grammar/import-inference blocker and the chip-ownership blocker. Pinned the ground strip to the bg/p assignments, the removed-step references to a visible "(gone)", n=0 handled without =ramp()=, the 0.02 neutral and 25° hue-gap thresholds, the sort tie-breakers, the warnings exemption, and the unchanged package seeding. See Review dispositions for the two reframed findings. - Why: Craig's "group by hex, never by name; rename anything freely" both answers the review's hardest questions and simplifies the design — the palette no longer changes ownership, so most of the missing contracts disappear. - Artifacts: review file deleted; Review dispositions section; this entry. Remaining fuzzy area is the hue-clustering threshold (pinned default, vNext tuning). +** 2026-06-10 Wed @ 00:03:39 -0500 — Codex — reviewer +- What changed or was recommended: re-reviewed the updated spec and found it implementation-ready. No new blocking review notes; the prior blockers are resolved by the hex-derived grouping model, accepted flat-palette persistence, accepted ramp-panel removal, explicit ground-strip source, retained per-chip flat-palette ownership, removed-step "(gone)" policy, pinned clustering/sort thresholds, and README/test acceptance criteria. +- Why: the updated design now gives the implementer stable behavior for grouping, regeneration, references, ground colors, import/export, and UI ownership while fitting the current =app-core.js= / =app.js= split. +- Artifacts: no new review file; this Ready verification entry. |
