From 41a9ddcb1e8f3ad8025c645f63e32651b3186293 Mon Sep 17 00:00:00 2001
From: Miguel Camba <miguel.camba@gmail.com>
Date: Thu, 28 Nov 2024 18:08:59 +0100
Subject: [PATCH] It works

---
 .../visual_editor/opacity_control.ex          | 44 ++++++++++++-------
 .../live/page_editor_live/form_component.ex   | 14 +-----
 2 files changed, 29 insertions(+), 29 deletions(-)

diff --git a/lib/beacon/live_admin/components/visual_editor/opacity_control.ex b/lib/beacon/live_admin/components/visual_editor/opacity_control.ex
index cae72264..49125a01 100644
--- a/lib/beacon/live_admin/components/visual_editor/opacity_control.ex
+++ b/lib/beacon/live_admin/components/visual_editor/opacity_control.ex
@@ -20,37 +20,49 @@ require Logger
     {:ok, assign_form(socket, "")}
   end
 
+  defp find_opacity_class(classes) do
+    classes
+    |> String.split(" ", trim: true)
+    |> Enum.find(fn s -> String.starts_with?(s, "opacity-") end)
+  end
+
+  defp extract_opacity(classes) do
+    case find_opacity_class(classes) do
+      nil -> nil
+      class ->
+        class
+        |> String.split("-")
+        |> List.last()
+    end
+  end
+
   def update(%{element: element}, socket) do
     path = Map.get(element, "path", "")
 
-    # TODO: extract opacity value from `class`
-    _class = get_in(element, ["attrs", "class"]) || ""
-    opacity = "100"
-    dbg(element)
-    dbg(path)
+    classes = get_in(element, ["attrs", "class"]) || ""
+    opacity = extract_opacity(classes) || "100";
     {:ok,
      socket
      |> assign(path: path)
+     |> assign(classes: classes)
      |> assign_form(opacity)}
   end
 
   # TODO: validate opacity value is valid is valid
   def handle_event("update", %{"value" => opacity}, socket) do
-    Logger.debug("#########################################################")
-    Logger.debug("#########################################################")
-    Logger.debug("#########################################################")
-    Logger.debug("#########################################################")
-    Logger.debug("################## handle_event(update) #################")
-    dbg(socket.assigns)
-    %{path: path} = socket.assigns
-    class = build_class(opacity)
+    %{path: path, classes: classes} = socket.assigns
+    class = build_class(classes, opacity)
     send(self(), {:updated_element, %{path: path, attrs: %{"class" => class}}})
-    Logger.debug("################## handle_event(update) last line #################")
     {:noreply, assign_form(socket, opacity)}
   end
 
-  defp build_class(opacity) do
-    "opacity-#{opacity}"
+  defp build_class(classes, opacity) do
+    other_classes =
+      classes
+      |> String.split(" ", trim: true)
+      |> Enum.reject(fn s -> String.starts_with?(s, "opacity-") end)
+    new_class = "opacity-#{opacity}"
+    Enum.join([new_class | other_classes], " ")
   end
 
   defp assign_form(socket, value) do
diff --git a/lib/beacon/live_admin/live/page_editor_live/form_component.ex b/lib/beacon/live_admin/live/page_editor_live/form_component.ex
index bb5bdadb..3e6e8a88 100644
--- a/lib/beacon/live_admin/live/page_editor_live/form_component.ex
+++ b/lib/beacon/live_admin/live/page_editor_live/form_component.ex
@@ -84,19 +84,7 @@ defmodule Beacon.LiveAdmin.PageEditorLive.FormComponent do
   end
 
   defp update_node(node, attrs) do
-    updated_attrs = Map.merge(node["attrs"], attrs, fn
-      "class", old_val, new_val -> merge_classes(old_val, new_val)
-      _key, _old_val, new_val -> new_val
-    end)
-
-    %{node | "attrs" => updated_attrs}
-  end
-
-  defp merge_classes(old_classes, new_classes) do
-    old_classes_list = String.split(old_classes, " ")
-    new_classes_list = String.split(new_classes, " ")
-    merged_classes_list = Enum.uniq(old_classes_list ++ new_classes_list)
-    Enum.join(merged_classes_list, " ")
+    %{node | "attrs" => Map.merge(node["attrs"], attrs)}
   end
 
   defp update_node_recursive(nodes, [index], attrs) do