fix: Better handling of Escape and Enter keys

raphamorim · Jan 19, 2025 · f9ca661 · f9ca661
1 parent 66257aa
commit f9ca661
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 13 deletions.
diff --git a/frontends/rioterm/src/router/mod.rs b/frontends/rioterm/src/router/mod.rs
@@ -101,10 +101,10 @@ impl Route<'_> {
 
     #[inline]
     pub fn report_error(&mut self, error: &RioError) {
-        if error.report == RioErrorType::ConfigurationNotFound {
-            self.path = RoutePath::Welcome;
-            return;
-        }
+        // if error.report == RioErrorType::ConfigurationNotFound {
+        //     self.path = RoutePath::Welcome;
+        //     return;
+        // }
 
         self.assistant.set(error.to_owned());
         self.path = RoutePath::Assistant;
@@ -133,32 +133,31 @@ impl Route<'_> {
         }
 
         let is_enter = key_event.logical_key == Key::Named(NamedKey::Enter);
-        if self.path == RoutePath::Assistant && is_enter {
-            if self.assistant.is_warning() {
+        if self.path == RoutePath::Assistant {
+            if self.assistant.is_warning() && is_enter {
                 self.assistant.clear();
                 self.path = RoutePath::Terminal;
+            } else {
+                return true;
             }
-
-            return true;
         }
 
         if self.path == RoutePath::ConfirmQuit {
             if key_event.logical_key == Key::Named(NamedKey::Escape) {
                 self.path = RoutePath::Terminal;
             } else if is_enter {
                 self.quit();
-            }
 
-            return true;
+                return true;
+            }
         }
 
         if self.path == RoutePath::Welcome && is_enter {
             rio_backend::config::create_config_file(None);
             self.path = RoutePath::Terminal;
-            return true;
         }
 
-        true
+        false
     }
 }
 

diff --git a/frontends/rioterm/src/router/routes/assistant.rs b/frontends/rioterm/src/router/routes/assistant.rs
@@ -83,7 +83,7 @@ pub fn screen(
         if report.level == RioErrorLevel::Warning {
             objects.push(Object::Text(Text::single_line(
                 (70., context_dimension.margin.top_y + 80.),
-                String::from("(press enter twice to continue)"),
+                String::from("(press enter to continue)"),
                 18.,
                 [1., 1., 1., 1.],
             )));