Set the background color of each widget

Instead of iterating over every cell and setting the background color of that,
just set the color of each widget, which achieves the same effect. This can also be overriden if
we add the ability to set the background color of each widget.

Author: dastarruer

src/canvas.rs

@@ -12,7 +12,6 @@ use tui::{
     Frame, Terminal,
     backend::Backend,
     layout::{Constraint, Direction, Flex, Layout, Rect},
-    style::Color,
     text::Span,
     widgets::Paragraph,
 };
@@ -102,17 +101,6 @@ impl Painter {
                 self.previous_width = terminal_width;
             }
 
-            // Draw the background color
-            let area = f.area();
-            let buf = f.buffer_mut();
-            for x in area.left()..area.right() {
-                for y in area.top()..area.bottom() {
-                    if let Some(cell) = buf.cell_mut((x, y)) {
-                        cell.set_bg(Color::Blue);
-                    }
-                }
-            }
-
             // TODO: We should probably remove this or make it done elsewhere, not the responsibility of the app.
             if app_state.should_get_widget_bounds() {
                 // If we're force drawing, reset ALL mouse boundaries.

src/canvas/drawing_utils.rs

@@ -1,8 +1,7 @@
 use std::time::Instant;
 
 use tui::{
-    layout::Rect,
-    widgets::{Block, BorderType, Borders},
+    layout::Rect, style::{Color, Style}, widgets::{Block, BorderType, Borders}
 };
 
 pub const SIDE_BORDERS: Borders = Borders::LEFT.union(Borders::RIGHT);
@@ -30,7 +29,7 @@ pub fn should_hide_x_label(
 
 /// Return a widget block.
 pub fn widget_block(is_basic: bool, is_selected: bool, border_type: BorderType) -> Block<'static> {
-    let mut block = Block::default().border_type(border_type);
+    let mut block = Block::default().border_type(border_type).style(Style::default().bg(Color::Blue));
 
     if is_basic {
         if is_selected {