valknar/llmx
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

tui: fix off-by-16 in terminal_palette (#4967)

Browse Source 
caught by a bad refactor in #4957
This commit is contained in:
Jeremy Rose
2025-10-08 14:57:32 -07:00
committed by GitHub
parent 3c5e12e2a4
commit f4bc03d7c0
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
codex-rs/tui/src/terminal_palette.rs
View File

@@ -11,7 +11,7 @@ pub fn best_color(target: (u8, u8, u8)) -> Color {
#[allow(clippy::disallowed_methods)] #[allow(clippy::disallowed_methods)]
Color::Rgb(r, g, b) Color::Rgb(r, g, b)
} else if color_level.has_256 } else if color_level.has_256
&& let Some((i, _)) = xterm_fixed_colors().enumerate().min_by(|(_, a), (_, b)| { && let Some((i, _)) = xterm_fixed_colors().min_by(|(_, a), (_, b)| {
perceptual_distance(*a, target) perceptual_distance(*a, target)
.partial_cmp(&perceptual_distance(*b, target)) .partial_cmp(&perceptual_distance(*b, target))
.unwrap_or(std::cmp::Ordering::Equal) .unwrap_or(std::cmp::Ordering::Equal)
@@ -236,8 +236,8 @@ mod imp {
} }
/// The subset of Xterm colors that are usually consistent across terminals. /// The subset of Xterm colors that are usually consistent across terminals.
pub fn xterm_fixed_colors() -> impl Iterator<Item = (u8, u8, u8)> { fn xterm_fixed_colors() -> impl Iterator<Item = (usize, (u8, u8, u8))> {
XTERM_COLORS.into_iter().skip(16) XTERM_COLORS.into_iter().enumerate().skip(16)
} }
// Xterm colors; derived from https://ss64.com/bash/syntax-colors.html // Xterm colors; derived from https://ss64.com/bash/syntax-colors.html