From 2934eb03f903fab2a2ee9e17e5f9b4cc7a7a4e92 Mon Sep 17 00:00:00 2001
From: Jason Zimdars <jz@37signals.com>
Date: Fri, 11 Apr 2025 13:04:19 -0500
Subject: [PATCH] Some actions shouldn't be available to closed cards

---
 app/views/cards/_image.html.erb                    | 2 +-
 app/views/cards/display/common/_assignees.html.erb | 2 +-
 app/views/cards/display/perma/_tags.html.erb       | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/app/views/cards/_image.html.erb b/app/views/cards/_image.html.erb
index b7b037be1..c1617792e 100644
--- a/app/views/cards/_image.html.erb
+++ b/app/views/cards/_image.html.erb
@@ -3,7 +3,7 @@
     <%= icon_tag "picture-remove" %>
     <span class="for-screen-reader">Clear background image</span>
   <% end %>
-<% else %>
+<% elsif !card.closed? %>
   <%= form_with model: card, url: collection_card_path(card.collection, card), data: { controller: "form" } do |form| %>
     <label class="btn">
       <%= icon_tag "picture-add" %>
diff --git a/app/views/cards/display/common/_assignees.html.erb b/app/views/cards/display/common/_assignees.html.erb
index 28cf1cf34..58b4b67c5 100644
--- a/app/views/cards/display/common/_assignees.html.erb
+++ b/app/views/cards/display/common/_assignees.html.erb
@@ -3,7 +3,7 @@
     <%= avatar_tag assignee, class: "avatar" %>
   <% end %>
 
-  <div class="position-relative" data-controller="dialog" data-action="keydown.esc->dialog#close click@document->dialog#closeOnClickOutside">
+  <div class="position-relative" data-controller="dialog" data-action="keydown.esc->dialog#close click@document->dialog#closeOnClickOutside" <%= "hidden" if card.closed? %>>
     <button class="btn card__hide-on-index <%= card.creating? || card.drafted? ? "fill-highlight" : "fill-transparent" %>" data-action="click->dialog#open:stop">
       <%= icon_tag "add" %>
       <span class="for-screen-reader">Assign</span>
diff --git a/app/views/cards/display/perma/_tags.html.erb b/app/views/cards/display/perma/_tags.html.erb
index f600986c0..7d57385a6 100644
--- a/app/views/cards/display/perma/_tags.html.erb
+++ b/app/views/cards/display/perma/_tags.html.erb
@@ -11,7 +11,7 @@
     <% end %>
   </div>
 
-  <div class="position-relative" data-controller="dialog" data-action="keydown.esc->dialog#close click@document->dialog#closeOnClickOutside">
+  <div class="position-relative" data-controller="dialog" data-action="keydown.esc->dialog#close click@document->dialog#closeOnClickOutside" <%= "hidden" if card.closed? %>>
     <button class="tag-picker__button btn card__hide-on-index <%= card.creating? || card.drafted? ? "fill-highlight" : "fill-transparent" %>" data-action="click->dialog#open:stop">
       <%= icon_tag "tag" %>
       <span class="for-screen-reader">Add a tag</span>