Fix for admin newsletter edit turbo frame ux

Author: rossta

app/controllers/admin/newsletters_controller.rb

@@ -17,6 +17,7 @@ def new
   # GET /admin/newsletters/1/edit
   def edit
     @newsletter = Newsletter.find(params[:id])
+    @newsletter.assign_attributes(newsletter_params)
   end
 
   # POST /admin/newsletters

app/views/admin/newsletters/_form.html.erb

@@ -24,7 +24,7 @@
   <fieldset>
     <%= form.submit class: "button primary" %>
     <%= form.submit "Preview", class: "button secondary",
-      formaction: new_admin_newsletter_path,
+      formaction: (newsletter.persisted? ? edit_admin_newsletter_path(newsletter) : new_admin_newsletter_path),
       formmethod: "get",
       formnovalidate: true,
       data: { turbo_frame: dom_id(newsletter, :markdown) } %>

app/views/admin/newsletters/_newsletter.html.erb

@@ -1,3 +1,3 @@
-<div id="<%= dom_id newsletter %>" class="joy-border p-4">
+<div id="<%= dom_id(newsletter, :markdown) %>" class="article-content joy-border-subtle p-4 mx--4">
   <%= basic_markdown (newsletter&.content || "").html_safe %>
 </div>