alternative solution to address the undesired eye movements

Author: kailiuca

crates/viewer/re_ui/src/command.rs

@@ -459,6 +459,8 @@ impl UICommand {
         egui_ctx.input_mut(|input| {
             for (kb_shortcut, command) in commands {
                 if input.consume_shortcut(&kb_shortcut) {
+                    // Clear the shortcut key from input to prevent it from propagating to other UI component.
+                    input.keys_down.remove(&kb_shortcut.logical_key);
                     return Some(command);
                 }
             }

crates/viewer/re_view_spatial/src/eye.rs

@@ -445,11 +445,11 @@ impl ViewEye {
             return false; // e.g. we're typing in a TextField
         }
 
-        // Navigation should not be activated if any modifiers is pressed
-        let modifiers_pressed = egui_ctx.input(|input| input.modifiers.any());
-        if modifiers_pressed {
-            return false;
-        }
+        // // Navigation should not be activated if any modifiers is pressed
+        // let modifiers_pressed = egui_ctx.input(|input| input.modifiers.any());
+        // if modifiers_pressed {
+        //     return false;
+        // }
 
         let mut did_interact = false;
         let mut requires_repaint = false;

crates/viewer/re_viewer/src/app.rs

@@ -768,11 +768,11 @@ impl App {
                 Some(re_log_types::StoreId::random(StoreKind::Recording))
             });
 
-        // Clear any shortcut input before firing up any UI components
-        egui_ctx.input_mut(|input| {
-            input.modifiers = Default::default();
-            input.keys_down = Default::default();
-        });
+        // // Clear any shortcut input before firing up any UI components
+        // egui_ctx.input_mut(|input| {
+        //     input.modifiers = Default::default();
+        //     input.keys_down = Default::default();
+        // });
 
         match cmd {
             UICommand::SaveRecording => {