asp/keep
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix: Correctly calculate max value width for info table

Browse Source 
Co-authored-by: aider (openai/andrew/openrouter/deepseek/deepseek-chat-v3.1) <aider@aider.chat>
This commit is contained in:
Andrew Phillips
2025-09-08 17:51:07 -03:00
parent 12665032a3
commit 08f37697d8
1 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/modes/info.rs
View File

@@ -147,10 +147,12 @@ fn show_item(
.unwrap_or(0); .unwrap_or(0);
debug!("Max label width: {}", max_label_width); debug!("Max label width: {}", max_label_width);
// Total width used: 1 (left border) + (max_label_width + 2) + 1 (middle separator) + (value_width + 2) + 1 (right border) // Total width used: 1 (left border) + 1 (left padding) + max_label_width + 1 (right padding) +
// We want: max_label_width + value_width + 7 <= terminal_width // 1 (middle separator) + 1 (left padding) + value_width + 1 (right padding) + 1 (right border)
// So max value width is: terminal_width - max_label_width - 7 // = max_label_width + value_width + 8
let max_value_width = terminal_width.saturating_sub(max_label_width + 7); // We want: max_label_width + value_width + 8 <= terminal_width
// So max value width is: terminal_width - max_label_width - 8
let max_value_width = terminal_width.saturating_sub(max_label_width + 8);
debug!("Max value width: {}", max_value_width); debug!("Max value width: {}", max_value_width);
// Process all existing rows to truncate their value cells // Process all existing rows to truncate their value cells