inline_forms 7.5.2 → 7.9.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bbf211e9092af732b645bf10e9f9c338ced10c55e3ebf0a533146537669d1a4d
-  data.tar.gz: 90bc8cf55083ca23bbc1d3122a49e835620bf2e4800c8c83a93b871230d54718
+  metadata.gz: 9cce3cdf65d2a40a3481fba51c95e57bfa6a65cd4a73359ad974fc327252dd7a
+  data.tar.gz: 7f57a6489ef1d88328474330fe70492f1266e5e394163260b2cc773c414571b8
 SHA512:
-  metadata.gz: 5f66995db3089c59e1654f92f1a7c78b9701ccad3ea0a12fb4cf7729ea5ed1bff70003847af80ebcae6347c05e29f7e6fc9e8a9317252ea120a7a09d790c2284
-  data.tar.gz: 2e04d9869aff6e13a1eb6556e9865e640725dee6d9ad5a56887ed23f125bd9378ab679c773fd84cbc72b8df408335dd7026998e113254366e4cc358155c70785
+  metadata.gz: fb6275f4b67706bee0e3b263e3b54b5a5a8be8a7f20676fd0ea0504be0e98edc3e3f41c98a807f1f28cdc4f5367e9a0f9a8c895bfa052978483f09148e1904e7
+  data.tar.gz: f168431ca3b73f07cfd2190b21ba211be91ab529861751d588ff33ed2e832e985411ec414ae2c5f958a9dd867e84083b5841e79424d2662bc558f35c10caac15

data/CHANGELOG.md

@@ -4,6 +4,116 @@ All notable changes to this project are documented in this file.
 
 ## [Unreleased]
 
