From f1e6e61bcaa503e7f78d9f6fa2a15de004517028 Mon Sep 17 00:00:00 2001 From: Edward Langley Date: Tue, 31 Mar 2026 22:50:10 -0700 Subject: [PATCH] Revert "test: use gpt-oss-20b to do some minor refactoring" This reverts commit bbd1f48b789749e57c16f214dd81628823b4cf5a. --- src/main.rs | 33 ++++++++++++--------------------- src/ui/grid.rs | 9 ++------- 2 files changed, 14 insertions(+), 28 deletions(-) diff --git a/src/main.rs b/src/main.rs index 1e544d7..58526be 100644 --- a/src/main.rs +++ b/src/main.rs @@ -26,11 +26,6 @@ use ratatui::{ use model::Model; use ui::app::{App, AppMode}; - -fn render_paragraph(f: &mut Frame, area: Rect, text: &str, style: Style) { - f.render_widget(Paragraph::new(text).style(style), area); -} - use ui::category_panel::CategoryPanel; use ui::formula_panel::FormulaPanel; use ui::grid::GridWidget; @@ -337,14 +332,14 @@ fn draw_title(f: &mut Frame, area: Rect, app: &App) { let right = " ?:help :q quit "; let pad = " ".repeat((area.width as usize).saturating_sub(title.len() + right.len())); let line = format!("{title}{pad}{right}"); - render_paragraph( - f, + f.render_widget( + Paragraph::new(line).style( + Style::default() + .fg(Color::Black) + .bg(Color::Blue) + .add_modifier(Modifier::BOLD), + ), area, - &line, - Style::default() - .fg(Color::Black) - .bg(Color::Blue) - .add_modifier(Modifier::BOLD), ); } @@ -455,7 +450,7 @@ fn draw_status(f: &mut Frame, area: Rect, app: &App) { _ => Style::default().fg(Color::Black).bg(Color::DarkGray), }; - render_paragraph(f, area, &line, badge_style); + f.render_widget(Paragraph::new(line).style(badge_style), area); } fn draw_command_bar(f: &mut Frame, area: Rect, app: &App) { @@ -465,11 +460,9 @@ fn draw_command_bar(f: &mut Frame, area: Rect, app: &App) { "" }; let line = format!(":{buf}▌"); - render_paragraph( - f, + f.render_widget( + Paragraph::new(line).style(Style::default().fg(Color::White).bg(Color::Black)), area, - &line, - Style::default().fg(Color::White).bg(Color::Black), ); } @@ -491,11 +484,9 @@ fn draw_export_prompt(f: &mut Frame, area: Rect, app: &App) { .title(" Export CSV — path (Esc cancel) "); let inner = block.inner(popup_area); f.render_widget(block, popup_area); - render_paragraph( - f, + f.render_widget( + Paragraph::new(format!("{buf}▌")).style(Style::default().fg(Color::Green)), inner, - &format!("{buf}▌"), - Style::default().fg(Color::Green), ); } diff --git a/src/ui/grid.rs b/src/ui/grid.rs index ca871b9..6277f50 100644 --- a/src/ui/grid.rs +++ b/src/ui/grid.rs @@ -291,7 +291,7 @@ impl<'a> GridWidget<'a> { }; let value = self.model.evaluate(&key); - let cell_str = format_cell_value(value.as_ref(), &view.number_format); + let cell_str = format_value(value.as_ref(), fmt_comma, fmt_decimals); let is_selected = ri == sel_row && ci == sel_col; let is_search_match = !self.search_query.is_empty() && cell_str @@ -435,7 +435,7 @@ impl<'a> Widget for GridWidget<'a> { } } -pub fn format_value(v: Option<&CellValue>, comma: bool, decimals: u8) -> String { +fn format_value(v: Option<&CellValue>, comma: bool, decimals: u8) -> String { match v { Some(CellValue::Number(n)) => format_f64(*n, comma, decimals), Some(CellValue::Text(s)) => s.clone(), @@ -443,11 +443,6 @@ pub fn format_value(v: Option<&CellValue>, comma: bool, decimals: u8) -> String } } -fn format_cell_value(v: Option<&CellValue>, fmt: &str) -> String { - let (comma, decimals) = parse_number_format(fmt); - format_value(v, comma, decimals) -} - pub fn parse_number_format(fmt: &str) -> (bool, u8) { let comma = fmt.contains(','); let decimals = fmt