From df4ca83aeb98c943f90093d919824d5c33ec94c1 Mon Sep 17 00:00:00 2001
From: Claudio Ortolina <cloud8421@gmail.com>
Date: Wed, 27 Aug 2025 21:11:13 +0300
Subject: [PATCH] Add ability to rotate dominant colors

---
 lib/music_library/records/record.ex           | 12 +++++++++++
 .../components/form_component.ex              | 17 +++++++++++++---
 priv/gettext/default.pot                      |  5 +++++
 priv/gettext/en/LC_MESSAGES/default.po        |  5 +++++
 test/music_library/records/record_test.exs    | 20 +++++++++++++++++++
 5 files changed, 56 insertions(+), 3 deletions(-)

diff --git a/lib/music_library/records/record.ex b/lib/music_library/records/record.ex
index 9d29de20..bad7f517 100644
--- a/lib/music_library/records/record.ex
+++ b/lib/music_library/records/record.ex
@@ -188,6 +188,18 @@ defmodule MusicLibrary.Records.Record do
     end
   end
 
+  def rotate_dominant_colors(%__MODULE__{dominant_colors: dominant_colors} = record) do
+    change(record, dominant_colors: rotate(dominant_colors))
+  end
+
+  def rotate_dominant_colors(changeset) do
+    dominant_colors = get_field(changeset, :dominant_colors)
+    put_change(changeset, :dominant_colors, rotate(dominant_colors))
+  end
+
+  defp rotate([]), do: []
+  defp rotate([first | rest]), do: rest ++ [first]
+
   defp update_release_ids(changeset) do
     case get_change(changeset, :musicbrainz_data) do
       nil ->
diff --git a/lib/music_library_web/components/form_component.ex b/lib/music_library_web/components/form_component.ex
index e80edd2c..04ccf6eb 100644
--- a/lib/music_library_web/components/form_component.ex
+++ b/lib/music_library_web/components/form_component.ex
@@ -60,9 +60,15 @@ defmodule MusicLibraryWeb.FormComponent do
           />
         </div>
         <div class="space-y-4">
-          <.label for="dominant_colors">
-            {gettext("Dominant Colors")}
-          </.label>
+          <div class="flex items-center gap-x-2">
+            <.label for="dominant_colors">
+              {gettext("Dominant Colors")}
+            </.label>
+            <.button type="button" size="sm" phx-click="rotate_dominant_colors" phx-target={@myself}>
+              <.icon name="hero-arrows-right-left" class="icon" aria-hidden="true" data-slot="icon" />
+              {gettext("Rotate colors")}
+            </.button>
+          </div>
           <div class="mt-2 grid grid-cols-5 gap-2">
             <div :for={color <- @form[:dominant_colors].value} class="flex flex-col items-center">
               <input
@@ -181,6 +187,11 @@ defmodule MusicLibraryWeb.FormComponent do
     handle_event("validate", params, socket)
   end
 
+  def handle_event("rotate_dominant_colors", _params, socket) do
+    changeset = Record.rotate_dominant_colors(socket.assigns.form.source)
+    {:noreply, assign(socket, form: to_form(changeset, action: :validate))}
+  end
+
   defp save_record(socket, record_params, uploaded_covers) do
     params =
       case uploaded_covers do
diff --git a/priv/gettext/default.pot b/priv/gettext/default.pot
index 2f47a850..6527f20b 100644
--- a/priv/gettext/default.pot
+++ b/priv/gettext/default.pot
@@ -1290,3 +1290,8 @@ msgstr ""
 #, elixir-autogen, elixir-format
 msgid "90 days"
 msgstr ""
+
+#: lib/music_library_web/components/form_component.ex
+#, elixir-autogen, elixir-format
+msgid "Rotate colors"
+msgstr ""
diff --git a/priv/gettext/en/LC_MESSAGES/default.po b/priv/gettext/en/LC_MESSAGES/default.po
index 47ccbfe5..78bb1d17 100644
--- a/priv/gettext/en/LC_MESSAGES/default.po
+++ b/priv/gettext/en/LC_MESSAGES/default.po
@@ -1290,3 +1290,8 @@ msgstr ""
 #, elixir-autogen, elixir-format
 msgid "90 days"
 msgstr ""
+
+#: lib/music_library_web/components/form_component.ex
+#, elixir-autogen, elixir-format
+msgid "Rotate colors"
+msgstr ""
diff --git a/test/music_library/records/record_test.exs b/test/music_library/records/record_test.exs
index ff8f1e3b..0d3763ba 100644
--- a/test/music_library/records/record_test.exs
+++ b/test/music_library/records/record_test.exs
@@ -117,4 +117,24 @@ defmodule MusicLibrary.Records.RecordTest do
              ]
     end
   end
+
+  describe "rotate_dominant_colors/1" do
+    test "rotates dominant colors" do
+      record = %Record{dominant_colors: []}
+      changeset = Record.rotate_dominant_colors(record)
+      assert Ecto.Changeset.get_change(changeset, :dominant_colors) == nil
+
+      record = %Record{dominant_colors: ["#FFF"]}
+      changeset = Record.rotate_dominant_colors(record)
+      assert Ecto.Changeset.get_change(changeset, :dominant_colors) == nil
+
+      record = %Record{dominant_colors: ["#FFF", "#CCC"]}
+      changeset = Record.rotate_dominant_colors(record)
+      assert Ecto.Changeset.get_change(changeset, :dominant_colors) == ["#CCC", "#FFF"]
+
+      record = %Record{dominant_colors: ["#FFF", "#CCC", "#000"]}
+      changeset = Record.rotate_dominant_colors(record)
+      assert Ecto.Changeset.get_change(changeset, :dominant_colors) == ["#CCC", "#000", "#FFF"]
+    end
+  end
 end