From 879258039cbb66e38ddc55e9528757f3bb00174e Mon Sep 17 00:00:00 2001 From: Stefan Holderbach Date: Thu, 13 Feb 2025 16:29:08 +0100 Subject: [PATCH] Revert `/` vi binding due to priority bug (#15111) Manually added bindings take priority to the vi-mode state machine in reedline thus this addition blocked the use of `f/`/`t/` etc. Partial revert of #14908 Addresses #15096 with a temporary fix. The full solution of that should resolve it on the reedline side so you can have both the search option and the availability of `/` in normal mode bindings --- crates/nu-cli/src/reedline_config.rs | 5 ----- 1 file changed, 5 deletions(-) diff --git a/crates/nu-cli/src/reedline_config.rs b/crates/nu-cli/src/reedline_config.rs index ea5af59c16..69be71eaf7 100644 --- a/crates/nu-cli/src/reedline_config.rs +++ b/crates/nu-cli/src/reedline_config.rs @@ -707,11 +707,6 @@ pub(crate) fn create_keybindings(config: &Config) -> Result { add_menu_keybindings(&mut insert_keybindings); add_menu_keybindings(&mut normal_keybindings); - normal_keybindings.add_binding( - KeyModifiers::NONE, - KeyCode::Char('/'), - ReedlineEvent::Menu("history_menu".to_string()), - ); } } for keybinding in parsed_keybindings {