inline_forms 7.2.11 → 7.5.2

data/app/helpers/form_elements/plain_text.rb

@@ -0,0 +1,23 @@
+# -*- encoding : utf-8 -*-
+InlineForms::SPECIAL_COLUMN_TYPES[:plain_text]=:text
+
+def plain_text_show(object, attribute)
+  InlineForms.assert_plain_text_column!(object: object, attribute: attribute, form_element: :plain_text)
+  value = object[attribute]
+  link_to_inline_edit object, attribute, (value.nil? || value.empty?) ? "<i class='fi-plus'></i>".html_safe : value, from_callee: __callee__
+end
+
+def plain_text_edit(object, attribute)
+  InlineForms.assert_plain_text_column!(object: object, attribute: attribute, form_element: :plain_text)
+  text_area_tag attribute, object[attribute], :class => 'attribute_text_area'
+end
+
+def plain_text_update(object, attribute)
+  InlineForms.assert_plain_text_column!(object: object, attribute: attribute, form_element: :plain_text)
+  object[attribute.to_sym] = params[attribute.to_sym]
+end
+
+def plain_text_info(object, attribute)
+  InlineForms.assert_plain_text_column!(object: object, attribute: attribute, form_element: :plain_text)
+  object[attribute]
+end

data/app/helpers/form_elements/plain_text_area.rb

@@ -2,13 +2,17 @@
 InlineForms::SPECIAL_COLUMN_TYPES[:plain_text_area]=:text
 
 def plain_text_area_show(object, attribute)
-  link_to_inline_edit object, attribute, (object[attribute].nil? || object[attribute].empty?) ? "<i class='fi-plus'></i>".html_safe : object[attribute], from_callee: __callee__
+  plain_text_show(object, attribute)
 end
 
 def plain_text_area_edit(object, attribute)
-  text_area_tag attribute, object[attribute], :class => 'attribute_text_area'
+  plain_text_edit(object, attribute)
 end
 
 def plain_text_area_update(object, attribute)
-  object[attribute.to_sym] = params[attribute.to_sym]
+  plain_text_update(object, attribute)
+end
+
+def plain_text_area_info(object, attribute)
+  plain_text_info(object, attribute)
 end

data/app/helpers/form_elements/text_area.rb

@@ -2,57 +2,17 @@
 InlineForms::SPECIAL_COLUMN_TYPES[:text_area]=:text
 
 def text_area_show(object, attribute)
-  if object.send(attribute).blank?
-    link_to_inline_edit object, attribute, "<i class='fi-plus'></i>".html_safe, from_callee: __callee__
-  else
-    if defined? Ckeditor
-      link_to_inline_edit object,
-        attribute,
-        '<div class="ckeditor_area">'.html_safe +
-        cktext_area_tag(
-        attribute,
-        object[attribute],
-        :id => "textarea_#{object.class.name.underscore}_#{object.id}_#{attribute.to_s}",
-        :ckeditor => {  :width => '100%',
-                        :height => '200px',
-                        :toolbar => "None",
-                        :readOnly => "true",
-                        :resize_enabled => "false",
-                        :toolbarCanCollapse => "false"
-                      }
-      ) +
-        image_tag(  'inline_forms/glass_plate.gif',
-        :class => "glass_plate",
-        :title => '' ) +
-        "<script>delete CKEDITOR.instances['textarea_#{object.class.name.underscore}_#{object.id}_#{attribute.to_s}']</script>".html_safe +
-        '</div>'.html_safe,
-        from_callee: __callee__
-    else
-      link_to_inline_edit object, attribute, object[attribute], from_callee: __callee__
-    end
-  end
+  rich_text_show(object, attribute)
 end
 
 def text_area_edit(object, attribute)
