fix: use yellow text color for selected command palette row

sassman · claude · sassman · commit 9b728508c9e8 · 2025-11-21T23:48:57.000+01:00
Changed selected row foreground color from theme.selected_fg to theme.active_fg (yellow) for consistency with build log error lines. Updated colors: - View model: Changed selected row fg_color to theme.active_fg - View: Changed indicator/shortcut column to theme.active_fg when selected - View: Changed category column to theme.active_fg when selected This makes the selected/active command palette row use yellow text, matching the visual style of error lines in the build log viewer for better consistency across the UI. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
diff --git a/crates/gh-pr-tui/src/view_models/command_palette.rs b/crates/gh-pr-tui/src/view_models/command_palette.rs
@@ -96,7 +96,8 @@ impl CommandPaletteViewModel {
 
                 // Colors
                 let (fg_color, bg_color) = if is_selected {
-                    (theme.selected_fg, theme.selected_bg)
+                    // Use yellow for selected row (same as error lines in build log)
+                    (theme.active_fg, theme.selected_bg)
                 } else {
                     (theme.text_primary, Color::Reset)
                 };
diff --git a/crates/gh-pr-tui/src/views/command_palette.rs b/crates/gh-pr-tui/src/views/command_palette.rs
@@ -108,7 +108,8 @@ pub fn render_command_palette(f: &mut Frame, area: Rect, app: &App) {
         let rows = vm.visible_rows.iter().map(|row_vm| {
             let indicator_shortcut_style = Style::default()
                 .fg(if row_vm.is_selected {
-                    theme.accent_primary
+                    // Use yellow for selected row (consistent with title)
+                    theme.active_fg
                 } else {
                     theme.text_muted
                 })
@@ -127,7 +128,8 @@ pub fn render_command_palette(f: &mut Frame, area: Rect, app: &App) {
                 });
 
             let category_style = Style::default().fg(if row_vm.is_selected {
-                theme.text_secondary
+                // Use yellow for selected row (consistent with title)
+                theme.active_fg
             } else {
                 theme.text_muted
             });
