Merge pull request #717 from basecamp/allow-goldens-in-considering

Allow making cards Golden in Considering

app/assets/stylesheets/cards.css

@@ -345,7 +345,7 @@
   /* Variants
   /* ------------------------------------------------------------------------ */
 
-  .card--golden.card--doing {
+  .card--golden:is(.card--doing, .card--considering) {
     --lch-golden: 89.1% 0.178 95.7;
 
     background-color: var(--color-canvas);
@@ -363,6 +363,17 @@
     @media (prefers-color-scheme: dark) {
       --lch-golden: var(--lch-blue-medium);
     }
+
+    &.card--considering {
+      .card__collection {
+        background-color: transparent;
+        color: color-mix(in srgb, var(--card-color) 40%, var(--color-ink));
+      }
+
+      .card__collection-name {
+        border-color: color-mix(in srgb, var(--color-ink) 33%, oklch(var(--lch-golden)));
+      }
+    }
   }
 
   .card--notification {

app/helpers/cards_helper.rb

@@ -35,6 +35,7 @@ module CardsHelper
     classes = [
       options.delete(:class),
       ("card--golden" if card.golden?),
+      ("card--considering" if card.considering?),
       ("card--doing" if card.doing?),
       ("card--drafted" if card.drafted?)
     ].compact.join(" ")

app/models/card/engageable.rb

@@ -9,7 +9,7 @@ module Card::Engageable
 
     scope :by_engagement_status, ->(status) do
       case status.to_s
-      when "considering" then considering
+      when "considering" then considering.with_golden_first
       when "doing"       then doing.with_golden_first
       end
     end

app/views/cards/_container.html.erb

@@ -3,7 +3,7 @@
   <%= render "cards/container/engagement", card: card %>
 
   <aside class="card-perma__actions card-perma__actions--left" role="toolbar">
-    <%= render "cards/container/gild", card: card if card.doing? %>
+    <%= render "cards/container/gild", card: card if @card.published? && !@card.closed? %>
     <%= render "cards/container/image", card: card %>
   </aside>