Keep plan navigation state current

Amp-Thread-ID: https://ampcode.com/threads/T-01a01147-8d44-7468-ac98-bf40b04e3cee
Co-authored-by: Amp <amp@ampcode.com>

Author: HamptonMakes

engine/app/controllers/coplan/plans_controller.rb

@@ -687,6 +687,7 @@ def set_plan
 
     def broadcast_plan_update(plan)
       Broadcaster.replace_to(plan, target: "plan-header", partial: "coplan/plans/header", locals: { plan: plan })
+      Broadcaster.replace_to(plan, target: "plan-nav-context", partial: "coplan/plans/nav_context", locals: { plan: plan })
     end
 
     # Turbo Streams for a visibility change: re-render the header (the
@@ -708,6 +709,7 @@ def visibility_streams(message)
     def archive_streams(message)
       [
         turbo_stream.replace("plan-header", partial: "coplan/plans/header", locals: { plan: @plan }),
+        turbo_stream.replace("plan-nav-context", partial: "coplan/plans/nav_context", locals: { plan: @plan }),
         turbo_stream.replace("plan-banner-slot", partial: "coplan/plans/banner", locals: { plan: @plan }),
         turbo_stream.replace("plan-toolbar", partial: "coplan/plans/toolbar", locals: { plan: @plan }),
         toast_stream(message, "notice")

engine/app/views/coplan/plans/_nav_context.html.erb

@@ -0,0 +1,8 @@
+<% author_placement = local_assigns.fetch(:author_placement) { plan.author_placement } %>
+<span id="plan-nav-context" class="site-nav__plan-context" data-controller="coplan--nav-title" aria-hidden="true">
+  <%= render "coplan/plans/location_link", plan: plan, placement: author_placement, location: :nav %>
+  <a href="#plan-header" class="site-nav__doc-title" data-action="click->coplan--nav-title#scrollToTop" tabindex="-1">
+    <%= plan_type_icon(plan, size: :sm) %>
+    <span class="site-nav__doc-title-text"><%= plan.title %></span>
+  </a>
+</span>

engine/app/views/coplan/plans/show.html.erb

@@ -16,13 +16,7 @@
     containing-folder control, icon, and title mirror the masthead;
     coplan--nav-title fades them in once #plan-header scrolls behind the bar. %>
 <% content_for :nav_title do %>
-  <span class="site-nav__plan-context" data-controller="coplan--nav-title" aria-hidden="true">
-    <%= render "coplan/plans/location_link", plan: @plan, placement: @author_placement, location: :nav %>
-    <a href="#plan-header" class="site-nav__doc-title" data-action="click->coplan--nav-title#scrollToTop" tabindex="-1">
-      <%= plan_type_icon(@plan, size: :sm) %>
-      <span class="site-nav__doc-title-text"><%= @plan.title %></span>
-    </a>
-  </span>
+  <%= render "coplan/plans/nav_context", plan: @plan, author_placement: @author_placement %>
 <% end %>
 
 <%= turbo_stream_from @plan %>

spec/system/human_editing_spec.rb

@@ -109,6 +109,7 @@ def open_plan_menu
 
   it "archives and restores the plan in place" do
     visit plan_path(plan)
+    expect(page).to have_css(".plan-location-link--nav", visible: :all)
 
     open_plan_menu
     within("#plan-menu") { click_button "Archive plan" }
@@ -117,6 +118,7 @@ def open_plan_menu
     # undo, no navigation away from the document.
     expect(page).to have_css(".plan-banner--archived", text: "hidden from lists")
     expect(page).to have_content("Editable Plan")
+    expect(page).not_to have_css(".plan-location-link", visible: :all)
     expect(plan.reload.archived?).to be(true)
 
     # Archive leaves the menu while archived.
@@ -126,6 +128,7 @@ def open_plan_menu
 
     within(".plan-banner--archived") { click_button "Restore" }
     expect(page).not_to have_css(".plan-banner--archived")
+    expect(page).to have_css(".plan-location-link--nav", visible: :all)
     expect(plan.reload.archived?).to be(false)
   end