+## [7.9.1] - 2026-05-17
+
+### Added
+
+- **Example app — apartment name required:** the `--example` installer injects `validates :name, presence: true` on `Apartment`. Top-level create without a name re-renders the new form instead of persisting.
+- **Regression tests:** `example_app_apartment_name_validation_test.rb` (model); `example_app_apartment_name_required_test.rb` (Turbo create).
+
+### Verified
+
+- **`bundle exec rails test`** (new validation tests) in `--example` MyApp — **3 runs, 10 assertions, 0 failures**.
+
+## [7.9.0] - 2026-05-16
+
+### Added
+
+- **Restore rich_text (ActionText) versions from the versions panel:** the Restore link now renders for `:rich_text` entries too. `InlineFormsController#revert` reifies the `ActionText::RichText` row, saves it, and `touch`es the parent so any timestamp display refreshes.
+- **CarrierWave history for `image_field` / `multi_image_field`:** the installer ships a `config/initializers/carrierwave.rb` with `remove_previously_stored_files_after_update = false` and patches the generated `app/uploaders/image_uploader.rb` with a no-op `remove!` plus a per-upload UUID `filename` prefix. PaperTrail reverts on an image column now restore the previous bytes (not just the previous filename). Source: <https://stackoverflow.com/questions/9423279/papertrail-and-carrierwave> (Answers 2, 4, 5).
+- **Regression tests:** `example_app_photo_revert_test.rb` (image-column revert restores bytes); rich_text revert assertions in `example_app_apartment_versions_turbo_test.rb`.
+
+### Changed
+
+- **`render_revert_turbo_streams`:** no longer mutates `@update_span` mid-method. The row and versions frames are rendered with explicit `locals: { update_span:, object:, inline_forms_turbo_row: }`. The `row_close` / `versions_panel` templates and their `_close` / `_versions` partials prefer `local_assigns[:…]` and fall back to the matching ivar.
+- **Frame ids in revert:** `row_id` and `versions_id` now derive from `@parent` (the rich_text branch sets `@parent = @rich_text_record.record`), so revert works identically for primary and rich_text versions.
+
+### Removed
+
+- **`format.html` fallback in `revert`:** the action only responds with `turbo_stream` now. The "Turbo POST on row frame" regression test was updated to send `Accept: text/vnd.turbo-stream.html`.
+
+### Trade-offs
+
+- The `ImageUploader` no longer deletes previous files on update or destroy. Files accumulate; sweep tooling is out of scope (would have to reconcile with `PaperTrail::Version#object` snapshots across the whole table).
+
+### Verified
+
+- **`bundle exec rails test`** in `--example` MyApp — **68 runs, 358 assertions, 0 failures, 0 errors, 0 skips**.
+
+## [7.8.1] - 2026-05-16
+
+### Fixed
+
+- **Restore from versions panel (Turbo 2 + Drive):** POSTs from inside `<turbo-frame id="…_versions">` sent `Turbo-Frame: …_versions` while the server returned only the **row** frame, so Turbo showed **Content missing**. Restore links now request **`turbo_stream`**; **`revert`** responds with **`turbo_stream.replace`** for both the row and the versions panel.
+
+### Added
+
+- **Regression tests:** versions list restore link carries `data-turbo-stream`; revert with versions-frame header returns stream (`example_app_apartment_versions_turbo_test.rb`).
+
+### Verified
+
+- **`bundle exec rails test`** in `--example` MyApp — **67 runs, 342 assertions, 0 failures**.
+
+## [7.8.0] - 2026-05-16
+
+### Changed
+
+- **Step 5 (UJS → Turbo):** **Turbo Drive** left at the default (**enabled**); removed **`Turbo.session.drive = false`** from **`inline_forms`** and **`application`** layouts.
+- **`app/assets/javascripts/inline_forms/inline_forms.js`:** dropped **`jquery_ujs`** and **`jquery.remotipart`**; inline navigation uses Turbo only.
+- **Installer Gemfile** (`bin/inline_forms_installer_core.rb`): removed **`remotipart`**; updated **`turbo-rails`** comment.
+- **Helpers / partials:** removed **`remote: true`** fallbacks (`close_link`, toolbar, new/versions links, field cancel, `link_to_inline_edit` legacy branch, **`_close`**, **`_new`**); all use **`data-turbo`** + frame targets.
+
+### Verified
+
+- **`bundle exec rails test`** in `--example` MyApp — **65 runs, 330 assertions, 0 failures**.
+
+## [7.7.3] - 2026-05-16
+
+### Changed
+
+- **Step 4 (UJS → Turbo) — versions panel:** `VersionsConcern#list_versions` is **HTML-only** (`versions_panel` / `versions_list_panel` inside matching `<turbo-frame>`). Removed **`versions.js.erb`** and **`versions_list.js.erb`**.
+
+### Fixed
+
+- **Nested Photo (and other `not_accessible_through_html?` children) versions restore:** `_versions_list` revert links targeted `photo_<id>` but nested list rows use `apartment_<aid>_photo_<id>`. Turbo could not find that frame, fell back to `photo_<id>_versions`, and `revert` returned **406** (`row_html_turbo_allowed?` false for bare `photo_<id>`). **`inline_forms_row_turbo_frame_id`** now matches `_list.html.erb` row ids.
+
+### Added
+
+- **Regression tests:** expanded-row versions open link (Turbo, not `data-remote`); revert from versions list via Turbo POST (`example_app_apartment_versions_turbo_test.rb`); nested Photo versions list restore + Turbo POST revert (`example_app_apartment_photos_pagination_test.rb`).
+
+### Verified
+
+- **`bundle exec rails test`** in `--example` MyApp — **65 runs, 330 assertions, 0 failures**.
+
+## [7.7.0] - 2026-05-16
+
+### Changed
+
+- **Step 4 (UJS → Turbo) — top-level list + create:** `/apartments` index list root is **`<turbo-frame id="apartments_list">`** with in-frame pagination (`update=apartments_list`); **`+ new` / cancel / create** use Turbo HTML (`new_record` / `create_list_frame`) instead of **`new.js.erb`** / **`list.js.erb`**. **`#outer_container > turbo-frame.list_container { width: 100% }`** fixes the 7.5.2 layout collapse.
+- **`:tree` / `:move` archived:** hierarchical child list (**`_tree.html.erb`**) and reparent helper (**`move.rb`**) moved to **`archived/form_elements/tree/`** — they expect host-app tree APIs (`#children`, `#hash_tree_to_collection`, `#add_child`), not defined in the gem; no example-app model. Registry blocks `:tree` / `:move` in `inline_forms_attribute_list`.
+- **Removed row/list UJS templates:** **`show.js.erb`**, **`close.js.erb`**, **`record_destroyed.js.erb`**, **`show_undo.js.erb`**, **`new.js.erb`**, **`list.js.erb`**; row actions and list flows use **`format.html`** only.
+
+### Added
+
+- **Regression tests:** `example_app_apartment_top_level_new_test.rb` (Turbo new/cancel/create), `example_app_apartment_top_level_pagination_test.rb`.
+
+### Verified
+
+- **`bundle exec rails test`** in `--example` MyApp — **62 runs, 312 assertions, 0 failures**.
+
+## [7.6.0] - 2026-05-16
+
+### Changed
+
+- **Project-specific form elements archived (not loaded by default):** `:geo_code_curacao` (helper + model + controller + views), **`chicas_*`** (`:chicas_photo_list`, `:chicas_family_photo_list`, `:chicas_dropdown_with_family_members`), and **`:kansen_slider`**. Sources live under **`archived/form_elements/`** with per-folder README restore steps.
+- **Registry:** **`InlineForms::ARCHIVED_FORM_ELEMENTS`**; boot raises **`ArchivedFormElementError`** if a model still declares an archived symbol (including **`:absence_list`**, removed in 6.3.0 without source in tree).
+- **Versioned archive:** **`archived/README.md`** catalogs all retired features.
+- **`_edit.html.erb`:** removed **`kansen_slider`** from `@BUTTONS_UNDER` (element archived).
+
+### Upgrade notes
+
+- **Breaking:** Remove or vendor archived symbols from `inline_forms_attribute_list`: **`:geo_code_curacao`**, **`:chicas_*`**, **`:kansen_slider`**. Copy from the matching **`archived/form_elements/<name>/`** README (geo also needs host route `get "geo_code_curacao", to: "geo_code_curacao#list_streets"`).
+
 ## [7.5.2] - 2026-05-16
 
 ### Fixed