-  if defined? Ckeditor
-    cktext_area_tag(
-      attribute,
-      object[attribute],
-      :id => "textarea_#{object.class.name.underscore}_#{object.id}_#{attribute.to_s}",
-      :ckeditor => {  :width => '100%',
-                      :height => '200px'
-                    }
-    )  +
-    "<script>delete CKEDITOR.instances['textarea_#{object.class.name.underscore}_#{object.id}_#{attribute.to_s}']</script>".html_safe
-  else
-    text_area_tag attribute, object[attribute], :class => 'attribute_text_area'
-  end
+  rich_text_edit(object, attribute)
 end
 
 def text_area_update(object, attribute)
-  object[attribute.to_sym] = params[attribute.to_sym]
+  rich_text_update(object, attribute)
 end
 
 def text_area_info(object, attribute)
-  object[attribute]
+  rich_text_info(object, attribute)
 end

data/app/helpers/form_elements/text_area_without_ckeditor.rb

@@ -1,18 +1,19 @@
 # -*- encoding : utf-8 -*-
+# Legacy alias for plain :text_area (CKEditor removed).
 InlineForms::SPECIAL_COLUMN_TYPES[:text_area_without_ckeditor]=:text
 
 def text_area_without_ckeditor_show(object, attribute)
-  link_to_inline_edit object, attribute, (object[attribute].nil? || object[attribute].empty?) ? "<i class='fi-plus'></i>".html_safe : object[attribute], from_callee: __callee__
+  plain_text_show(object, attribute)
 end
 
 def text_area_without_ckeditor_edit(object, attribute)
-  text_area_tag attribute, object[attribute], :class => 'attribute_text_area'
+  plain_text_edit(object, attribute)
 end
 
 def text_area_without_ckeditor_update(object, attribute)
-  object[attribute.to_sym] = params[attribute.to_sym]
+  plain_text_update(object, attribute)
 end
 
 def text_area_without_ckeditor_info(object, attribute)
-  object[attribute]
+  plain_text_info(object, attribute)
 end

data/app/helpers/form_elements/text_field.rb

@@ -1,8 +1,8 @@
 # -*- encoding : utf-8 -*-
 InlineForms::SPECIAL_COLUMN_TYPES[:text_field]=:string
 
-def text_field_show(object, attribute)
-  link_to_inline_edit object, attribute, object[attribute].blank? ? "<i class='fi-plus'></i>".html_safe : object[attribute], from_callee: __callee__
+def text_field_show(object, attribute, turbo_frame: false)
+  link_to_inline_edit object, attribute, object[attribute].blank? ? "<i class='fi-plus'></i>".html_safe : object[attribute], from_callee: __callee__, turbo_frame: turbo_frame
 end
 
 def text_field_edit(object, attribute)

data/app/helpers/inline_forms_helper.rb

@@ -52,6 +52,14 @@ module InlineFormsHelper
     entries.sort_by { |entry| entry[:version].created_at }
   end
 
+  # Turbo Frames navigation for row toolbar, versions, and nested +new+ (Step 3).
+  # +update_span+ must match the target +<turbo-frame id="…">+.
+  def inline_forms_turbo_link_data(update_span, method: :get)
+    data = { turbo: true, turbo_frame: update_span }
+    data[:turbo_method] = method.to_s.downcase unless method == :get
+    { data: data }
+  end
+
   private
 
   def validation_hints_as_list_for(object, attribute)
@@ -59,69 +67,74 @@ module InlineFormsHelper
   end
 
   # close link
-  def close_link( object, update_span, html_class = 'button close_button' )
-    link_to "<i class='fi-x'></i>".html_safe,
-      polymorphic_path(
-        object,
-        :update => update_span,
-        :close => true ),
-      :remote => true,
-      :class => html_class,
-      :title => t('inline_forms.view.close')
+  def close_link(object, update_span, html_class = "button close_button", turbo_row: false)
+    path = polymorphic_path(
+      object,
+      update: update_span,
+      close: true
+    )
+    opts = { class: html_class, title: t("inline_forms.view.close") }
+    if turbo_row
+      opts[:data] = { turbo: true, turbo_frame: "_self" }
+      link_to "<i class='fi-x'></i>".html_safe, path, opts
+    else
+      link_to "<i class='fi-x'></i>".html_safe, path, opts.merge(remote: true)
+    end
   end
 
   # delete link. Mind the difference between delete and destroy.
