From b370d6f61870d26679560171b126ec81ec5ea8a8 Mon Sep 17 00:00:00 2001 From: liushiao Date: Wed, 20 May 2026 17:10:47 +0800 Subject: [PATCH] refactor(tui): use Magenta + DarkGray for Terminal theme accents Adopt Gemini code-assist review on PR #1831: - mode_plan: Yellow -> Magenta. Plan chip now contrasts with status_warning (still Yellow) so the two never visually collide in the status row. - status_ready: Reset -> DarkGray. Ready chip now reads as a distinct subdued accent instead of blending into body text (which also resolves to Reset on this theme). No surface change otherwise -- backgrounds and body text still use Color::Reset to inherit the host terminal's color scheme. --- crates/tui/src/palette.rs | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/crates/tui/src/palette.rs b/crates/tui/src/palette.rs index 7de9d730..d0d8e93b 100644 --- a/crates/tui/src/palette.rs +++ b/crates/tui/src/palette.rs @@ -846,8 +846,13 @@ pub const TERMINAL_UI_THEME: UiTheme = UiTheme { footer_bg: Color::Reset, mode_agent: Color::Blue, mode_yolo: Color::Red, - mode_plan: Color::Yellow, - status_ready: Color::Reset, + // Magenta keeps Plan visually distinct from `status_warning` (yellow) + // so the mode indicator and warning chip don't collide on themes that + // render both in the status row. + mode_plan: Color::Magenta, + // DarkGray gives "Ready" a low-contrast but still distinguishable hue + // versus default body text (which is `Color::Reset` on this theme). + status_ready: Color::DarkGray, status_working: Color::Cyan, status_warning: Color::Yellow, text_dim: Color::Reset,