data/README.rdoc

@@ -48,6 +48,16 @@ The +:image_field+ form element uses CarrierWave. Generated apps depend on +carr
 
 To switch to S3, add +carrierwave-aws+ (or use the bundled fog backend) and configure a +CarrierWave.configure+ block in +config/initializers/carrierwave.rb+; nothing in inline_forms hard-codes local storage.
 
+=== PaperTrail-driven restore keeps previous image bytes
+
+PaperTrail snapshots the column scalar (a CarrierWave filename) on update; CarrierWave's defaults overwrite the previous file on disk and reuse the same filename, so a vanilla +version.reify; save!+ ends up restoring a filename whose bytes are gone. The generated +ImageUploader+ ships three knobs that fix this:
+
+* +CarrierWave.configure { |c| c.remove_previously_stored_files_after_update = false }+ in +config/initializers/carrierwave.rb+ — covers +:multi_image_field+ uploaders too.
+* +remove!+ overridden to a no-op, so hard-destroyed records keep their bytes and revert-after-destroy can still find them.
+* +filename+ prefixed with a per-upload UUID, so successive uploads never collide on disk.
+
+Trade-off: files accumulate on disk; periodic sweeping is out of scope of the gem. Source: https://stackoverflow.com/questions/9423279/papertrail-and-carrierwave (Answers 2, 4 and 5).
+
 For long text fields, use +:plain_text+ for a plain textarea backed by a DB +text+ column, or +:rich_text+ for ActionText/Trix content. +:plain_text+ requires an actual column on the model table; if the column is missing, inline_forms now raises +InlineForms::PlainTextColumnMissingError+ during controller boot/runtime checks.
 
 Note: generated apps also depend on ActiveStorage transitively because the +:rich_text+ form element uses ActionText (+active_storage:install+ runs during +inline_forms create+). Image uploads still go through CarrierWave; ActiveStorage is only there to back ActionText embeds.

