From 651ee5f6fec812f878fcbecb914e7ff046896015 Mon Sep 17 00:00:00 2001
From: Jorge Manrubia <jorge@37signals.com>
Date: Tue, 3 Jun 2025 06:32:38 +0200
Subject: [PATCH] Use CTRL-ENTER or CMD-ENTER to toggle, since space is a legit
 search character

---
 .../controllers/navigable_list_controller.js         | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/app/javascript/controllers/navigable_list_controller.js b/app/javascript/controllers/navigable_list_controller.js
index 0a610a457..10f776375 100644
--- a/app/javascript/controllers/navigable_list_controller.js
+++ b/app/javascript/controllers/navigable_list_controller.js
@@ -25,8 +25,7 @@ export default class extends Controller {
   }
 
   navigate(event) {
-    const key = event.key === " " ? "Space" : event.key
-    this.#keyHandlers[key]?.call(this, event)
+    this.#keyHandlers[event.key]?.call(this, event)
   }
 
   select({ target }) {
@@ -126,10 +125,11 @@ export default class extends Controller {
       this.#handleArrowKey(event, this.#selectPrevious.bind(this))
     },
     Enter(event) {
-      this.#clickCurrentItem(event)
-    },
-    Space(event) {
-      this.#toggleCurrentItem(event)
+      if (event.ctrlKey || event.metaKey) {
+        this.#toggleCurrentItem(event)
+      } else {
+        this.#clickCurrentItem(event)
+      }
     }
   }
 }