Skip to content

feat: drop the chart tokens; charts use the existing colors - #20

Merged
zyx1121 merged 1 commit into
mainfrom
feat/drop-chart-tokens
Oct 7, 2026
Merged

zyx1121 merged 1 commit into
mainfrom
feat/drop-chart-tokens

Conversation

@zyx1121

@zyx1121 zyx1121 commented Oct 7, 2026

Copy link
Copy Markdown
Member

Summary

  • Remove chart-1 to chart-5 (a gray ramp no component used) from the base and the site.
  • DESIGN.md: charts use primary (main series), muted-foreground (comparison) and success / destructive / warning (series with a meaning, such as portal's income and expense lines, which today hard-code #22c55e / #ef4444). More than two series → small multiples or a table.
  • tokens:check rejects *-chart-*.

Test plan

  • Sandbox: all checks, build; tokens:check fails on fill-chart-1
  • CI green

🤖 Generated with Claude Code

chart-1 to chart-5 were a gray ramp nothing used. Charts take primary
for the main series, muted-foreground for a comparison, and the status
colors for series with a meaning; more than two series means split
the chart. tokens:check rejects chart-*.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@zyx1121
zyx1121 merged commit 25e21ab into main Oct 7, 2026
2 checks passed
@zyx1121
zyx1121 deleted the feat/drop-chart-tokens branch October 7, 2026 06:31
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.

1 participant