data/app/assets/javascripts/inline_forms/inline_forms.js

@@ -1,24 +1,17 @@
 //= require jquery
-//= require jquery_ujs
 //= require jquery.ui.all
 //= require jquery.timepicker.js
 //= require foundation
-//= require jquery.remotipart
 //= require autocomplete-rails
 
 // Turbo / Hotwire is intentionally NOT required into this Sprockets bundle.
 // `turbo-rails` ships only an ES-module build of turbo (`app/assets/javascripts/turbo.js`
 // ends with `export { Turbo, cable }`). Concatenating that into a regular
-// `<script>` bundle produces a parse error at the top-level `export`, which
-// silently kills jquery-ujs initialization (forms then submit as plain
-// HTML POST and inline_forms controllers respond with `format.js` only,
-// raising `ActionController::UnknownFormat`).
+// `<script>` bundle produces a parse error at the top-level `export`.
 //
 // Turbo is loaded separately by `app/views/layouts/inline_forms.html.erb`
-// (and `application.html.erb`) as `<script type="module">`, with
-// `Turbo.session.drive = false` so existing UJS-driven links/forms keep
-// working unchanged. `<turbo-frame>` and `format.turbo_stream` are
-// available for the per-view conversions that follow.
+// (and `application.html.erb`) as `<script type="module">`. Inline flows use
+// `<turbo-frame>` + HTML responses; jquery-ujs / remotipart were removed in 7.8.0.
 
 $(function(){ $(document).foundation(); });
 // initialize datepickers

data/app/assets/stylesheets/inline_forms/inline_forms.scss

@@ -224,13 +224,14 @@ select:hover, select:focus {
   font-size: 110%;
 }
 // Custom elements default to `display: inline`, which collapses the row layout
-// of an empty/just-mounted top-level `<turbo-frame id="apartments_list">` and
-// hides its rows under the fixed top bar. Force block layout so frames behave
-// like the legacy `<div class="list_container">` and `<div class="row">` they
-// replaced (see app/views/inline_forms/_list.html.erb).
+// of a top-level `<turbo-frame id="apartments_list">` inside `#outer_container`.
 turbo-frame {
   display: block;
 }
