Fix layout rendering for turbo_streams

Closes #22

app/controllers/application_controller.rb

@@ -1,4 +1,11 @@
 class ApplicationController < ActionController::Base
+  # Turbo-rails disables layout rendering for turbo_frame requests (i.e.
+  # requests that specify 'turbo-frame:' header).
+  # As a side effect, this also disables layout for turbo_stream requests that
+  # happen to originate from within turbo frame (e.g. turbo_frame_tag or tag
+  # with 'is="turbo-frame"' attribute). To fix this, either frame tags must not be
+  # used, or custom layout method needs to be defined.
+
   helper_method :current_user_disguised?
 
   before_action :authenticate_user!
@@ -47,8 +54,6 @@ class ApplicationController < ActionController::Base
 
   def run_and_render(action)
     send action
-    # 2024-01-17, Rails 7.1.2: For unknown reason turbo_stream layout is omitted
-    # during render on POST method only (GET, DESTROY are ok).
-    render action, layout: 'application'
+    render action
   end
 end

app/views/units/edit.turbo_stream.erb

@@ -1,5 +1,4 @@
-<%# TODO: make sure turbo_stream layout is used in new/edit %>
-<%= turbo_stream.update :unit_form_frame do %>
+<%= turbo_stream.replace :unit_form do %>
   <%= form_with model: @unit, html: {id: :unit_form} do %>
   <% end %>
 <% end %>

app/views/units/index.html.erb

@@ -1,13 +1,11 @@
 <div class="rightside">
   <% if current_user.at_least(:active) %>
-    <%= turbo_frame_tag do %>
-      <%= image_link_to t('.add_unit'), 'plus-outline', new_unit_path, id: :add_unit,
-        onclick: 'this.blur();', data: {turbo_stream: true} %>
-    <% end %>
+    <%= image_link_to t('.add_unit'), 'plus-outline', new_unit_path, id: :add_unit,
+      onclick: 'this.blur();', data: {turbo_stream: true} %>
   <% end %>
 </div>
 
-<%= turbo_frame_tag 'unit_form_frame' %>
+<%= tag.div id: :unit_form %>
 
 <table class="main items">
   <thead>
@@ -20,7 +18,7 @@
       <% end %>
     </tr>
   </thead>
-  <tbody id="units" is="turbo-frame">
+  <tbody id="units">
     <%= render(@units) || render_no_items %>
   </tbody>
 </table>

app/views/units/new.turbo_stream.erb

@@ -1,7 +1,7 @@
 <% link_id = dom_id(@unit.base || @unit, :add) %>
 <%= turbo_stream.disable link_id -%>
 
-<%= turbo_stream.update :unit_form_frame do %>
+<%= turbo_stream.replace :unit_form do %>
   <%= form_with model: @unit, html: {id: :unit_form} do %>
   <% end %>
 <% end %>