From b76ff27b96f219af4a7850448dbcba311a5c20c8 Mon Sep 17 00:00:00 2001
From: Jorge Manrubia <jorge@hey.com>
Date: Mon, 3 Nov 2025 09:25:46 +0100
Subject: [PATCH 1/2] Keep the last search in place

https://app.box-car.com/5986089/cards/2703
---
 app/javascript/controllers/bar_controller.js | 29 ++++++++++----------
 app/views/bar/_bar.html.erb                  |  2 +-
 app/views/searches/_form.html.erb            |  5 ++--
 3 files changed, 18 insertions(+), 18 deletions(-)

diff --git a/app/javascript/controllers/bar_controller.js b/app/javascript/controllers/bar_controller.js
index 0f995ae65..6b10a27b4 100644
--- a/app/javascript/controllers/bar_controller.js
+++ b/app/javascript/controllers/bar_controller.js
@@ -2,7 +2,7 @@ import { Controller } from "@hotwired/stimulus"
 import { post } from "@rails/request.js"
 
 export default class extends Controller {
-  static targets = [ "turboFrame", "searchInput", "buttonsContainer" ]
+  static targets = [ "turboFrame", "search", "searchInput", "form", "buttonsContainer" ]
   static outlets = [ "dialog" ]
   static values = {
     searchUrl: String,
@@ -17,18 +17,13 @@ export default class extends Controller {
     this.dialogOutlet.close()
     this.#clearTurboFrame()
 
-    this.searchInputTarget.querySelector("input").value = ""
     this.#showItem(this.buttonsContainerTarget)
-    this.#hideItem(this.searchInputTarget)
+    this.#hideItem(this.searchTarget)
   }
 
   showModalAndSubmit(event) {
-    event.preventDefault()
-
-    const form = event.target.closest("form")
-
     this.showModal()
-    form.requestSubmit()
+    this.formTarget.requestSubmit()
   }
 
   showModal() {
@@ -36,10 +31,18 @@ export default class extends Controller {
   }
 
   search(event) {
-    this.#openInTurboFrame(this.searchUrlValue)
-
-    this.#showItem(this.searchInputTarget)
+    this.#showItem(this.searchTarget)
     this.#hideItem(this.buttonsContainerTarget)
+
+    if (this.searchInputTarget.value.trim()) {
+      this.showModalAndSubmit()
+    } else {
+      this.#loadTurboFrame()
+    }
+  }
+
+  #loadTurboFrame() {
+    this.turboFrameTarget.src = this.searchUrlValue
   }
 
   #clearTurboFrame() {
@@ -47,10 +50,6 @@ export default class extends Controller {
     this.turboFrameTarget.innerHtml = ""
   }
 
-  #openInTurboFrame(url) {
-    this.turboFrameTarget.src = url
-  }
-
   #showItem(element) {
     element.removeAttribute("hidden")
 
diff --git a/app/views/bar/_bar.html.erb b/app/views/bar/_bar.html.erb
index 4858ce2a8..095e7c0b2 100644
--- a/app/views/bar/_bar.html.erb
+++ b/app/views/bar/_bar.html.erb
@@ -7,7 +7,7 @@
     <% end %>
   </div>
 
-  <div class="bar__input" data-bar-target="searchInput" hidden>
+  <div class="bar__input" data-bar-target="search" hidden>
     <%= render "searches/form", query_terms: "" %>
   </div>
 
diff --git a/app/views/searches/_form.html.erb b/app/views/searches/_form.html.erb
index 5c956c38f..379126465 100644
--- a/app/views/searches/_form.html.erb
+++ b/app/views/searches/_form.html.erb
@@ -1,4 +1,4 @@
-<%= form_with url: search_path, method: :get, class: "search__form flex align-center justify-center gap-half", data: { turbo_frame: "bar_content" } do |form| %>
+<%= form_with url: search_path, method: :get, class: "search__form flex align-center justify-center gap-half", data: { bar_target: "form", turbo_frame: "bar_content" } do |form| %>
   <%= form.label :q, "Search BOXCAR", class: "font-weight-black txt-nowrap" %>
   <%= text_field_tag :q, query_terms,
         class: "search__input input",
@@ -7,7 +7,8 @@
         autocomplete: "off",
         autofocus: true,
         data: {
-          action: "keydown.enter->bar#showModalAndSubmit keydown.esc->bar#reset" } %>
+          bar_target: "searchInput",
+          action: "keydown.enter->bar#showModalAndSubmit:prevent keydown.esc->bar#reset" } %>
   <button class="search__reset btn btn--circle borderless" data-action="bar#reset">
     <%= icon_tag "close" %>
     <span class="for-screen-reader">Close search</span>

From ebe1617f42541dfc0d75a1eb9fff7cbfa619122c Mon Sep 17 00:00:00 2001
From: Jorge Manrubia <jorge@hey.com>
Date: Mon, 3 Nov 2025 09:29:43 +0100
Subject: [PATCH 2/2] Format

---
 app/controllers/users/joins_controller.rb             | 1 -
 test/controllers/accounts/settings_controller_test.rb | 2 +-
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/app/controllers/users/joins_controller.rb b/app/controllers/users/joins_controller.rb
index 630c579bc..493887452 100644
--- a/app/controllers/users/joins_controller.rb
+++ b/app/controllers/users/joins_controller.rb
@@ -29,4 +29,3 @@ class Users::JoinsController < ApplicationController
       params.expect(user: [ :name, :avatar ])
     end
 end
-
diff --git a/test/controllers/accounts/settings_controller_test.rb b/test/controllers/accounts/settings_controller_test.rb
index 78b9f13ac..9a9a45d3b 100644
--- a/test/controllers/accounts/settings_controller_test.rb
+++ b/test/controllers/accounts/settings_controller_test.rb
@@ -11,7 +11,7 @@ class Account::SettingsControllerTest < ActionDispatch::IntegrationTest
   end
 
   test "update" do
-    put account_settings_path, params: { account: { name: "New Account Name" }}
+    put account_settings_path, params: { account: { name: "New Account Name" } }
     assert_equal "New Account Name", Account.sole.name
     assert_redirected_to account_settings_path
   end