Skip to content

fix(theme): align color tone role mappings#21

Merged
artiphishle merged 6 commits intomainfrom
chatgpt/align-color-tone-role-mappings
May 4, 2026
Merged

fix(theme): align color tone role mappings#21
artiphishle merged 6 commits intomainfrom
chatgpt/align-color-tone-role-mappings

Conversation

@artiphishle
Copy link
Copy Markdown
Contributor

Summary

Closes #18.

Audits and tightens Surface color-tone role palette mappings against the app-facing background/foreground lane direction without adding a ZORA dependency.

Behavior changes:

  • fluorescent now uses obsidian background/surface roles and fluorescent action roles.
  • pastel keeps pastel background/surface roles but moves foreground/action roles to jewel-like roles.
  • earth keeps earth background/surface roles but moves primary/secondary action roles to mineral-like roles.

Tests added for semantic behavior instead of exact snapshots:

  • obsidian/fluorescent dark surfaces remain low-lightness
  • fluorescent action colors remain higher chroma than neutral actions
  • pastel backgrounds remain lower chroma than foreground/action colors
  • earth action chroma aligns with mineral-like chroma
  • generated palettes still emit required semantic roles for every ColorTone

Scope boundaries

No ZORA dependency, no ColorTone rename, no new dependencies, and no full engine rewrite.

Verification

Not run locally from this environment. GitHub CI should run on the PR.

@artiphishle artiphishle merged commit 7886a47 into main May 4, 2026
1 check passed
@artiphishle artiphishle deleted the chatgpt/align-color-tone-role-mappings branch May 4, 2026 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[2h] 07 — Audit Surface colorTone mappings against ZORA tone lanes

1 participant