From 3468ccd41d96524cac40bf5adc46e91ddab7de2c Mon Sep 17 00:00:00 2001
From: Jason Zimdars <jz@37signals.com>
Date: Fri, 25 Apr 2025 13:58:41 -0500
Subject: [PATCH] Improve terminal history behavior

---
 app/javascript/controllers/toggle_class_controller.js | 8 ++++++++
 app/views/terminals/_event.html.erb                   | 2 +-
 app/views/terminals/edit.html.erb                     | 3 +--
 3 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/app/javascript/controllers/toggle_class_controller.js b/app/javascript/controllers/toggle_class_controller.js
index 694106ea2..ca29ad771 100644
--- a/app/javascript/controllers/toggle_class_controller.js
+++ b/app/javascript/controllers/toggle_class_controller.js
@@ -6,4 +6,12 @@ export default class extends Controller {
   toggle() {
     this.element.classList.toggle(this.toggleClass)
   }
+
+  add() {
+    this.element.classList.add(this.toggleClass)
+  }
+
+  remove() {
+    this.element.classList.remove(this.toggleClass)
+  }
 }
diff --git a/app/views/terminals/_event.html.erb b/app/views/terminals/_event.html.erb
index b82c37ba6..041ca94c4 100644
--- a/app/views/terminals/_event.html.erb
+++ b/app/views/terminals/_event.html.erb
@@ -1,5 +1,5 @@
 <li class="min-width overflow-ellipsis">
   <%= link_to event_action_sentence(event), collection_card_path(event.collection, event.eventable),
       class: "terminal__link",
-      data: { turbo_frame: "_top" } %>
+      data: { action: "toggle-class#remove", turbo_frame: "_top" } %>
 </li>
diff --git a/app/views/terminals/edit.html.erb b/app/views/terminals/edit.html.erb
index 38496e662..735162d56 100644
--- a/app/views/terminals/edit.html.erb
+++ b/app/views/terminals/edit.html.erb
@@ -12,8 +12,7 @@
         autocapitalize: "off",
         class: "terminal__input input fill-transparent unpad",
         data: { 
-          controller: "filter", 
-          action: "keydown.up->toggle-class#toggle:prevent input->filter#filter keydown.esc->form#cancel:prevent"
+          action: "keydown.up->toggle-class#add:prevent keydown.down->toggle-class#remove:prevent keydown.esc->form#cancel:prevent"
         },
         id: "fizzy_do",
         placeholder: "Search or type commands…" %>