From 9d4655d35e83ba30f1367de2f42ad17340d017e8 Mon Sep 17 00:00:00 2001
From: Adrien Maston <adrien@37signals.com>
Date: Thu, 5 Feb 2026 11:12:50 +0100
Subject: [PATCH] Make filters "open" by default then hide in mobile apps

---
 app/javascript/controllers/expandable_on_native_controller.js | 4 +---
 app/views/filters/_settings.html.erb                          | 1 +
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/app/javascript/controllers/expandable_on_native_controller.js b/app/javascript/controllers/expandable_on_native_controller.js
index a58ddfa9a..913437809 100644
--- a/app/javascript/controllers/expandable_on_native_controller.js
+++ b/app/javascript/controllers/expandable_on_native_controller.js
@@ -9,8 +9,6 @@ export default class extends Controller {
   static values = { autoExpandSelector: String }
 
   connect() {
-    if (this.hasAutoExpandSelectorValue && this.element.querySelector(this.autoExpandSelectorValue)) {
-      this.element.open = true
-    }
+    this.element.open = this.hasAutoExpandSelectorValue && this.element.querySelector(this.autoExpandSelectorValue)
   }
 }
diff --git a/app/views/filters/_settings.html.erb b/app/views/filters/_settings.html.erb
index 66bbe3d54..d0a7d3d3e 100644
--- a/app/views/filters/_settings.html.erb
+++ b/app/views/filters/_settings.html.erb
@@ -1,4 +1,5 @@
 <%= tag.details class: "expandable-on-native",
+      open: true,
       data: {
         controller: "expandable-on-native",
         expandable_on_native_auto_expand_selector_value: "[data-filter-show=true]" } do %>