Skip to content

refactor(tui): centralize UI colors in Theme tokens#114

Merged
samzong merged 1 commit into
mainfrom
feat/tui-theme-tokens
Jul 19, 2026
Merged

refactor(tui): centralize UI colors in Theme tokens#114
samzong merged 1 commit into
mainfrom
feat/tui-theme-tokens

Conversation

@samzong

@samzong samzong commented Jul 19, 2026

Copy link
Copy Markdown
Owner

What's changed?

  • Add src/tui/theme.rs with a THEME token map for TUI colors
  • Route search, popups, usage, viewing, and shared UI styles through theme tokens instead of ad-hoc Color::* literals
  • Update existing UI style assertions to assert against THEME values

Why

  • Color choices were scattered across render modules, which made palette changes error-prone and inconsistent
  • A single token source keeps borders, selection, accents, roles, and usage series aligned without changing the current look

Verification

  • cargo test -q --lib tui::ui — 9 passed
  • cargo clippy -q --lib -- -D warnings — clean
  • pre-ship --committed round 1/2 — MUST-FIX: 0

Signed-off-by: samzong <samzong.lu@gmail.com>
@samzong
samzong merged commit 22625bf into main Jul 19, 2026
1 check passed
@samzong
samzong deleted the feat/tui-theme-tokens branch July 19, 2026 17:18
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