chore: adjusted create strand overlay implementation in strands live …

…view

controlling overlay visibility with assigns instead of routes, avoiding complex handling of strands loading

lib/lanttern_web/live/pages/strands/strands_live.ex

@@ -51,6 +51,7 @@ defmodule LantternWeb.StrandsLive do
   def mount(_params, _session, socket) do
     {:ok,
      socket
+     |> assign(:is_creating_strand, false)
      |> assign(:current_subjects, [])
      |> assign(:current_years, [])}
   end
@@ -73,6 +74,14 @@ defmodule LantternWeb.StrandsLive do
   # event handlers
 
   @impl true
+  def handle_event("create-strand", _params, socket) do
+    {:noreply, assign(socket, :is_creating_strand, true)}
+  end
+
+  def handle_event("cancel-strand-creation", _params, socket) do
+    {:noreply, assign(socket, :is_creating_strand, false)}
+  end
+
   def handle_event("load-more", _params, socket) do
     {:noreply, load_strands(socket)}
   end
@@ -101,13 +110,6 @@ defmodule LantternWeb.StrandsLive do
      |> push_navigate(to: path(socket, ~p"/strands?#{params}"))}
   end
 
-  # info handlers
-
-  @impl true
-  def handle_info({StrandFormComponent, {:saved, strand}}, socket) do
-    {:noreply, stream_insert(socket, :strands, strand)}
-  end
-
   # helpers
 
   defp handle_assigns(socket, params) do

lib/lanttern_web/live/pages/strands/strands_live.html.heex

@@ -6,7 +6,7 @@
       <.filter_buttons type="years" items={@current_years} />,
       <.filter_buttons type="subjects" items={@current_subjects} />
     </p>
-    <.collection_action type="link" patch={~p"/strands/new"} icon_name="hero-plus-circle">
+    <.collection_action type="button" icon_name="hero-plus-circle" phx-click="create-strand">
       Create new strand
     </.collection_action>
   </div>
@@ -92,16 +92,22 @@
   </:actions>
 </.slide_over>
 <.slide_over
-  :if={@live_action == :new}
+  :if={@is_creating_strand}
   id="strand-form-overlay"
   show={true}
-  on_cancel={JS.patch(~p"/strands")}
+  on_cancel={JS.push("cancel-strand-creation")}
 >
   <:title>New strand</:title>
   <.live_component
     module={StrandFormComponent}
     id={:new}
-    strand={%Strand{curriculum_items: [], subjects: [], years: []}}
+    strand={
+      %Strand{
+        curriculum_items: [],
+        subjects: @current_subjects,
+        years: @current_years
+      }
+    }
     action={:new}
     navigate={fn strand -> ~p"/strands/#{strand}" end}
   />

test/lanttern_web/live/pages/strands/strands_live_test.exs

@@ -52,8 +52,7 @@ defmodule LantternWeb.StrandsLiveTest do
       {:ok, view, _html} = live(conn, @live_view_path)
 
       # open create strand overlay
-      view |> element("a", "Create new strand") |> render_click()
-      assert_patch(view, "/strands/new")
+      view |> element("button", "Create new strand") |> render_click()
       assert view |> has_element?("h2", "New strand")
 
       # add subject