Add data-action to all collection controller forms

maebeale · claude · maebeale · commit acd06a5df14e · 2026-03-10T08:24:39.000-04:00
The collection controller was refactored to use data-action instead
of addEventListener, but only video_recordings was updated. All 17
other forms using the collection controller were missing the action
attributes, breaking search/filter auto-submit.

Co-Authored-By: Claude Opus 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/app/views/bookmarks/_search_boxes.html.erb b/app/views/bookmarks/_search_boxes.html.erb
@@ -1,6 +1,7 @@
 <%= form_with url: url,
               method: :get,
               data: { controller: "collection",
+                      action: "change->collection#handleChange input->collection#handleInput",
                       turbo_frame: frame_id },
               autocomplete: "off",
               class: "space-y-4" do |f| %>
diff --git a/app/views/bookmarks/tally.html.erb b/app/views/bookmarks/tally.html.erb
@@ -3,7 +3,7 @@
       <h1 class="text-2xl font-bold mb-4">Bookmarks tally</h1>
 
       <!-- Filter form -->
-      <%= form_tag('/bookmarks/tally', method: :get, class: "space-y-4", data: { controller: "collection" }) do %>
+      <%= form_tag('/bookmarks/tally', method: :get, class: "space-y-4", data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput" }) do %>
         <%= render "search_fields" %>
       <% end %>
 
diff --git a/app/views/community_news/_search_boxes.html.erb b/app/views/community_news/_search_boxes.html.erb
@@ -2,6 +2,7 @@
 <%= form_with url: community_news_index_path,
               method: :get,
               data: {controller: "collection",
+                    action: "change->collection#handleChange input->collection#handleInput",
                     turbo_frame: "community_news_results" },
                     autocomplete: "off",
               class: "space-y-2" do |f| %>
diff --git a/app/views/events/_manage_search.html.erb b/app/views/events/_manage_search.html.erb
@@ -1,5 +1,5 @@
 <%= form_with url: manage_event_path(@event), method: :get,
-  data: { controller: "collection", turbo_frame: "manage_results" },
+  data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput", turbo_frame: "manage_results" },
   autocomplete: "off",
   class: "flex flex-col md:flex-row md:flex-wrap md:items-end gap-4 mb-6" do %>
 
diff --git a/app/views/faqs/_search_boxes.html.erb b/app/views/faqs/_search_boxes.html.erb
@@ -1,4 +1,4 @@
-<%= form_tag(faqs_path, method: :get, class: "space-y-4", data: { controller: "collection" }) do |f| %>
+<%= form_tag(faqs_path, method: :get, class: "space-y-4", data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput" }) do |f| %>
   <div class="flex flex-col md:flex-row md:items-end md:gap-4 mb-6">
     <!-- Search -->
     <div class="w-full md:flex-1 mb-3 md:mb-0">
diff --git a/app/views/notifications/_search_boxes.html.erb b/app/views/notifications/_search_boxes.html.erb
@@ -3,6 +3,7 @@
   <%= form_with url: notifications_path,
                 method: :get,
                 data: {controller: "collection",
+                      action: "change->collection#handleChange input->collection#handleInput",
                       turbo_frame: "notifications_results" },
                       autocomplete: "off",
                 class: "flex flex-wrap gap-4" do |f| %>
diff --git a/app/views/organizations/_search_boxes.html.erb b/app/views/organizations/_search_boxes.html.erb
@@ -1,6 +1,6 @@
 <div class="mb-4">
   <%= form_with url: organizations_path, method: :get,
-    data: { controller: "collection", turbo_frame: "organization_results" },
+    data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput", turbo_frame: "organization_results" },
     autocomplete: "off",
     class: "flex flex-wrap items-end gap-3" do %>
     <div>
diff --git a/app/views/people/_search_boxes.html.erb b/app/views/people/_search_boxes.html.erb
@@ -1,5 +1,5 @@
 <%= form_tag(people_path, method: :get,
-  data: { controller: "collection", turbo_frame: "people_results" },
+  data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput", turbo_frame: "people_results" },
   autocomplete: "off",
   class: "space-y-4") do |f| %>
   <div class="flex flex-col md:flex-row md:flex-wrap md:items-end gap-4 mb-6">
diff --git a/app/views/quotes/_search_boxes.html.erb b/app/views/quotes/_search_boxes.html.erb
@@ -2,7 +2,8 @@
 <div class="mb-6 p-4 bg-white border border-gray-200 rounded-lg">
   <%= form_with url: quotes_path,
                 method: :get,
-                data: {controller: "collection"},
+                data: {controller: "collection",
+                      action: "change->collection#handleChange input->collection#handleInput"},
                       autocomplete: "off",
                 class: "flex flex-wrap gap-4" do |f| %>
     <div class="flex-grow min-w-[200px]">
diff --git a/app/views/resources/_search_boxes.html.erb b/app/views/resources/_search_boxes.html.erb
@@ -6,6 +6,7 @@
 
 <%= form_tag resources_path, method: :get,
   data: {controller: "collection share-url",
+         action: "change->collection#handleChange input->collection#handleInput",
          collection_unselected_class: default_btn,
          collection_selected_class: selected_btn,
          turbo_frame: "resource_results" },
diff --git a/app/views/stories/_search_boxes.html.erb b/app/views/stories/_search_boxes.html.erb
@@ -2,6 +2,7 @@
 <%= form_with url: stories_path,
               method: :get,
               data: {controller: "collection",
+                    action: "change->collection#handleChange input->collection#handleInput",
                     turbo_frame: "story_results" },
                     autocomplete: "off",
               class: "space-y-2" do |f| %>
diff --git a/app/views/story_ideas/_search_boxes.html.erb b/app/views/story_ideas/_search_boxes.html.erb
@@ -1,4 +1,4 @@
-<%= form_tag(story_ideas_path, method: :get, class: "space-y-4", data: { controller: "collection" }) do %>
+<%= form_tag(story_ideas_path, method: :get, class: "space-y-4", data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput" }) do %>
   <div class="flex flex-col md:flex-row md:items-end md:gap-4 mb-6">
     <!-- Search -->
     <div class="w-full md:flex-1 mb-3 md:mb-0">
diff --git a/app/views/users/_search_boxes.html.erb b/app/views/users/_search_boxes.html.erb
@@ -1,5 +1,5 @@
 <%= form_tag(users_path, method: :get,
-  data: { controller: "collection", turbo_frame: "users_results" },
+  data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput", turbo_frame: "users_results" },
   autocomplete: "off",
   class: "w-full") do %>
   <div class="grid grid-cols-1 sm:grid-cols-2 lg:grid-cols-4 gap-4 mb-6 items-end">
diff --git a/app/views/users/sections/_account_activity.html.erb b/app/views/users/sections/_account_activity.html.erb
@@ -9,7 +9,7 @@
     </div>
 
     <%= form_tag(user_path(user), method: :get,
-      data: { controller: "collection", turbo_frame: "account_activity_section" },
+      data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput", turbo_frame: "account_activity_section" },
       autocomplete: "off",
       class: "mb-4") do %>
       <%= hidden_field_tag :section, "account_activity" %>
diff --git a/app/views/workshop_variation_ideas/_search_boxes.html.erb b/app/views/workshop_variation_ideas/_search_boxes.html.erb
@@ -1,4 +1,4 @@
-<%= form_tag(workshop_variation_ideas_path, method: :get, class: "space-y-4", data: { controller: "collection" }) do %>
+<%= form_tag(workshop_variation_ideas_path, method: :get, class: "space-y-4", data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput" }) do %>
   <div class="flex flex-col md:flex-row md:items-end md:gap-4 mb-6">
     <!-- Search -->
     <div class="w-full md:flex-1 mb-3 md:mb-0">
diff --git a/app/views/workshop_variations/_search_boxes.html.erb b/app/views/workshop_variations/_search_boxes.html.erb
@@ -1,4 +1,4 @@
-<%= form_tag(workshop_variations_path, method: :get, class: "space-y-4", data: { controller: "collection" }) do %>
+<%= form_tag(workshop_variations_path, method: :get, class: "space-y-4", data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput" }) do %>
   <div class="flex flex-col md:flex-row md:items-end md:gap-4 mb-6">
     <!-- Search -->
     <div class="w-full md:flex-1 mb-3 md:mb-0">
diff --git a/app/views/workshops/index.html.erb b/app/views/workshops/index.html.erb
@@ -30,7 +30,8 @@
   <div class="w-full">
     <%= form_tag(workshops_path, method: :get,
                  data: {turbo_stream: true, turbo_frame: "workshop_results",
-                        controller: "collection share-url"}, autocomplete: "off", class: "space-y-6") do %>
+                        controller: "collection share-url",
+                        action: "change->collection#handleChange input->collection#handleInput"}, autocomplete: "off", class: "space-y-6") do %>
       <%= render "filters" %>
 
       <div><hr class="border-gray-300 my-6"></div>

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-<%= form_tag(faqs_path, method: :get, class: "space-y-4", data: { controller: "collection" }) do \|f\| %>`
	`1`	`+<%= form_tag(faqs_path, method: :get, class: "space-y-4", data: { controller: "collection", action: "change->collection#handleChange input->collection#handleInput" }) do \|f\| %>`
`2`	`2`	`<div class="flex flex-col md:flex-row md:items-end md:gap-4 mb-6">`
`3`	`3`	`<!-- Search -->`
`4`	`4`	`<div class="w-full md:flex-1 mb-3 md:mb-0">`