From e7c92ce4439fba5e416c71dfae1f11fc4fd0617b Mon Sep 17 00:00:00 2001
From: Jorge Manrubia <jorge@hey.com>
Date: Fri, 26 Sep 2025 13:29:39 +0200
Subject: [PATCH] Keep columns open when D&D cards

---
 app/controllers/columns/cards/drops/not_nows_controller.rb  | 2 +-
 .../controllers/collapsible_columns_controller.js           | 6 ++++++
 app/views/collections/show/_closed.html.erb                 | 3 ++-
 app/views/collections/show/_column.html.erb                 | 1 +
 app/views/collections/show/_not_now.html.erb                | 3 ++-
 app/views/collections/show/_stream.html.erb                 | 1 +
 6 files changed, 13 insertions(+), 3 deletions(-)

diff --git a/app/controllers/columns/cards/drops/not_nows_controller.rb b/app/controllers/columns/cards/drops/not_nows_controller.rb
index 959cdb96e..a8fe74298 100644
--- a/app/controllers/columns/cards/drops/not_nows_controller.rb
+++ b/app/controllers/columns/cards/drops/not_nows_controller.rb
@@ -4,6 +4,6 @@ class Columns::Cards::Drops::NotNowsController < ApplicationController
   def create
     @card.postpone
 
-    render turbo_stream: turbo_stream.replace("not-now", partial: "collections/show/not_now", locals: { collection: @card.collection })
+    render turbo_stream: turbo_stream.replace("not-now", partial: "collections/show/not_now", method: :morph, locals: { collection: @card.collection })
   end
 end
diff --git a/app/javascript/controllers/collapsible_columns_controller.js b/app/javascript/controllers/collapsible_columns_controller.js
index fbb8b97bb..6655e5adf 100644
--- a/app/javascript/controllers/collapsible_columns_controller.js
+++ b/app/javascript/controllers/collapsible_columns_controller.js
@@ -19,4 +19,10 @@ export default class extends Controller {
       clickedColumn.classList.remove(this.collapsedClass)
     }
   }
+
+  preventToggle(event) {
+    if (event.detail.attributeName === "class") {
+      event.preventDefault()
+    }
+  }
 }
diff --git a/app/views/collections/show/_closed.html.erb b/app/views/collections/show/_closed.html.erb
index 74b0226ec..ff074db66 100644
--- a/app/views/collections/show/_closed.html.erb
+++ b/app/views/collections/show/_closed.html.erb
@@ -1,6 +1,7 @@
 <section id="closed-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-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 607622ff0..ce917f14d 100644
--- a/app/views/collections/show/_column.html.erb
+++ b/app/views/collections/show/_column.html.erb
@@ -2,6 +2,7 @@
   class="cards cards--doing is-collapsed" style="--card-color: <%= column.color %> %>"
   data-collapsible-columns-target="column"
   data-drag-and-drop-target="container"
+  data-action="turbo:before-morph-attribute->collapsible-columns#preventToggle"
   data-drag-and-drop-url="<%= columns_card_drops_column_path("__id__", column_id: column.id) %>"
 >
   <%= cards_expander column.name, column.cards.active.count %>
diff --git a/app/views/collections/show/_not_now.html.erb b/app/views/collections/show/_not_now.html.erb
index 3a8e27657..62ff7e0d5 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="now-now-cards" class="cards cards--on-deck is-collapsed" style="--card-color: var(--color-card-complete);"
+<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-action="turbo:before-morph-attribute->collapsible-columns#preventToggle"
   data-drag-and-drop-url="<%= columns_card_drops_not_now_path("__id__") %>"
 >
 
diff --git a/app/views/collections/show/_stream.html.erb b/app/views/collections/show/_stream.html.erb
index fcbf10a46..5eea73531 100644
--- a/app/views/collections/show/_stream.html.erb
+++ b/app/views/collections/show/_stream.html.erb
@@ -1,5 +1,6 @@
 <section id="the-stream" class="cards cards--considering"
   data-drag-and-drop-target="container"
+  data-action="turbo:before-morph-attribute->collapsible-columns#preventToggle"
   data-drag-and-drop-url="<%= columns_card_drops_stream_path("__id__") %>">
   <div class="cards__decoration"></div>