refactor: remove input_poll_rate from Ui, instead use const POLL_RATE

This commit is contained in:
Jack Wills
2024-12-04 22:24:46 +00:00
parent 35aec5060f
commit 69f6c96b70
+4 -5
View File
@@ -37,11 +37,11 @@ use crate::{
}; };
pub const ORANGE: ratatui::style::Color = ratatui::style::Color::Rgb(255, 178, 36); pub const ORANGE: ratatui::style::Color = ratatui::style::Color::Rgb(255, 178, 36);
const POLL_RATE: Duration = std::time::Duration::from_millis(100);
pub struct Ui { pub struct Ui {
app_data: Arc<Mutex<AppData>>, app_data: Arc<Mutex<AppData>>,
gui_state: Arc<Mutex<GuiState>>, gui_state: Arc<Mutex<GuiState>>,
input_poll_rate: Duration,
input_tx: Sender<InputMessages>, input_tx: Sender<InputMessages>,
is_running: Arc<AtomicBool>, is_running: Arc<AtomicBool>,
now: Instant, now: Instant,
@@ -75,7 +75,6 @@ impl Ui {
app_data, app_data,
cursor_position, cursor_position,
gui_state, gui_state,
input_poll_rate: std::time::Duration::from_millis(100),
input_tx, input_tx,
is_running, is_running,
now: Instant::now(), now: Instant::now(),
@@ -145,7 +144,7 @@ impl Ui {
Ok(()) Ok(())
} }
/// Use exeternal docker cli to exec into a container /// Use external docker cli to exec into a container
async fn exec(&mut self) { async fn exec(&mut self) {
let exec_mode = self.gui_state.lock().get_exec_mode(); let exec_mode = self.gui_state.lock().get_exec_mode();
@@ -181,7 +180,7 @@ impl Ui {
return Err(AppError::Terminal); return Err(AppError::Terminal);
} }
if crossterm::event::poll(self.input_poll_rate).unwrap_or(false) { if crossterm::event::poll(POLL_RATE).unwrap_or(false) {
if let Ok(event) = event::read() { if let Ok(event) = event::read() {
if let Event::Key(key) = event { if let Event::Key(key) = event {
if key.kind == event::KeyEventKind::Press { if key.kind == event::KeyEventKind::Press {
@@ -221,7 +220,7 @@ impl Ui {
} }
} }
/// Frequent data required by multiple framde drawing functions, can reduce mutex reads by placing it all in here /// Frequent data required by multiple frame drawing functions, can reduce mutex reads by placing it all in here
#[derive(Debug, Clone)] #[derive(Debug, Clone)]
pub struct FrameData { pub struct FrameData {
chart_data: Option<(CpuTuple, MemTuple)>, chart_data: Option<(CpuTuple, MemTuple)>,