Let the binding functions set the binding variable

This ensures they can just be called and "the right thing" will happen -
fish_user_key_bindings will be executed, the variable will reflect the bindings.

2 files changed, 8 insertions, 0 deletions

diff --git a/share/functions/fish_default_key_bindings.fish b/share/functions/fish_default_key_bindings.fish
index c057754a..e679fac0 100644
--- a/share/functions/fish_default_key_bindings.fish
+++ b/share/functions/fish_default_key_bindings.fish
@@ -1,6 +1,10 @@
 
 function fish_default_key_bindings -d "Default (Emacs-like) key bindings for fish" -a mode
 	if not set -q mode[1]
+		if test "$fish_key_bindings" != "fish_default_key_bindings"
+			set fish_key_bindings fish_default_key_bindings # This triggers the handler, which calls us again and ensures the user_key_bindings are executed
+			return
+		end
 		# Clear earlier bindings, if any
 		bind --erase --all
 	end
diff --git a/share/functions/fish_vi_key_bindings.fish b/share/functions/fish_vi_key_bindings.fish
index c2a40361..1956a11b 100644
--- a/share/functions/fish_vi_key_bindings.fish
+++ b/share/functions/fish_vi_key_bindings.fish
@@ -1,4 +1,8 @@
 function fish_vi_key_bindings --description 'vi-like key bindings for fish'
+	if test "$fish_key_bindings" != "fish_vi_key_bindings"
+		set fish_key_bindings fish_vi_key_bindings # This triggers the handler, which calls us again and ensures the user_key_bindings are executed
+		return
+	end
     # The default escape timeout is 300ms. But for users of Vi bindings that can
     # be slightly annoying when trying to switch to Vi "normal" mode. Too,
     # vi-mode users are unlikely to use escape-as-meta. So set a much shorter