From 4981037e99a322ee9cf29bc8730cb64d263fa00b Mon Sep 17 00:00:00 2001 From: Eric Traut Date: Wed, 23 Sep 2026 03:44:30 +0000 Subject: [PATCH] Use secondary text styling for the transcript footer shortcut hint (#47441) Apply `secondary_text_style()` to the shortcut hint separator and label so they use a readable foreground color instead of terminal-dependent dimming. GitOrigin-RevId: 6cd9ced75deca9353c1224d3e5fefc6c4ad6ae8f --- codex-rs/tui/src/bottom_pane/chat_composer/footer_state.rs | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/codex-rs/tui/src/bottom_pane/chat_composer/footer_state.rs b/codex-rs/tui/src/bottom_pane/chat_composer/footer_state.rs index fe4a6dc186..c734ccad7d 100644 --- a/codex-rs/tui/src/bottom_pane/chat_composer/footer_state.rs +++ b/codex-rs/tui/src/bottom_pane/chat_composer/footer_state.rs @@ -19,7 +19,7 @@ use super::super::footer::shows_passive_footer_line; use super::ActivePopup; use ratatui::buffer::Buffer; use ratatui::layout::Rect; -use ratatui::style::Stylize; +use ratatui::style::Styled; use ratatui::text::Line; use ratatui::text::Text; use ratatui::widgets::Paragraph; @@ -30,6 +30,7 @@ use crate::bottom_pane::footer::FooterMode; use crate::bottom_pane::footer::GoalStatusIndicator; use crate::key_hint::KeyBinding; use crate::key_hint::ShortcutHint; +use crate::style::secondary_text_style; use std::time::Duration; /// Resolved rectangles shared by painting and cursor placement, without retained layout state. @@ -175,10 +176,10 @@ impl super::ChatComposer { { let mut shortcuts = Line::default(); if line.width() > 0 { - shortcuts.push_span(" · ".dim()); + shortcuts.push_span(" · ".set_style(secondary_text_style())); } shortcuts.extend(key.spans()); - shortcuts.push_span(" shortcuts".dim()); + shortcuts.push_span(" shortcuts".set_style(secondary_text_style())); if line.width() + shortcuts.width() <= usize::from(hint_area.width) { line.extend(shortcuts.spans); }