From 7925cd7bf969059d6cf5e41baf77ffa44978fb14 Mon Sep 17 00:00:00 2001
From: Jorge Manrubia <jorge@37signals.com>
Date: Mon, 21 Apr 2025 14:52:21 +0200
Subject: [PATCH] Extract partials

---
 app/views/cards/_container.html.erb           | 20 +++----------------
 app/views/cards/container/_edit.html.erb      |  4 ++++
 .../cards/container/footer/_draft.html.erb    |  4 ++++
 .../container/footer/_published.html.erb      |  7 +++++++
 4 files changed, 18 insertions(+), 17 deletions(-)
 create mode 100644 app/views/cards/container/_edit.html.erb
 create mode 100644 app/views/cards/container/footer/_draft.html.erb
 create mode 100644 app/views/cards/container/footer/_published.html.erb

diff --git a/app/views/cards/_container.html.erb b/app/views/cards/_container.html.erb
index 3a1620036..034106e59 100644
--- a/app/views/cards/_container.html.erb
+++ b/app/views/cards/_container.html.erb
@@ -3,12 +3,7 @@
   <%= render "cards/container/engagement", card: card %>
 
   <aside class="card-perma__actions card-perma__actions--left" role="toolbar">
-    <% if card.published? %>
-      <%= link_to edit_collection_card_path(card.collection, card), class: "btn", data: { turbo_frame: dom_id(card, :edit) } do %>
-        <%= icon_tag "pencil" %>
-        <span class="for-screen-reader">Edit</span>
-      <% end %>
-    <% end %>
+    <%= render "cards/container/edit", card: card if card.published? %>
     <%= render "cards/container/gild", card: card if card.doing? %>
     <%= render "cards/container/image", card: card %>
 
@@ -38,17 +33,8 @@
   </div>
 
   <% if card.published? %>
-    <%# FIXME: Let's move this aside outside of the card container section so these frames don't reload/flicker when card is replaced %>
-    <aside class="card-perma__actions card-perma__actions--right" role="toolbar">
-      <%= turbo_frame_tag card, :watch, src: card_watch_path(card) %>
-      <%= turbo_frame_tag card, :pin, src: card_pin_path(card) %>
-    </aside>
-
-    <%= render "cards/container/closure", card: card %>
+    <%= render "cards/container/footer/published", card: card %>
   <% elsif card.creating? %>
-    <div class="card-perma__notch card-perma__notch--bottom">
-      <%= button_to "Create card", card_publish_path(card), class: "btn" %>
-      <%= button_to "Save as draft", collection_card_path(card.collection, card), name: "card[status]", value: "drafted", method: :put, class: "btn btn--reversed" %>
-    </div>
+    <%= render "cards/container/footer/draft", card: card %>
   <% end %>
 </section>
diff --git a/app/views/cards/container/_edit.html.erb b/app/views/cards/container/_edit.html.erb
new file mode 100644
index 000000000..23ea9f48d
--- /dev/null
+++ b/app/views/cards/container/_edit.html.erb
@@ -0,0 +1,4 @@
+<%= link_to edit_collection_card_path(card.collection, card), class: "btn", data: { turbo_frame: dom_id(card, :edit) } do %>
+  <%= icon_tag "pencil" %>
+  <span class="for-screen-reader">Edit</span>
+<% end %>
diff --git a/app/views/cards/container/footer/_draft.html.erb b/app/views/cards/container/footer/_draft.html.erb
new file mode 100644
index 000000000..f46e08b41
--- /dev/null
+++ b/app/views/cards/container/footer/_draft.html.erb
@@ -0,0 +1,4 @@
+<div class="card-perma__notch card-perma__notch--bottom">
+  <%= button_to "Create card", card_publish_path(card), class: "btn" %>
+  <%= button_to "Save as draft", collection_card_path(card.collection, card), name: "card[status]", value: "drafted", method: :put, class: "btn btn--reversed" %>
+</div>
diff --git a/app/views/cards/container/footer/_published.html.erb b/app/views/cards/container/footer/_published.html.erb
new file mode 100644
index 000000000..c4e0a0cdf
--- /dev/null
+++ b/app/views/cards/container/footer/_published.html.erb
@@ -0,0 +1,7 @@
+<%# FIXME: Let's move this aside outside of the card container section so these frames don't reload/flicker when card is replaced %>
+<aside class="card-perma__actions card-perma__actions--right" role="toolbar">
+  <%= turbo_frame_tag card, :watch, src: card_watch_path(card) %>
+  <%= turbo_frame_tag card, :pin, src: card_pin_path(card) %>
+</aside>
+
+<%= render "cards/container/closure", card: card %>