From a7e185fe6f6a64c597bab038c03f17166c90e5f4 Mon Sep 17 00:00:00 2001
From: Jose Farias <jose@farias.mx>
Date: Thu, 7 Mar 2024 11:50:51 -0600
Subject: [PATCH] Revert "Reroute listbox streams to dialog if necessary"

This reverts commit 36446c0f5f7c5e79d799a5b50ac7040efdb12723.
---
 .../hw_combobox/models/combobox/dialog.js          | 10 ----------
 .../hw_combobox/models/combobox/filtering.js       |  2 +-
 .../hw_combobox/models/combobox/toggle.js          |  2 +-
 app/presenters/hotwire_combobox/component.rb       |  3 +--
 test/dummy/app/views/layouts/application.html.erb  |  6 ------
 test/system/hotwire_combobox_test.rb               | 14 +++++++-------
 6 files changed, 10 insertions(+), 27 deletions(-)

diff --git a/app/assets/javascripts/hw_combobox/models/combobox/dialog.js b/app/assets/javascripts/hw_combobox/models/combobox/dialog.js
index 09f7b73..958cb63 100644
--- a/app/assets/javascripts/hw_combobox/models/combobox/dialog.js
+++ b/app/assets/javascripts/hw_combobox/models/combobox/dialog.js
@@ -1,12 +1,6 @@
 import Combobox from "hw_combobox/models/combobox/base"
 
 Combobox.Dialog = Base => class extends Base {
-  rerouteListboxStreamToDialog({ detail: { newStream } }) {
-    if (newStream.target == this.listboxTarget.id && this._dialogIsOpen) {
-      newStream.setAttribute("target", this.dialogListboxTarget.id)
-    }
-  }
-
   _connectDialog() {
     if (window.visualViewport) {
       window.visualViewport.addEventListener("resize", this._resizeDialog)
@@ -56,8 +50,4 @@ Combobox.Dialog = Base => class extends Base {
   get _smallViewport() {
     return window.matchMedia(`(max-width: ${this.smallViewportMaxWidthValue})`).matches
   }
-
-  get _dialogIsOpen() {
-    return this.dialogTarget.open
-  }
 }
diff --git a/app/assets/javascripts/hw_combobox/models/combobox/filtering.js b/app/assets/javascripts/hw_combobox/models/combobox/filtering.js
index 99e5b5c..0294aa3 100644
--- a/app/assets/javascripts/hw_combobox/models/combobox/filtering.js
+++ b/app/assets/javascripts/hw_combobox/models/combobox/filtering.js
@@ -41,7 +41,7 @@ Combobox.Filtering = Base => class extends Base {
       this._selectNew()
     } else if (isDeleteEvent(event)) {
       this._deselect()
-    } else if (this._isOpen) {
+    } else {
       this._select(this._visibleOptionElements[0])
     }
   }
diff --git a/app/assets/javascripts/hw_combobox/models/combobox/toggle.js b/app/assets/javascripts/hw_combobox/models/combobox/toggle.js
index 3e057fd..eedc66d 100644
--- a/app/assets/javascripts/hw_combobox/models/combobox/toggle.js
+++ b/app/assets/javascripts/hw_combobox/models/combobox/toggle.js
@@ -72,7 +72,7 @@ Combobox.Toggle = Base => class extends Base {
   _collapse() {
     this._actingCombobox.setAttribute("aria-expanded", false) // needs to happen before resetting acting combobox
 
-    if (this._dialogIsOpen) {
+    if (this.dialogTarget.open) {
       this._closeInDialog()
     } else {
       this._closeInline()
diff --git a/app/presenters/hotwire_combobox/component.rb b/app/presenters/hotwire_combobox/component.rb
index 0f00b4f..84ede72 100644
--- a/app/presenters/hotwire_combobox/component.rb
+++ b/app/presenters/hotwire_combobox/component.rb
@@ -232,8 +232,7 @@ def input_data
           input->hw-combobox#filter
           keydown->hw-combobox#navigate
           click@window->hw-combobox#closeOnClickOutside
-          focusin@window->hw-combobox#closeOnFocusOutside
-          turbo:before-stream-render@document->hw-combobox#rerouteListboxStreamToDialog".squish,
+          focusin@window->hw-combobox#closeOnFocusOutside".squish,
         hw_combobox_target: "combobox",
         async_id: canonical_id
     end
diff --git a/test/dummy/app/views/layouts/application.html.erb b/test/dummy/app/views/layouts/application.html.erb
index 67959f4..7d00c59 100644
--- a/test/dummy/app/views/layouts/application.html.erb
+++ b/test/dummy/app/views/layouts/application.html.erb
@@ -26,12 +26,6 @@
       .container aside.empty {
         display: none;
       }
-
-      @media (max-width: 600px) {
-        .container aside {
-          display: none;
-        }
-      }
     </style>
   </head>
 
diff --git a/test/system/hotwire_combobox_test.rb b/test/system/hotwire_combobox_test.rb
index 6ba870d..d572721 100644
--- a/test/system/hotwire_combobox_test.rb
+++ b/test/system/hotwire_combobox_test.rb
@@ -458,21 +458,21 @@ class HotwireComboboxTest < ApplicationSystemTestCase
 
   test "dialog" do
     on_small_screen do
-      visit async_html_path
+      visit html_options_path
 
       assert_no_selector "dialog[open]"
-      open_combobox "#movie-field"
+      open_combobox "#state-field"
       assert_selector "dialog[open]"
 
       within "dialog" do
-        type_in_combobox "#movie-field-hw-dialog-combobox", "whi"
-        assert_combobox_display "#movie-field-hw-dialog-combobox", "Whiplash"
-        assert_selected_option_with text: "Whiplash"
+        type_in_combobox "#state-field-hw-dialog-combobox", "Flor"
+        assert_combobox_display "#state-field-hw-dialog-combobox", "Florida"
+        assert_selected_option_with text: "Florida"
       end
-      assert_combobox_value "#movie-field", movies(:whiplash).id
+      assert_combobox_value "#state-field", "FL"
 
       click_away
-      assert_combobox_display_and_value "#movie-field", "Whiplash", movies(:whiplash).id
+      assert_combobox_display_and_value "#state-field", "Florida", "FL"
       assert_no_selector "dialog[open]"
     end
   end