-  def link_to_soft_delete( object, update_span )
-    soft=''
+  def link_to_soft_delete(object, update_span, turbo_row: true)
+    soft = ""
     if (object.soft_deletable? rescue false)
-      if object.deleted?     && (cancan_disabled? || ( can? :soft_restore, object ))
-        soft = link_to "<i class='fi-refresh'></i>".html_safe,
-          send( 'soft_restore_' + object.class.to_s.underscore + '_path',
-          object,
-          :update => update_span ),
-          :method => :post,
-          :remote => true,
-          :title => t('inline_forms.view.undelete')
-      elsif !object.deleted? && (cancan_disabled? || ( can? :soft_delete, object ))
-        soft = link_to "<i class='fi-trash'></i>".html_safe,
-        send( 'soft_delete_' + object.class.to_s.underscore + '_path',
-        object,
-        :update => update_span ),
-        :method => :post,
-        :remote => true,
-        :title => t('inline_forms.view.trash')
+      if object.deleted? && (cancan_disabled? || (can? :soft_restore, object))
+        path = send("soft_restore_#{object.class.to_s.underscore}_path", object, update: update_span)
+        opts = { title: t("inline_forms.view.undelete") }
+        opts.merge!(turbo_row ? inline_forms_turbo_link_data(update_span, method: :post) : { method: :post, remote: true })
+        soft = link_to "<i class='fi-refresh'></i>".html_safe, path, opts
+      elsif !object.deleted? && (cancan_disabled? || (can? :soft_delete, object))
+        path = send("soft_delete_#{object.class.to_s.underscore}_path", object, update: update_span)
+        opts = { title: t("inline_forms.view.trash") }
+        opts.merge!(turbo_row ? inline_forms_turbo_link_data(update_span, method: :post) : { method: :post, remote: true })
+        soft = link_to "<i class='fi-trash'></i>".html_safe, path, opts
       end
     end
     soft.html_safe
   end
 
   # destroy link. Mind the difference between delete and destroy.
-  def link_to_destroy( object, update_span )
-    hard=''
-    if cancan_disabled? || ( can? :destroy, object )
-      hard = link_to "&nbsp;&nbsp;<font color='FF0000'><i class='fi-x'></i></font>".html_safe,
-        polymorphic_path(
-          object,
-          :update => update_span ),
-        :method => :delete,
-        :remote => true,
-        :title => t('inline_forms.view.trash')
+  def link_to_destroy(object, update_span, turbo_row: true)
+    hard = ""
+    if cancan_disabled? || (can? :destroy, object)
+      path = polymorphic_path(object, update: update_span)
+      opts = { title: t("inline_forms.view.trash") }
+      opts.merge!(turbo_row ? inline_forms_turbo_link_data(update_span, method: :delete) : { method: :delete, remote: true })
+      hard = link_to "&nbsp;&nbsp;<font color='FF0000'><i class='fi-x'></i></font>".html_safe, path, opts
     end
     hard.html_safe
   end
 
   # new link
-  def link_to_new_record(model, path_to_new, update_span, parent_class = nil, parent_id = nil, html_class = 'button new_button')
-    out = (link_to "<i class='fi-plus'></i>".html_safe,
-                   send(path_to_new,
-                        :update => update_span,
-                        :parent_class => parent_class,
-                        :parent_id => parent_id,
-                       ),
-                   :remote => true,
-                   :class => html_class,
-                   :title => t('inline_forms.view.add_new', :model => model.model_name.human )
-          )
+  #
+  # +turbo_row:+ controls Turbo opt-in. Default +true+ matches the nested has_many
+  # contract (parent +<turbo-frame>+ swap). The top-level callsite (+_inline_forms_tabs+)
+  # passes no +parent_class+; for that case we fall back to UJS (+remote: true+) so
+  # +new.js.erb+ / +list.js.erb+ swap +#apartments_list+ contents instead of trying
+  # to navigate a Turbo frame the page does not have. Without this the Turbo data
+  # attributes either logged "Content missing" on cancel/create or fell back to a
+  # full-page navigation that broke the inline-edit experience (7.5.1 regression).
+  def link_to_new_record(model, path_to_new, update_span, parent_class = nil, parent_id = nil, html_class = "button new_button", turbo_row: true)
+    path = send(
+      path_to_new,
+      update: update_span,
+      parent_class: parent_class,
+      parent_id: parent_id
+    )
+    opts = {
+      class: html_class,
+      title: t("inline_forms.view.add_new", model: model.model_name.human)
+    }
+    opts.merge!(turbo_row && parent_class.present? ? inline_forms_turbo_link_data(update_span) : { remote: true })
+    out = link_to "<i class='fi-plus'></i>".html_safe, path, opts
     if cancan_enabled?
       if can? :create, model
         if parent_class.nil?
