From 78f72cfbd9230c9f4aaed53f638c3433ebd59584 Mon Sep 17 00:00:00 2001
From: Andy Smith <andy@37signals.com>
Date: Fri, 26 Sep 2025 13:27:04 -0500
Subject: [PATCH] Reconnect drag audio after latest updates

---
 app/javascript/controllers/drag_and_strum_controller.js | 1 -
 app/views/cards/index/engagement/_closed.html.erb       | 1 -
 app/views/cards/index/engagement/_doing.html.erb        | 1 -
 app/views/cards/index/engagement/_on_deck.html.erb      | 1 -
 app/views/collections/show.html.erb                     | 3 ++-
 app/views/collections/show/_closed.html.erb             | 1 +
 app/views/collections/show/_column.html.erb             | 1 +
 app/views/collections/show/_not_now.html.erb            | 1 +
 8 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/app/javascript/controllers/drag_and_strum_controller.js b/app/javascript/controllers/drag_and_strum_controller.js
index e19b65b87..8946f360a 100644
--- a/app/javascript/controllers/drag_and_strum_controller.js
+++ b/app/javascript/controllers/drag_and_strum_controller.js
@@ -49,7 +49,6 @@ export default class extends Controller {
   connect() {
     this.instrumentIndex = 0
     this.preloadedAudioFiles = []
-
     document.addEventListener("keydown", this.handleKeyDown.bind(this));
   }
 
diff --git a/app/views/cards/index/engagement/_closed.html.erb b/app/views/cards/index/engagement/_closed.html.erb
index 49a467a75..33eac95cb 100644
--- a/app/views/cards/index/engagement/_closed.html.erb
+++ b/app/views/cards/index/engagement/_closed.html.erb
@@ -6,7 +6,6 @@
 
 <section id="closed-cards" class="cards cards--closed is-collapsed" style="--card-color: var(--color-card-complete);"
     data-drag-and-drop-target="container"
-    data-drag-and-strum-target="container"
     data-drop-target="closed"
     data-collapsible-columns-target="column">
 
diff --git a/app/views/cards/index/engagement/_doing.html.erb b/app/views/cards/index/engagement/_doing.html.erb
index 7f390d3c2..510224e25 100644
--- a/app/views/cards/index/engagement/_doing.html.erb
+++ b/app/views/cards/index/engagement/_doing.html.erb
@@ -2,7 +2,6 @@
     class="cards cards--doing is-collapsed" style="--card-color: <%= stage.color if defined?(stage) && stage %>"
     data-collapsible-columns-target="column"
     data-drag-and-drop-target="container"
-    data-drag-and-strum-target="container"
     data-drop-target="doing"
     data-stage-id="<%= stage.id if defined?(stage) && stage %>">
   <%
diff --git a/app/views/cards/index/engagement/_on_deck.html.erb b/app/views/cards/index/engagement/_on_deck.html.erb
index b3de96e74..bc4486229 100644
--- a/app/views/cards/index/engagement/_on_deck.html.erb
+++ b/app/views/cards/index/engagement/_on_deck.html.erb
@@ -1,7 +1,6 @@
 <section id="on-deck-cards" class="cards cards--on-deck is-collapsed" style="--card-color: var(--color-card-complete);"
     data-collapsible-columns-target="column"
     data-drag-and-drop-target="container"
-    data-drag-and-strum-target="container"
     data-drop-target="on_deck">
 
   <%= cards_expander "Not Now", column.filter.cards.count %>
diff --git a/app/views/collections/show.html.erb b/app/views/collections/show.html.erb
index 207e7c4c6..4c70542c2 100644
--- a/app/views/collections/show.html.erb
+++ b/app/views/collections/show.html.erb
@@ -26,12 +26,13 @@
 <%= render "filters/settings", user_filtering: @user_filtering, no_filtering_url: collection_path(@collection) %>
 
 <%= tag.div data: {
-      controller: "drag-and-drop",
+      controller: "drag-and-drop drag-and-strum",
       drag_and_drop_dragged_item_class: "drag-and-drop__dragged-item",
       drag_and_drop_hover_container_class: "drag-and-drop__hover-container",
       action: "
         dragstart->drag-and-drop#dragStart
         dragover->drag-and-drop#dragOver
+        dragenter->drag-and-strum#dragEnter
         drop->drag-and-drop#drop
         dragend->drag-and-drop#dragEnd" } do %>
 
diff --git a/app/views/collections/show/_closed.html.erb b/app/views/collections/show/_closed.html.erb
index 7101a1778..747c002e3 100644
--- a/app/views/collections/show/_closed.html.erb
+++ b/app/views/collections/show/_closed.html.erb
@@ -1,5 +1,6 @@
 <section id="closed-cards" class="cards cards--on-deck is-collapsed" style="--card-color: var(--color-card-complete);"
   data-drag-and-drop-target="container"
+  data-drag-and-strum-target="container"
   data-collapsible-columns-target="column"
   data-action="turbo:before-morph-attribute->collapsible-columns#preventToggle"
   data-drag-and-drop-url="<%= columns_card_drops_closure_path("__id__") %>"
diff --git a/app/views/collections/show/_column.html.erb b/app/views/collections/show/_column.html.erb
index 670db2e03..e61b7b3ea 100644
--- a/app/views/collections/show/_column.html.erb
+++ b/app/views/collections/show/_column.html.erb
@@ -1,6 +1,7 @@
 <section id="<%= dom_id(column) %>"
   class="cards cards--doing is-collapsed" style="--card-color: <%= column.color %>;"
   data-drag-and-drop-target="container"
+  data-drag-and-strum-target="container"
   data-collapsible-columns-target="column"
   data-action="turbo:before-morph-attribute->collapsible-columns#preventToggle"
   data-drag-and-drop-url="<%= columns_card_drops_column_path("__id__", column_id: column.id) %>"
diff --git a/app/views/collections/show/_not_now.html.erb b/app/views/collections/show/_not_now.html.erb
index 58b516da5..365050ba7 100644
--- a/app/views/collections/show/_not_now.html.erb
+++ b/app/views/collections/show/_not_now.html.erb
@@ -1,6 +1,7 @@
 <section id="not-now" class="cards cards--on-deck is-collapsed" style="--card-color: var(--color-card-complete);"
   data-collapsible-columns-target="column"
   data-drag-and-drop-target="container"
+  data-drag-and-strum-target="container"
   data-action="turbo:before-morph-attribute->collapsible-columns#preventToggle"
   data-drag-and-drop-url="<%= columns_card_drops_not_now_path("__id__") %>"
 >