diff --git a/crates/nu-command/src/commands/table/command.rs b/crates/nu-command/src/commands/table/command.rs index 135770bfb..d686a68a4 100644 --- a/crates/nu-command/src/commands/table/command.rs +++ b/crates/nu-command/src/commands/table/command.rs @@ -238,7 +238,8 @@ async fn table( if !input.is_empty() { let t = from_list(&input, &configuration, start_number, &color_hm); - draw_table(&t, term_width, &color_hm); + let output = draw_table(&t, term_width, &color_hm); + println!("{}", output); } start_number += input.len(); diff --git a/crates/nu-table/src/table.rs b/crates/nu-table/src/table.rs index 0178eaaf2..49bf3a5fd 100644 --- a/crates/nu-table/src/table.rs +++ b/crates/nu-table/src/table.rs @@ -631,7 +631,7 @@ impl WrappedTable { &self, separator_position: SeparatorPosition, color_hm: &HashMap, - ) { + ) -> String { let column_count = self.column_widths.len(); let mut output = String::new(); let sep_color = color_hm @@ -773,16 +773,22 @@ impl WrappedTable { } } } - - println!("{}", output); + output.push('\n'); + output } - fn print_cell_contents(&self, cells: &[WrappedCell], color_hm: &HashMap) { + fn print_cell_contents( + &self, + cells: &[WrappedCell], + color_hm: &HashMap, + ) -> String { let sep_color = color_hm .get("separator_color") .unwrap_or(&Style::default()) .to_owned(); + let mut total_output = String::new(); + for current_line in 0.. { let mut lines_printed = 0; @@ -859,30 +865,37 @@ impl WrappedTable { if lines_printed == 0 { break; } else { - println!("{}", output); + total_output.push_str(output.as_str()); + total_output.push('\n'); } } + total_output } - fn print_table(&self, color_hm: &HashMap) { + fn print_table(&self, color_hm: &HashMap) -> String { + let mut output = String::new(); + #[cfg(windows)] { let _ = nu_ansi_term::enable_ansi_support(); } if self.data.is_empty() { - return; + return output; } if self.theme.print_top_border { - self.print_separator(SeparatorPosition::Top, &color_hm); + output.push_str( + self.print_separator(SeparatorPosition::Top, &color_hm) + .as_str(), + ); } let skip_headers = (self.headers.len() == 2 && self.headers[1].max_width == 0) || (self.headers.len() == 1 && self.headers[0].max_width == 0); if !self.headers.is_empty() && !skip_headers { - self.print_cell_contents(&self.headers, &color_hm); + output.push_str(self.print_cell_contents(&self.headers, &color_hm).as_str()); } let mut first_row = true; @@ -890,22 +903,32 @@ impl WrappedTable { for row in &self.data { if !first_row { if self.theme.separate_rows { - self.print_separator(SeparatorPosition::Middle, &color_hm); + output.push_str( + self.print_separator(SeparatorPosition::Middle, &color_hm) + .as_str(), + ) } } else { first_row = false; if self.theme.separate_header && !self.headers.is_empty() && !skip_headers { - self.print_separator(SeparatorPosition::Middle, &color_hm); + output.push_str( + self.print_separator(SeparatorPosition::Middle, &color_hm) + .as_str(), + ); } } - self.print_cell_contents(row, &color_hm); + output.push_str(self.print_cell_contents(row, &color_hm).as_str()); } if self.theme.print_bottom_border { - self.print_separator(SeparatorPosition::Bottom, &color_hm); + output.push_str( + self.print_separator(SeparatorPosition::Bottom, &color_hm) + .as_str(), + ); } + output } } @@ -995,7 +1018,7 @@ pub fn maybe_truncate_columns(termwidth: usize, processed_table: &mut ProcessedT } } -pub fn draw_table(table: &Table, termwidth: usize, color_hm: &HashMap) { +pub fn draw_table(table: &Table, termwidth: usize, color_hm: &HashMap) -> String { // Remove the edges, if used let termwidth = if table.theme.print_left_border && table.theme.print_right_border { termwidth - 2 @@ -1018,7 +1041,7 @@ pub fn draw_table(table: &Table, termwidth: usize, color_hm: &HashMap