@@ -136,54 +149,97 @@ module InlineFormsHelper
   end
 
   # link to versions list
-  def link_to_versions_list(path_to_versions_list, object, update_span, html_class = 'button new_button')
+  def link_to_versions_list(path_to_versions_list, object, update_span, html_class = "button new_button", turbo_row: true)
     if can? :list_versions, object
       if defined?(PaperTrail) && object.respond_to?(:versions)
-        out = (link_to "<i class='fi-list'></i>".html_safe,
-                       send(path_to_versions_list,
-                            object,
-                            :update => update_span,
-                           ),
-                       :remote => true,
-                       :class => html_class,
-                       :title => t('inline_forms.view.list_versions')
-              )
-        raw out
+        path = send(path_to_versions_list, object, update: update_span)
+        opts = { class: html_class, title: t("inline_forms.view.list_versions") }
+        opts.merge!(turbo_row ? inline_forms_turbo_link_data(update_span) : { remote: true })
+        raw link_to("<i class='fi-list'></i>".html_safe, path, opts)
       end
     end
   end
 
   # close versions list link
-  def close_versions_list_link(object, update_span, html_class = 'button close_button' )
-    link_to "<i class='fi-x'></i>".html_safe,
-      send('list_versions_' + @object.class.to_s.underscore + "_path",
-          object,
-          :update => update_span,
-          :close => true
-      ),
-      :remote => true,
-      :class => html_class,
-      :title => t('inline_forms.view.close_versions_list')
+  def close_versions_list_link(object, update_span, html_class = "button close_button", turbo_row: true)
+    path = send(
+      "list_versions_#{object.class.to_s.underscore}_path",
+      object,
+      update: update_span,
+      close: true
+    )
+    opts = { class: html_class, title: t("inline_forms.view.close_versions_list") }
+    opts.merge!(turbo_row ? inline_forms_turbo_link_data(update_span) : { remote: true })
+    link_to "<i class='fi-x'></i>".html_safe, path, opts
+  end
+
+  # Renders the +*_show+ helper for +form_element+, passing +turbo_frame:+ when supported.
+  def inline_forms_field_show(object, attribute, form_element, turbo_frame: false)
+    show_method = "#{form_element}_show"
+    if turbo_frame
+      send(show_method, object, attribute, turbo_frame: true)
+    else
+      send(show_method, object, attribute)
+    end
+  rescue ArgumentError
+    send(show_method, object, attribute)
+  end
+
+  # Cancel control for single-field +_edit+ forms. Navigation attrs live on the outer
+  # +link_to+; the visible control is +input[type=button]+ so it matches the +ok+ submit
+  # height (Foundation sizes +a.button+ taller than +input.button+ in collapse rows).
+  def inline_forms_field_cancel_link(object, attribute, form_element, update_span, sub_id: nil, turbo_frame: false)
+    path = polymorphic_path(
+      object,
+      update: update_span,
+      attribute: attribute,
+      form_element: form_element,
+      sub_id: sub_id
+    )
+    opts = { class: "inline_forms-field-cancel" }
+    if turbo_frame
+      # Inside a <turbo-frame>: plain GET link; no data-method (jquery-ujs fights Turbo).
+      opts[:data] = { turbo: true, turbo_frame: "_self" }
+    else
+      opts[:remote] = true
+      opts[:method] = :get
+    end
+    link_to path, opts do
+      tag.input(
+        type: "button",
+        name: "cancel",
+        value: "cancel",
+        class: "postfix button alert",
+        tabindex: "-1"
+      )
+    end
   end
 
   # link_to_inline_edit
   #
   # Pass +from_callee:+ +__callee__+ from the enclosing +*_show+ method so the edit route receives the correct form element name.