+#outer_container > turbo-frame.list_container {
+  display: block;
+  width: 100%;
+}
 .list_container {
   .row {
     font-size: 1.2rem;

data/app/controllers/concerns/versions_concern.rb

@@ -12,12 +12,10 @@ module VersionsConcern
     if close
       respond_to do |format|
         format.html { render "inline_forms/versions_panel", layout: "turbo_rails/frame" }
-        format.js { render :versions }
       end
     else
       respond_to do |format|
         format.html { render "inline_forms/versions_list_panel", layout: "turbo_rails/frame" }
-        format.js { render :versions_list }
       end
     end
   end

data/app/controllers/inline_forms_controller.rb

@@ -44,7 +44,12 @@ class InlineFormsController < ApplicationController
       foreign_key = @Klass.reflect_on_association(@parent_class.underscore.to_sym).options[:foreign_key] || @parent_class.foreign_key
       conditions =  [ "#{foreign_key} = ?", @parent_id ]
     end
-    # if we are using cancan, then make sure to select only accessible records
+    # CanCan's load_and_authorize_resource sets @apartments (etc.); keep @objects in sync.
+    collection_ivar = :"@#{controller_name}"
+    if instance_variable_defined?(collection_ivar)
+      loaded = instance_variable_get(collection_ivar)
+      @objects = loaded unless loaded.nil?
+    end
     @objects ||= @Klass.accessible_by(current_ability) if cancan_enabled?
     @objects ||= @Klass
     @objects = @objects.order(@Klass.table_name + "." + @Klass.order_by_clause) if @Klass.respond_to?(:order_by_clause) && ! @Klass.order_by_clause.nil?
@@ -79,12 +84,14 @@ class InlineFormsController < ApplicationController
         format.html do
           if @parent_class.present?
             render_nested_associated_list_html
+          elsif turbo_frame_request? && list_frame_id?(params[:update])
+            @ul_needed = true
+            render "inline_forms/_list", layout: "turbo_rails/frame"
           else
             render "inline_forms/_list", layout: "inline_forms"
           end
         end
       end
-      format.js { render :list }
     end
   end
 
@@ -103,8 +110,7 @@ class InlineFormsController < ApplicationController
 
     @object.inline_forms_attribute_list = @inline_forms_attribute_list if @inline_forms_attribute_list
     respond_to do |format|
-      format.html { render_turbo_new } if associated_list_html_allowed? || !@Klass.not_accessible_through_html?
-      format.js { }
+      format.html { render_turbo_new } if html_list_flow_allowed?
     end
   end
 
@@ -128,7 +134,7 @@ class InlineFormsController < ApplicationController
     attributes = @inline_forms_attribute_list || @object.inline_forms_attribute_list
     attributes.each do | attribute, name, form_element |
       InlineForms.assert_plain_text_column!(object: @object, attribute: attribute, form_element: form_element)
-      send("#{form_element.to_s}_update", @object, attribute) unless form_element == :tree || form_element == :associated || (cancan_enabled? && cannot?(:read, @object, attribute))
+      send("#{form_element.to_s}_update", @object, attribute) unless form_element == :associated || (cancan_enabled? && cannot?(:read, @object, attribute))
     end
     @parent_class = params[:parent_class]
     @parent_id = params[:parent_id]
@@ -150,16 +156,14 @@ class InlineFormsController < ApplicationController
       @objects = @objects.where(conditions).paginate(:page => params[:page])
       @object = nil
       respond_to do |format|
-        format.html { render_associated_list_frame } if associated_list_html_allowed?
-        format.js { render :list }
+        format.html { render_list_frame_after_save } if html_list_flow_allowed?
       end
     else
       flash.now[:header] = ["Kan #{@object.class.to_s.underscore} niet aanmaken."]
       flash.now[:error] = @object.errors.to_a
       respond_to do |format|
         @object.inline_forms_attribute_list = attributes
-        format.html { render_turbo_new } if associated_list_html_allowed? || !@Klass.not_accessible_through_html?
-        format.js { render :new }
+        format.html { render_turbo_new } if html_list_flow_allowed?
       end
     end
   end
@@ -203,10 +207,8 @@ class InlineFormsController < ApplicationController
         @attributes = @object.inline_forms_attribute_list
         if close
           format.html { render_row_turbo(:close) } if row_html_turbo_allowed?
-          format.js { render :close }
         else
           format.html { render_row_turbo(:show) } if row_html_turbo_allowed?
-          format.js { render :show }
         end
       end
     else
@@ -223,7 +225,6 @@ class InlineFormsController < ApplicationController
     @object.soft_delete(current_user)
     respond_to do |format|
       format.html { render_row_turbo(:close) } if row_html_turbo_allowed?
-      format.js { render :close }
     end
   end
 
@@ -234,7 +235,6 @@ class InlineFormsController < ApplicationController
     @object.soft_restore
     respond_to do |format|
       format.html { render_row_turbo(:close) } if row_html_turbo_allowed?
-      format.js { render :close }
     end
   end
 
@@ -247,23 +247,42 @@ class InlineFormsController < ApplicationController
       @object.destroy
       respond_to do |format|
         format.html { render_row_turbo_destroyed } if row_html_turbo_allowed?
-        format.js { render :record_destroyed }
       end
     end
   end
 
   # :revert works like undo.
   # Thanks Ryan Bates: http://railscasts.com/episodes/255-undo-with-paper-trail
+  #
+  # Two reify paths:
+  #
+  # * Primary version: the reified object IS the row (Apartment, Photo, ...).
+  #   Save it and we're done; CarrierWave keeps the previous file on disk
+  #   (see app/uploaders/image_uploader.rb) so a restored `image` column
+  #   still points at real bytes.
+  # * ActionText (rich_text) version: the reified object is the
+  #   `ActionText::RichText` row hanging off the parent. Save the rich text,
+  #   then `touch` the parent so any timestamp display refreshes. Frame ids
+  #   below derive from `@parent` for both branches.
   def revert
     @update_span = params[:update]
     if current_user.role? :superadmin
       @version = PaperTrail::Version.find(params[:id])
       @object = @version.reify
-      @object.save!
-      authorize!(:revert, @object) if cancan_enabled?
+      if defined?(ActionText::RichText) && @object.is_a?(ActionText::RichText)
+        @rich_text_record = @object
+        @parent = @rich_text_record.record
+        @rich_text_record.save!
+        @parent.touch if @parent.respond_to?(:touch)
+      else
+        @parent = @object
+        @parent.save!
+      end
+      authorize!(:revert, @parent) if cancan_enabled?
+      return unless row_html_turbo_allowed?
+
       respond_to do |format|
-        format.html { render_row_turbo(:close) } if row_html_turbo_allowed?
-        format.js { render :close }
+        format.turbo_stream { render_revert_turbo_streams }
       end
     end
   end
@@ -282,6 +301,30 @@ class InlineFormsController < ApplicationController
 
   private
 
+  # Versions list lives in +<turbo-frame id="…_versions">+; POST +restore+ would otherwise
+  # send +Turbo-Frame: …_versions+ while +row_close+ only returns the row frame. Stream
+  # replaces both the row and the versions panel in one response.
+  def render_revert_turbo_streams
+    row_id = helpers.inline_forms_row_turbo_frame_id(@parent)
+    versions_id = "#{@parent.class.name.underscore}_#{@parent.id}_versions"
+    row_html = render_to_string(
+      "inline_forms/row_close",
+      layout: false,
+      formats: [:html],
+      locals: { update_span: row_id, object: @parent, inline_forms_turbo_row: true }
+    )
+    versions_html = render_to_string(
+      "inline_forms/versions_panel",
+      layout: false,
+      formats: [:html],
+      locals: { update_span: versions_id, object: @parent, inline_forms_turbo_row: true }
+    )
+    render turbo_stream: [
+      turbo_stream.replace(row_id, row_html),
+      turbo_stream.replace(versions_id, versions_html)
+    ]
+  end
+
   # HTML field edit/show inside a +<turbo-frame>+ (Step 3). Scalar fields no longer
   # use UJS; +format.html+ is always registered for edit/update/single-attribute show.
   #
@@ -289,9 +332,8 @@ class InlineFormsController < ApplicationController
   # +*_show+ helpers it wraps) to emit Turbo data attributes. The flag is set in
   # +_show.html.erb+ when a row first opens, but bare +field_show+ / +field_edit+
   # responses (on +cancel+ / +update+) do not re-render +_show+. Without setting
-  # it here the link in the swapped frame falls back to +remote: true+, which the
-  # legacy +jquery_ujs+ bundle intercepts as a JS request -- the controller only
-  # registers +format.html+, so the click silently fails (no swap, no edit form).
+  # it here the link in the swapped frame would not target the field frame and
+  # inline edit would not open reliably.
   def render_turbo_field(template, turbo_field_show: false)
     @turbo_frame = true if template == :field_edit
     @turbo_field_show_turbo_frame = turbo_field_show
@@ -315,10 +357,23 @@ class InlineFormsController < ApplicationController
   end
 
   # Nested has_many +new+ / cancel / +create+ inside a parent +<turbo-frame>+ (e.g. Apartment → Photo).
+  def html_list_flow_allowed?
+    params[:update].present? && (@parent_class.present? || !@Klass.not_accessible_through_html?)
+  end
+
   def associated_list_html_allowed?
     @parent_class.present? && params[:update].present?
   end
 
+  def list_frame_id?(update)
+    update.present? && update.to_s.end_with?("_list")
+  end
+
+  def render_list_frame_after_save
+    @ul_needed = true
+    render "inline_forms/create_list_frame", layout: associated_list_frame_layout
+  end
+
   def associated_list_frame_layout
     # Use full inline_forms chrome so the swapped frame is styled; Turbo extracts
     # the matching <turbo-frame id="…"> from the response body.

data/app/helpers/inline_forms_helper.rb

@@ -1,8 +1,7 @@
 # -*- encoding : utf-8 -*-
 module InlineFormsHelper
-  # load form elements. Each element goes into a separate file
-  # and defines a _show, _edit and _update method.
-  #
+  # Load active form elements (top-level *.rb only). Retired elements live under
+  # archived/form_elements/ — see archived/README.md.
   INLINE_FORMS_PATH = File.dirname(__FILE__) + "/form_elements/"
   Dir[INLINE_FORMS_PATH + "*.rb"].each do |form_element|
     require form_element
@@ -54,12 +53,36 @@ module InlineFormsHelper
 
   # 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)
+  #
+  # When +turbo_stream: true+ (e.g. restore from inside +*_versions+ frame), the client
+  # requests +text/vnd.turbo-stream.html+ so Turbo does not expect a single matching
+  # frame in the response — POSTs from nested +…_versions+ frames otherwise send
+  # +Turbo-Frame: …_versions+ while the server returns the row frame (+Content missing+).
+  def inline_forms_turbo_link_data(update_span, method: :get, turbo_stream: false)
     data = { turbo: true, turbo_frame: update_span }
     data[:turbo_method] = method.to_s.downcase unless method == :get
+    data[:turbo_stream] = true if turbo_stream
     { data: data }
   end
 
+  # +<turbo-frame>+ id for a list row's open/close/revert target.
+  # Top-level models: +apartment_5+. Nested +not_accessible_through_html?+ children
+  # (e.g. Photo under Apartment): +apartment_5_photo_2+ — must match +_list.html.erb+.
+  def inline_forms_row_turbo_frame_id(object)
+    frame_id = "#{object.class.name.underscore}_#{object.id}"
+    return frame_id unless object.class.respond_to?(:not_accessible_through_html?) &&
+      object.class.not_accessible_through_html?
+
+    belongs_to = object.class.reflect_on_all_associations(:belongs_to)
+      .find { |assoc| !assoc.polymorphic? }
+    return frame_id unless belongs_to
+
+    parent = object.public_send(belongs_to.name)
+    return frame_id unless parent
+
+    "#{parent.class.name.underscore}_#{parent.id}_#{object.class.name.underscore}_#{object.id}"
+  end
+
   private
 
   def validation_hints_as_list_for(object, attribute)
@@ -74,12 +97,8 @@ module InlineFormsHelper
       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
+    opts[:data] = { turbo: true, turbo_frame: "_self" }
+    link_to "<i class='fi-x'></i>".html_safe, path, opts
   end
 
   # delete link. Mind the difference between delete and destroy.
@@ -89,12 +108,12 @@ module InlineFormsHelper
       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 })
