fix(windows): Handle keypresses only once

On Windows the UI would often jump around when pressing navigation
buttons. That's because crossterm sends a Press and a Release event for
each keystroke on Windows.

Signed-off-by: Daniel Schaefer <dhs@frame.work>

Author: JohnAZoidberg

src/tui.rs

@@ -5,7 +5,7 @@ pub mod theme;
 use std::sync::Arc;
 
 use ratatui::{
-    crossterm::event::{Event, KeyCode},
+    crossterm::event::{Event, KeyCode, KeyEventKind},
     layout::{Constraint, Flex, Layout},
     prelude::Backend,
     style::Style,
@@ -98,7 +98,7 @@ impl Tui {
 
     pub fn handle_input(&mut self, event: Event) -> color_eyre::Result<Option<AppEvent>> {
         let top_level_event = match &event {
-            Event::Key(key) => match key.code {
+            Event::Key(key) if key.kind == KeyEventKind::Press => match key.code {
                 KeyCode::Char('q') => Some(AppEvent::Quit),
                 KeyCode::Char('b') => {
                     self.previous_theme();

src/tui/component/brightness_panel.rs

@@ -1,7 +1,7 @@
 use std::sync::Arc;
 
 use ratatui::{
-    crossterm::event::{Event, KeyCode},
+    crossterm::event::{Event, KeyCode, KeyEventKind},
     layout::{Constraint, Layout, Rect},
     prelude::*,
     style::Styled,
@@ -194,6 +194,9 @@ impl Component for BrightnessPanelComponent {
 
         if self.panel.is_selected() {
             if let Event::Key(key) = event {
+                if key.kind != KeyEventKind::Press {
+                    return app_event;
+                }
                 match key.code {
                     KeyCode::Down => self.panel.cycle_controls_down(),
                     KeyCode::Up => self.panel.cycle_controls_up(),

src/tui/component/charge_panel.rs

@@ -1,5 +1,5 @@
 use ratatui::{
-    crossterm::event::{Event, KeyCode},
+    crossterm::event::{Event, KeyCode, KeyEventKind},
     layout::{Constraint, Layout, Rect},
     prelude::*,
     style::Styled,
@@ -311,6 +311,9 @@ impl Component for ChargePanelComponent {
 
         if self.0.is_selected() {
             if let Event::Key(key) = event {
+                if key.kind != KeyEventKind::Press {
+                    return app_event;
+                }
                 match key.code {
                     KeyCode::Down => self.0.cycle_controls_down(),
                     KeyCode::Up => self.0.cycle_controls_up(),

src/tui/component/main.rs

@@ -1,7 +1,7 @@
 use std::sync::Arc;
 
 use ratatui::{
-    crossterm::event::{Event, KeyCode},
+    crossterm::event::{Event, KeyCode, KeyEventKind},
     layout::{Constraint, Layout, Rect},
     Frame,
 };
@@ -96,8 +96,8 @@ impl MainComponent {
 
 impl Component for MainComponent {
     fn handle_input(&mut self, event: Event) -> Option<crate::app::AppEvent> {
-        if let Event::Key(key) = event {
-            if key.code == KeyCode::Tab {
+        if let Event::Key(key) = &event {
+            if key.kind == KeyEventKind::Press && key.code == KeyCode::Tab {
                 self.switch_panels();
             }
         }