-  def link_to_inline_edit(object, attribute, attribute_value='', from_callee:)
+  # When +turbo_frame:+ is true the link omits +remote: true+; navigation is handled by the enclosing +<turbo-frame>+.
+  def link_to_inline_edit(object, attribute, attribute_value='', from_callee:, turbo_frame: false)
     form_element = InlineForms.form_element_string_from_callee(from_callee)
     attribute_value = attribute_value.to_s
     spaces = attribute_value.length > 40 ? 0 : 40 - attribute_value.length
     value = h(attribute_value) + ("&nbsp;" * spaces).html_safe
     css_class_id = "#{object.class.to_s.underscore}_#{object.id}_#{attribute}"
+    use_turbo_frame = turbo_frame || (@inline_forms_turbo_field == true)
     if (cancan_disabled? rescue true) || ( can? :update, object, attribute )
       # some problem with concerns makes this function not available when called direct. FIXME
+      link_opts = if use_turbo_frame
+        { data: { turbo: true, turbo_frame: "_self" } }
+      else
+        { remote: true }
+      end
       link_to value,
         edit_polymorphic_path(
           object,
           :attribute => attribute.to_s,
           :form_element => form_element,
           :update => css_class_id ),
-        :remote => true
+        link_opts
     else
       h(attribute_value)
     end

data/app/views/devise/sessions/_form.html.erb

@@ -1,6 +1,9 @@
 <%= form_for(resource,  as:  resource_name,
                         url: session_path(resource_name),
-                        html: { id:  'inline_forms_devise_form' }
+                        html: {
+                          id: 'inline_forms_devise_form',
+                          data: { turbo: false }
+                        }
             ) do |f| %>
 
   <div class="row">

data/app/views/inline_forms/_close.html.erb

@@ -1,13 +1,15 @@
+<% row_turbo = @inline_forms_turbo_row %>
+<% presentation_link_opts = row_turbo ? { data: { turbo: true, turbo_frame: @update_span } } : { remote: true } %>
 <% if cancan_disabled? || ( can? :soft_delete, @object ) %>
   <div  class="small-1 column">
-    <%= link_to_soft_delete(@object, @update_span) -%>
-    <%= link_to_destroy(@object, @update_span) -%>
+    <%= link_to_soft_delete(@object, @update_span, turbo_row: row_turbo) -%>
+    <%= link_to_destroy(@object, @update_span, turbo_row: row_turbo) -%>
   </div>
   <div  class="small-11 column">
-    <%= link_to h(@object._presentation), polymorphic_path(@object, :update => @update_span), :remote => true -%>
+    <%= link_to h(@object._presentation), polymorphic_path(@object, :update => @update_span), presentation_link_opts -%>
   </div>
 <% else %>
   <div  class="small-12 column">
-    <%= link_to h(@object._presentation), polymorphic_path(@object, :update => @update_span), :remote => true -%>
+    <%= link_to h(@object._presentation), polymorphic_path(@object, :update => @update_span), presentation_link_opts -%>
   </div>
 <% end %>

data/app/views/inline_forms/_edit.html.erb

@@ -1,33 +1,14 @@
 <%  @BUTTONS_UNDER = [ "text_area", "kansen_slider", "rich_text" ] %>
+<%  edit_form_opts = { method: :put, multipart: true, class: "edit_form", abide: true } %>
+<%  edit_form_opts[:remote] = true unless @turbo_frame %>
 <%= form_tag polymorphic_path(@object,
     :update => @update_span,
     :attribute => @attribute,
     :form_element => @form_element,
     :sub_id => @sub_id ),
