Skip to content

Feat(csc-ui): Refine dark mode subtle tint contrast and identity - #262

Merged
razorfever merged 1 commit into
mainfrom
development
Sep 3, 2026
Merged

Feat(csc-ui): Refine dark mode subtle tint contrast and identity#262
razorfever merged 1 commit into
mainfrom
development

Conversation

@villeerikssoncsc

Copy link
Copy Markdown
Contributor

Adjusts semantic token mappings for subtle tints and their foreground inks. Primary, secondary, info, error, and link subtle tints move to *-700 to ensure visual distinction from the slate-800 page background. Accent, success, and warning subtle tints remain *-800 to preserve WCAG AA contrast for their on-*-subtle inks. All on-*-subtle inks are updated from *-50 (near-white) to a more vivid *-200 to maintain color identity, especially for status indicators like alert icons.

Adjusts semantic token mappings for subtle tints and their foreground inks. Primary, secondary, info, error, and link subtle tints move to `*-700` to ensure visual distinction from the `slate-800` page background. Accent, success, and warning subtle tints remain `*-800` to preserve WCAG AA contrast for their `on-*-subtle` inks. All `on-*-subtle` inks are updated from `*-50` (near-white) to a more vivid `*-200` to maintain color identity, especially for status indicators like alert icons.
@razorfever
razorfever merged commit 25fa3ae into main Sep 3, 2026
2 checks passed
villeerikssoncsc added a commit that referenced this pull request Sep 3, 2026
…quire a new changeset per PR

PR #262 (dark-mode subtle ink back to `*-200`) released nothing: it only
edited `.changeset/dark-mode-vivid-retune.md`, which the alpha.8 "Version
packages" PR had already consumed. In changesets pre mode a consumed
changeset stays on disk but is listed in `.changeset/pre.json` and skipped
by the release plan, so amending it is inert. The changeset check passed
because it accepted any `.changeset/*.md` in the diff.

- Add `dark-mode-subtle-ink.md` (patch) so the next merge to main opens a
  Version packages PR for the change already on main.
- `changeset-check.yml` now requires an ADDED changeset file
  (`--diff-filter=A`) and explains why an edited one does not count.
  Replayed locally: blocks #262, passes #260.
- CLAUDE.md release notes state the rule.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
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.

2 participants