fix: When the terminal is insufficient to display all the commands, the cmdbar_bg configuration color does not fully take effect

Author: wugeer

CHANGELOG.md

@@ -11,6 +11,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 * use default shell instead of bash on Unix-like OS [[@yerke](https://github.com/yerke)] ([#2343](https://github.com/extrawurst/gitui/pull/2343))
 
 ### Fixes
+* When the terminal is insufficient to display all the commands, the cmdbar_bg configuration color does not fully take effect. ([#2347](https://github.com/extrawurst/gitui/issues/2347))
 * respect env vars like `GIT_CONFIG_GLOBAL` ([#2298](https://github.com/extrawurst/gitui/issues/2298))
 * Set `CREATE_NO_WINDOW` flag when executing Git hooks on Windows ([#2371](https://github.com/extrawurst/gitui/pull/2371))
 

src/cmdbar.rs

@@ -20,7 +20,6 @@ enum DrawListEntry {
 struct Command {
 	txt: String,
 	enabled: bool,
-	line: usize,
 }
 
 /// helper to be used while drawing
@@ -106,7 +105,6 @@ impl CommandBar {
 			self.draw_list.push(DrawListEntry::Command(Command {
 				txt: c.text.name.to_string(),
 				enabled: c.enabled,
-				line: lines.saturating_sub(1) as usize,
 			}));
 		}
 
@@ -157,9 +155,7 @@ impl CommandBar {
 							DrawListEntry::Command(c) => {
 								Span::styled(
 									Cow::from(c.txt.as_str()),
-									self.theme.commandbar(
-										c.enabled, c.line,
-									),
+									self.theme.commandbar(c.enabled),
 								)
 							}
 							DrawListEntry::LineBreak => {

src/ui/style.rs

@@ -16,7 +16,6 @@ pub struct Theme {
 	selection_bg: Color,
 	selection_fg: Color,
 	cmdbar_bg: Color,
-	cmdbar_extra_lines_bg: Color,
 	disabled_fg: Color,
 	diff_line_add: Color,
 	diff_line_delete: Color,
@@ -204,17 +203,13 @@ impl Theme {
 		self.line_break.clone()
 	}
 
-	pub fn commandbar(&self, enabled: bool, line: usize) -> Style {
+	pub fn commandbar(&self, enabled: bool) -> Style {
 		if enabled {
 			Style::default().fg(self.command_fg)
 		} else {
 			Style::default().fg(self.disabled_fg)
 		}
-		.bg(if line == 0 {
-			self.cmdbar_bg
-		} else {
-			self.cmdbar_extra_lines_bg
-		})
+		.bg(self.cmdbar_bg)
 	}
 
 	pub fn commit_hash(&self, selected: bool) -> Style {
@@ -335,7 +330,6 @@ impl Default for Theme {
 			selection_bg: Color::Blue,
 			selection_fg: Color::White,
 			cmdbar_bg: Color::Blue,
-			cmdbar_extra_lines_bg: Color::Blue,
 			disabled_fg: Color::DarkGray,
 			diff_line_add: Color::Green,
 			diff_line_delete: Color::Red,