-    :method => :put, # this is going to the update method!
-    :multipart => true,
-    :class  => "edit_form",
-    :abide => true,
-    :remote => true do -%>
-  <div id="flash" class="row">
-      <% flash.each do |key, value| %>
-        <div class="flash <%= key %>">
-          <ul>
-            <% if value.is_a?(Array) %>
-              <% value.each do |msg| %>
-                <li>
-                  <%= msg %>
-                </li>
-              <% end %>
-            <% else %>
-              <li>
-                <%= value %>
-              </li>
-            <% end %>
-          </ul>
-        </div>
-      <% end %>
-    </div>
+    edit_form_opts do -%>
+  <%# Do not render `flash` here: this partial is injected into a field span via %>
+  <%# UJS; session flash (e.g. Devise :notice) would otherwise appear inside the editor. %>
 
   <% if @BUTTONS_UNDER.include? @form_element %>
     <div class="row collapse">
@@ -43,14 +24,7 @@
         <%= submit_tag "ok", :class => "postfix button"-%>
       </div>
       <div class="small-2 columns">
-        <%= link_to( polymorphic_path( @object, :update => @update_span || "field_#{@attribute}_#{@object.id.to_s}",
-        :attribute => @attribute,
-        :form_element => @form_element,
-        :sub_id => @sub_id ),
-        :method => :get, # this is going to the show method!
-        :remote => true ) do %>
-          <input type="button" name="cancel" value="cancel" class="button alert postfix radius" />
-        <% end %>
+        <%= inline_forms_field_cancel_link(@object, @attribute, @form_element, @update_span, sub_id: @sub_id, turbo_frame: @turbo_frame) %>
       </div>
     </div>
   <% else %>
@@ -62,14 +36,7 @@
         <%= submit_tag "ok", :class => "postfix button"-%>
       </div>
       <div class="small-2 columns">
-        <%= link_to( polymorphic_path( @object, :update => @update_span || "field_#{@attribute}_#{@object.id.to_s}",
-        :attribute => @attribute,
-        :form_element => @form_element,
-        :sub_id => @sub_id ),
-        :method => :get, # this is going to the show method!
-        :remote => true ) do %>
-          <input type="button" name="cancel" value="cancel" class="button alert postfix radius" />
-        <% end %>
+        <%= inline_forms_field_cancel_link(@object, @attribute, @form_element, @update_span, sub_id: @sub_id, turbo_frame: @turbo_frame) %>
       </div>
     </div>
   <% end%>

data/app/views/inline_forms/_list.html.erb

@@ -45,8 +45,11 @@
   Container element. For the nested has_many case (parent_class set) we
   emit a `<turbo-frame>` so pagination clicks (and any future in-frame
   re-render of this list) swap only the matching frame instead of doing
-  a full-page navigation. Top-level lists (parent_class.nil?) keep the
-  classic <div> + full-page pagination they already had.
+  a full-page navigation. Top-level lists keep the classic `<div>` and
+  full-page navigation; their `+ new` link is plain (no Turbo frame
+  target) so cancel / create just navigate -- a top-level `<turbo-frame>`
+  collapses inside `position: absolute` `#outer_container` and hides the
+  list (regression hit during the 7.5.2 spike).
 -%>
 <% if ul_needed -%>
   <% if parent_class.nil? -%>
@@ -65,48 +68,58 @@
     <% css_class_id = parent_class.to_s.underscore + '_' + parent_id.to_s + '_' + attribute.to_s.singularize.underscore + "_" + object.id.to_s -%>
     <% path_to_object = attribute.to_s.singularize.underscore + "_path" %>
   <% end %>
+  <% row_open_link_opts = { data: { turbo: true, turbo_frame: css_class_id } } %>
   <%#