+        opts.merge!(inline_forms_turbo_link_data(update_span, method: :post))
         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 })
+        opts.merge!(inline_forms_turbo_link_data(update_span, method: :post))
         soft = link_to "<i class='fi-trash'></i>".html_safe, path, opts
       end
     end
@@ -107,7 +126,7 @@ module InlineFormsHelper
     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 })
+      opts.merge!(inline_forms_turbo_link_data(update_span, method: :delete))
       hard = link_to "&nbsp;&nbsp;<font color='FF0000'><i class='fi-x'></i></font>".html_safe, path, opts
     end
     hard.html_safe
@@ -115,13 +134,8 @@ module InlineFormsHelper
 
   # new link
   #
-  # +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).
+  # +turbo_row:+ kept for API compatibility; navigation always targets +update_span+
+  # as a +<turbo-frame>+ (no jquery-ujs).
   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,
@@ -133,7 +147,7 @@ module InlineFormsHelper
       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 })
+    opts.merge!(inline_forms_turbo_link_data(update_span))
     out = link_to "<i class='fi-plus'></i>".html_safe, path, opts
     if cancan_enabled?
       if can? :create, model
@@ -154,7 +168,7 @@ module InlineFormsHelper
       if defined?(PaperTrail) && object.respond_to?(:versions)
         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 })
