tyler.durden
/
bottom
mirror of https://github.com/ClementTsang/bottom.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

bug: Fix hide table gap option not working in battery widget (#386) 

Fixes the `hide_table_gap` option not working with the battery widget.
This commit is contained in:
Clement Tsang 2021-01-05 22:18:56 -05:00 committed by GitHub
parent 429978d1aa
commit cfddb7e223
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 15 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
src/canvas/widgets/battery_display.rs
View File

@ -35,6 +35,11 @@ impl BatteryDisplayWidget for Painter {
} else {
self.colours.border_style
};
let table_gap = if draw_loc.height < TABLE_GAP_HEIGHT_LIMIT {
0
} else {
app_state.app_config_fields.table_gap
};
let title = if app_state.is_expanded {
const TITLE_BASE: &str = " Battery ── Esc to go back ";
@ -156,16 +161,20 @@ impl BatteryDisplayWidget for Painter {
Table::new([""].iter(), battery_rows)
.block(battery_block)
.header_style(self.colours.table_header_style)
.widths(&[Constraint::Percentage(50), Constraint::Percentage(50)]),
.widths(&[Constraint::Percentage(50), Constraint::Percentage(50)])
.header_gap(table_gap),
margined_draw_loc,
);
} else {
let mut contents = vec![Spans::default(); table_gap as usize];
contents.push(Spans::from(Span::styled(
"No data found for this battery",
self.colours.text_style,
)));
f.render_widget(
Paragraph::new(Span::styled(
"No data found for this battery",
self.colours.text_style,
))
.block(battery_block),
Paragraph::new(contents).block(battery_block),
margined_draw_loc,
);
}