-    For NESTED rows (parent_class present) the surrounding
-    <turbo-frame> would otherwise intercept the UJS-driven inline-edit
-    links/forms swapped in by show.js.erb / edit.js.erb / new.js.erb
-    -- those all do `$('#<row_id>').html(...)`, so the swapped content
-    stays a descendant of this row. Turbo would submit the multipart
-    `<form>` from _edit.html.erb with `Accept: text/html`, which
-    inline_forms_controller's update / create / destroy only respond
-    to via format.js, raising `ActionController::UnknownFormat` (HTTP
-    406) AFTER the DB write already happened. `data-turbo="false"`
-    here keeps UJS handling those, unchanged.
-
-    For TOP-LEVEL rows (parent_class.nil?) there is no surrounding
-    frame, so the attribute would be a no-op for the row's own click
-    -- BUT after UJS swaps in the inline edit (which itself contains
-    the nested-photos `<turbo-frame>`), Turbo's `elementIsNavigatable`
-    walks up from any link inside that inner frame looking for the
-    closest `[data-turbo]` ancestor; it does NOT stop at the
-    intervening `<turbo-frame>`. A `data-turbo="false"` on the
-    top-level row therefore poisons every descendant including the
-    pagination links inside the inner photos frame, breaking
-    in-frame navigation. So we only emit it on nested rows.
+    Each list row is `<turbo-frame id="…">` with a Turbo presentation link
+    (GET `show` / `close` as HTML). Nested rows use the same contract as
+    top-level; `not_accessible_through_html?` child models (Photo) still
+    serve `format.html` when `params[:update]` matches a nested row id.
   -%>
-  <div class="row <%= cycle('odd', 'even') %><%= " top-level" if parent_class.nil? %>" id="<%= css_class_id -%>"<%= raw ' data-turbo="false"' unless parent_class.nil? %>>
-    <% if cancan_disabled? || ( can? :soft_delete, object ) %>
-      <div  class="small-1 column">
-        <%= link_to_soft_delete(object, css_class_id) -%>
-        <%= link_to_destroy(object, css_class_id) -%>
-      </div>
-      <div  class="small-11 column">
-        <%= link_to h(object._presentation),
-          send( path_to_object, object, :update => css_class_id),
-          :remote => true -%>
+  <% if parent_class.nil? -%>
+    <%= tag.turbo_frame(id: css_class_id) do %>
+      <div class="row <%= cycle('odd', 'even') %> top-level">
+        <% if cancan_disabled? || ( can? :soft_delete, object ) %>
+          <div  class="small-1 column">
+            <%= link_to_soft_delete(object, css_class_id) -%>
+            <%= link_to_destroy(object, css_class_id) -%>
+          </div>
+          <div  class="small-11 column">
+            <%= link_to h(object._presentation),
+              send( path_to_object, object, :update => css_class_id),
+              row_open_link_opts -%>
+          </div>
+        <% else %>
+          <div  class="small-12 column">
+            <%= link_to h(object._presentation),
+              send( path_to_object, object, :update => css_class_id),
+              row_open_link_opts -%>
+          </div>
+        <% end %>
       </div>
-    <% else %>
-      <div  class="small-12 column">
-        <%= link_to h(object._presentation),
-          send( path_to_object, object, :update => css_class_id),
-          :remote => true -%>
+    <% end %>
+  <% else -%>
+    <%= tag.turbo_frame(id: css_class_id) do %>
+      <div class="row <%= cycle('odd', 'even') %>">
+        <% if cancan_disabled? || ( can? :soft_delete, object ) %>
+          <div  class="small-1 column">
+            <%= link_to_soft_delete(object, css_class_id) -%>
+            <%= link_to_destroy(object, css_class_id) -%>
+          </div>
+          <div  class="small-11 column">
+            <%= link_to h(object._presentation),
+              send( path_to_object, object, :update => css_class_id),
+              row_open_link_opts -%>
+          </div>
+        <% else %>
+          <div  class="small-12 column">
+            <%= link_to h(object._presentation),
+              send( path_to_object, object, :update => css_class_id),
+              row_open_link_opts -%>
+          </div>
+        <% end %>
       </div>
     <% end %>
-  </div>
+  <% end -%>
 <% end -%>
 <!--  # pagination -->
 <% if parent_id.nil? -%>