+        opts.merge!(inline_forms_turbo_link_data(update_span))
         raw link_to("<i class='fi-list'></i>".html_safe, path, opts)
       end
     end
@@ -169,7 +183,7 @@ module InlineFormsHelper
       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 })
+    opts.merge!(inline_forms_turbo_link_data(update_span))
     link_to "<i class='fi-x'></i>".html_safe, path, opts
   end
 
@@ -198,11 +212,10 @@ module InlineFormsHelper
     )
     opts = { class: "inline_forms-field-cancel" }
     if turbo_frame
-      # Inside a <turbo-frame>: plain GET link; no data-method (jquery-ujs fights Turbo).
+      # Inside a <turbo-frame>: plain GET link; no data-method (legacy ujs fought Turbo).
       opts[:data] = { turbo: true, turbo_frame: "_self" }
     else
-      opts[:remote] = true
-      opts[:method] = :get
+      opts[:data] = { turbo: true, turbo_frame: update_span }
     end
     link_to path, opts do
       tag.input(
@@ -218,7 +231,8 @@ module InlineFormsHelper
   # link_to_inline_edit
   #
   # Pass +from_callee:+ +__callee__+ from the enclosing +*_show+ method so the edit route receives the correct form element name.
-  # When +turbo_frame:+ is true the link omits +remote: true+; navigation is handled by the enclosing +<turbo-frame>+.
+  # When +turbo_frame:+ is true the link targets +_self+; otherwise it targets the
+  # field frame id (+css_class_id+) so edit works without a surrounding +_show+ wrap.
   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
@@ -231,7 +245,7 @@ module InlineFormsHelper
       link_opts = if use_turbo_frame
         { data: { turbo: true, turbo_frame: "_self" } }
       else
-        { remote: true }
+        { data: { turbo: true, turbo_frame: css_class_id } }
       end
       link_to value,
         edit_polymorphic_path(

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

@@ -1,15 +1,17 @@
-<% 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 ) %>
+<% update_span = local_assigns[:update_span] || @update_span -%>
+<% object = local_assigns[:object] || @object -%>
+<% inline_forms_turbo_row = local_assigns.fetch(:inline_forms_turbo_row, @inline_forms_turbo_row) -%>
+<% presentation_link_opts = { data: { turbo: true, turbo_frame: update_span } } %>
+<% if cancan_disabled? || ( can? :soft_delete, object ) %>
   <div  class="small-1 column">
-    <%= link_to_soft_delete(@object, @update_span, turbo_row: row_turbo) -%>
-    <%= link_to_destroy(@object, @update_span, turbo_row: row_turbo) -%>
+    <%= link_to_soft_delete(object, update_span, turbo_row: inline_forms_turbo_row) -%>
+    <%= link_to_destroy(object, update_span, turbo_row: inline_forms_turbo_row) -%>
   </div>
   <div  class="small-11 column">
-    <%= link_to h(@object._presentation), polymorphic_path(@object, :update => @update_span), presentation_link_opts -%>
+    <%= 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), presentation_link_opts -%>
+    <%= 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,4 +1,4 @@
-<%  @BUTTONS_UNDER = [ "text_area", "kansen_slider", "rich_text" ] %>
+<%  @BUTTONS_UNDER = [ "text_area", "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,