changeset 6828:aed55b52115e draft

Completion: Remove (S-)tab These bindings fight a little with the snippet placeholders. I wish we could remove them entirely (i.e. upstream) but I suspect that nearly everyone uses tab currently. I always use C-n/C-p.
author Michael Davis <mcarsondavis@gmail.com>
date Fri, 22 Mar 2024 11:29:45 -0400
parents 0cde36878b49
children 06c33a969fde
files helix-term/src/ui/menu.rs
diffstat 1 files changed, 4 insertions(+), 4 deletions(-) [+]
line wrap: on
line diff
--- a/helix-term/src/ui/menu.rs	Sun Nov 20 10:56:57 2022 -0600
+++ b/helix-term/src/ui/menu.rs	Fri Mar 22 11:29:45 2024 -0400
@@ -239,14 +239,14 @@
                 (self.callback_fn)(cx.editor, self.selection(), MenuEvent::Abort);
                 return EventResult::Consumed(close_fn);
             }
-            // arrow up/ctrl-p/shift-tab prev completion choice (including updating the doc)
-            shift!(Tab) | key!(Up) | ctrl!('p') => {
+            // arrow up/ctrl-p prev completion choice (including updating the doc)
+            key!(Up) | ctrl!('p') => {
                 self.move_up();
                 (self.callback_fn)(cx.editor, self.selection(), MenuEvent::Update);
                 return EventResult::Consumed(None);
             }
-            key!(Tab) | key!(Down) | ctrl!('n') => {
-                // arrow down/ctrl-n/tab advances completion choice (including updating the doc)
+            key!(Down) | ctrl!('n') => {
+                // arrow down/ctrl-n advances completion choice (including updating the doc)
                 self.move_down();
                 (self.callback_fn)(cx.editor, self.selection(), MenuEvent::Update);
                 return EventResult::Consumed(None);