Revert "Revert "Color a11y""

app/assets/stylesheets/_global.css

@@ -209,14 +209,6 @@
   /* Colors: Cards */
   --color-card-default: oklch(var(--lch-blue-dark));
   --color-card-complete: var(--color-ink-darker);
-  --color-card-1: oklch(var(--lch-ink-medium));
-  --color-card-2: oklch(var(--lch-uncolor-medium));
-  --color-card-3: oklch(var(--lch-yellow-medium));
-  --color-card-4: oklch(var(--lch-lime-medium));
-  --color-card-5: oklch(var(--lch-aqua-medium));
-  --color-card-6: oklch(var(--lch-violet-medium));
-  --color-card-7: oklch(var(--lch-purple-medium));
-  --color-card-8: oklch(var(--lch-pink-medium));
 
   /* Colors: Syntax highlighting */
   --color-code-token__att: oklch(var(--lch-blue-dark));

app/models/card/colored.rb

@@ -1,24 +1,7 @@
 module Card::Colored
   extend ActiveSupport::Concern
 
-  COLORS = %w[
-    var(--color-card-1)
-    var(--color-card-2)
-    var(--color-card-3)
-    var(--color-card-4)
-    var(--color-card-5)
-    var(--color-card-6)
-    var(--color-card-7)
-    var(--color-card-8)
-  ]
-  DEFAULT_COLOR = "var(--color-card-default)"
-
   def color
-    color_from_column || DEFAULT_COLOR
-  end
-
-  private
-    def color_from_column
-      column&.color&.presence
+    column&.color || Column::Colored::DEFAULT_COLOR
   end
 end

app/models/color.rb

@@ -0,0 +1,25 @@
+Color = Struct.new(:name, :value)
+
+class Color
+  class << self
+    def for_value(value)
+      COLORS.find { |it| it.value == value }
+    end
+  end
+
+  def to_s
+    value
+  end
+
+  COLORS = {
+    "Blue" => "var(--color-card-default)",
+    "Gray" => "oklch(var(--lch-ink-dark))",
+    "Tan" => "oklch(var(--lch-uncolor-medium))",
+    "Yellow" => "oklch(var(--lch-yellow-medium))",
+    "Lime" => "oklch(var(--lch-lime-medium))",
+    "Aqua" => "oklch(var(--lch-aqua-medium))",
+    "Violet" => "oklch(var(--lch-violet-medium))",
+    "Purple" => "oklch(var(--lch-purple-medium))",
+    "Pink" => "oklch(var(--lch-pink-medium))"
+  }.collect { |name, value| new(name, value) }.freeze
+end

app/models/column.rb

@@ -1,10 +1,9 @@
 class Column < ApplicationRecord
-  include Positioned
+  include Colored, Positioned
 
   belongs_to :board, touch: true
   has_many :cards, dependent: :nullify
 
-  before_validation    -> { self.color ||= Card::DEFAULT_COLOR }
   after_save_commit    -> { cards.touch_all }, if: -> { saved_change_to_name? || saved_change_to_color? }
   after_destroy_commit -> { board.cards.touch_all }
 end

app/models/column/colored.rb

@@ -0,0 +1,13 @@
+module Column::Colored
+  extend ActiveSupport::Concern
+
+  DEFAULT_COLOR = Color::COLORS.first
+
+  included do
+    before_validation -> { self[:color] ||= DEFAULT_COLOR.value }
+  end
+
+  def color
+    Color.for_value(super) || DEFAULT_COLOR
+  end
+end

app/views/boards/show/menu/_column_form.html.erb

@@ -3,11 +3,11 @@
         required: true, autocomplete: "off", pattern: ".*\\S.*", title: "Column name cannot be blank", data: { action: "focus->form#select" } %>
 
   <div class="color-picker__colors">
-    <% ([Card::DEFAULT_COLOR] + Card::COLORS).each do |color| %>
-      <label class="btn txt-small borderless" style="--btn-background: <%= color %>" title="<%= color %>">
-        <%= form.radio_button :color, color, checked: (column.color == color || (column.new_record? && color == Card::DEFAULT_COLOR)) %>
+    <% Color::COLORS.each do |color| %>
+      <label class="btn txt-small borderless" style="--btn-background: <%= color %>" title="<%= color.name %>">
+        <%= form.radio_button :color, color.value, checked: (column.color == color || (column.new_record? && color == Column::Colored::DEFAULT_COLOR)) %>
         <%= icon_tag "check", class: "checked" %>
-        <span class="for-screen-reader"><%= color %></span>
+        <span class="for-screen-reader"><%= color.name %></span>
       </label>
     <% end %>
   </div>

test/models/card/colored_test.rb

@@ -3,7 +3,7 @@ require "test_helper"
 class Card::ColoredTest < ActiveSupport::TestCase
   test "use default color if no column" do
     cards(:logo).update! column: nil
-    assert_equal Card::DEFAULT_COLOR, cards(:logo).color
+    assert_equal Column::Colored::DEFAULT_COLOR, cards(:logo).color
   end
 
   test "infer color from column" do

test/models/column/colored_test.rb

@@ -0,0 +1,16 @@
+require "test_helper"
+
+class Column::ColoredTest < ActiveSupport::TestCase
+  test "creates column with default color when color not provided" do
+    column = boards(:writebook).columns.create!(name: "New Column")
+
+    assert_equal Column::Colored::DEFAULT_COLOR, column.color
+  end
+
+  test "update the column color" do
+    columns(:writebook_triage).update!(color: "oklch(var(--lch-yellow-medium))")
+
+    assert_not_nil columns(:writebook_triage).color
+    assert_equal Color.for_value("oklch(var(--lch-yellow-medium))"), columns(:writebook_triage).color
+  end
+end

test/models/column_test.rb

@@ -1,12 +1,6 @@
 require "test_helper"
 
 class ColumnTest < ActiveSupport::TestCase
-  test "creates column with default color when color not provided" do
-    column = boards(:writebook).columns.create!(name: "New Column")
-
-    assert_equal Card::DEFAULT_COLOR, column.color
-  end
-
   test "touch all the cards when the name or color changes" do
     column = columns(:writebook_triage)