From aef75feead9a1c0fae3e47bcf6fea2d9cc8263a2 Mon Sep 17 00:00:00 2001
From: Jorge Manrubia <jorge@37signals.com>
Date: Fri, 11 Apr 2025 11:45:06 +0200
Subject: [PATCH] Only consider golden those cards in "doing"

---
 app/helpers/cards_helper.rb               | 4 ++++
 app/views/cards/display/_perma.html.erb   | 2 +-
 app/views/cards/display/_preview.html.erb | 2 +-
 3 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/app/helpers/cards_helper.rb b/app/helpers/cards_helper.rb
index ba75a6d4a..8c9cc0498 100644
--- a/app/helpers/cards_helper.rb
+++ b/app/helpers/cards_helper.rb
@@ -26,4 +26,8 @@ module CardsHelper
       class: "btn txt-small",
       **options
   end
+
+  def golden_class_for(card)
+    "card--golden" if card.doing? && card.golden?
+  end
 end
diff --git a/app/views/cards/display/_perma.html.erb b/app/views/cards/display/_perma.html.erb
index bdc1a3c4d..adba7fb7f 100644
--- a/app/views/cards/display/_perma.html.erb
+++ b/app/views/cards/display/_perma.html.erb
@@ -1,5 +1,5 @@
 <% cache card do %>
-  <article class="card shadow border flex flex-column position-relative txt-align-start border-radius <%= 'card--golden' if card.golden? %>"
+  <article class="card shadow border flex flex-column position-relative txt-align-start border-radius <%= golden_class_for(card) %>"
       style="--card-color: <%= card.color %>; view-transition-name: <%= dom_id(card, :ticket) %>;"
       id="<%= dom_id(card, :ticket) %>">
     <header class="card__header flex align-center gap min-width">
diff --git a/app/views/cards/display/_preview.html.erb b/app/views/cards/display/_preview.html.erb
index c8fcbe8c5..3d59c8fb7 100644
--- a/app/views/cards/display/_preview.html.erb
+++ b/app/views/cards/display/_preview.html.erb
@@ -1,5 +1,5 @@
 <% cache [ card ] do %>
-  <article class="card shadow border flex flex-column position-relative txt-align-start full-width border-radius <%= 'card--golden' if card.golden? %>"
+  <article class="card shadow border flex flex-column position-relative txt-align-start full-width border-radius <%= golden_class_for(card) %>"
     style="--card-color: <%= card.color %>; view-transition-name: <%= dom_id(card, :ticket) %>;"
     id="<%= dom_id(card, :ticket) %>">
     <header class="card__header flex align-center gap min-width">