inline_forms 7.2.11 → 7.5.2

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

@@ -1,6 +1,10 @@
 <div class="new_record"">
  <div id="flash" class="row">
-    <% flash.each do |key, value| %>
+    <%# Only validation feedback from InlineFormsController#create (flash.now); %>
+    <%# never dump session flash (:notice / :alert from Devise, etc.). %>
+    <% %w[header error success].each do |key| %>
+      <% value = flash[key] %>
+      <% next if value.blank? %>
       <div class="flash <%= key %>">
         <ul>
           <% if value.is_a?(Array) %>
@@ -25,10 +29,17 @@
     </div>
   </div>
 
-  <%= form_tag  polymorphic_path(@Klass,  :update => @update_span,
-                                                                  :parent_class => @parent_class,
-                                                                  :parent_id => @parent_id ),
-                                                                  :multipart => true, :remote => true, :class  => "edit_form" do -%>
+  <% new_form_opts = { multipart: true, class: "edit_form" } %>
+  <% if @turbo_frame %>
+    <% new_form_opts[:data] = { turbo: true, turbo_frame: @update_span } %>
+  <% else %>
+    <% new_form_opts[:remote] = true %>
+  <% end %>
+  <%= form_tag polymorphic_path(@Klass,
+        update: @update_span,
+        parent_class: @parent_class,
+        parent_id: @parent_id),
+      new_form_opts do -%>
     <% attributes = @inline_forms_attribute_list || @object.inline_forms_attribute_list -%>
     <% attributes.each do | attribute, name, form_element | -%>
       <% unless form_element.to_sym == :associated || form_element.to_sym == :tree || (cancan_enabled? && cannot?(:read, @object, attribute)) -%>
@@ -56,12 +67,13 @@
                   &nbsp;
                 </div>
     <div class='small-11  column' >
-        <%= link_to( polymorphic_path(@Klass, :update => @update_span,
-                                                                      :parent_class => @parent_class,
-                                                                      :parent_id => @parent_id,
-                                                                      :ul_needed => true ),
-                                                                      :remote => true,
-                   ) do -%>
+        <% cancel_path = polymorphic_path(@Klass,
+              update: @update_span,
+              parent_class: @parent_class,
+              parent_id: @parent_id,
+              ul_needed: true) %>
+        <% cancel_opts = @turbo_frame ? inline_forms_turbo_link_data(@update_span) : { remote: true } %>
+        <%= link_to cancel_path, cancel_opts do -%>
           <input type="button" name="cancel" value="cancel" class="button alert" />
         <% end %>
         <%= submit_tag "ok", :class => "button "-%>

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

@@ -1,3 +1,5 @@
+<%# Scalar field inline edit uses <turbo-frame> + HTML responses (Step 3). %>
+<% @inline_forms_turbo_field = true %>
 <div class="row">
   <% if (INLINE_FORMS_SHOW_INDENT rescue true) %>
     <div class='medium-1 large-1 column'>
@@ -13,7 +15,7 @@
         <%= h(@object._presentation) -%>
       </div>
       <div class="small-1 column close_link" >
-        <%= close_link(@object, @update_span) -%>
+        <%= close_link(@object, @update_span, turbo_row: @inline_forms_turbo_row) -%>
       </div>
     </div>
     <% end %>
@@ -67,16 +69,16 @@
                   <div class='medium-1 large-1 column'>
                     &nbsp;
                   </div>
-                  <div id="<%= css_class_id -%>" class='small-11 column'>
+                  <turbo-frame id="<%= css_class_id -%>" class="small-11 column">
                 <% else %>
-      	          <div id="<%= css_class_id -%>" class='small-12 column' >
+      	          <turbo-frame id="<%= css_class_id -%>" class="small-12 column">
                 <% end %>
                   <%= render  :partial => "inline_forms/list",
       	            :locals => {  :parent_class => @object.class,
       		                  :parent_id    => @object.id,
                                   :attribute    => attribute,
                                   :form_element => form_element } %>
-      	          </div>
+      	          </turbo-frame>
       	      </div>
             <% else %>
               <% if form_element == :has_one %>
@@ -84,13 +86,13 @@
                   <div class='medium-1 large-1 column'>
                     &nbsp;
                   </div>
-                  <div id="<%= css_class_id -%>" class='small-11 column' >
+                  <turbo-frame id="<%= css_class_id -%>" class="small-11 column">
                     <%= render  :partial => "inline_forms/list",
                       :locals => {  :parent_class => @object.class,
                                     :parent_id    => @object.id,
                                     :attribute    => attribute,
                                     :form_element => form_element } %>
-                  </div>
+                  </turbo-frame>
                 </div>
               <% else %>
                 <div class="row <%= cycle('odd', 'even') %>">
@@ -104,9 +106,9 @@
                     <% end %>
                   </div>
                   <div class='medium-7 large-7 column' >
-                    <span id="<%= css_class_id -%>" >
+                    <turbo-frame id="<%= css_class_id -%>">
                       <%= send("#{form_element}_show", @object, attribute) -%>
-                    </span>
+                    </turbo-frame>
                   </div>
                 </div>
               <% end %>
@@ -117,9 +119,9 @@
     <% end %>
     <% if can? :list_versions, @object  %>
       <% css_class_id = "#{@object.class.name.underscore}_#{@object.id}_versions" -%>
-      <div id="<%= css_class_id -%>">
-        <%= render 'versions' %>
-      </div>
+      <turbo-frame id="<%= css_class_id -%>">
+        <%= render "versions" %>
+      </turbo-frame>
     <% end %>
     <div class="record_footer"></div>
   </div>

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

@@ -46,14 +46,10 @@
   <div class="row <%= cycle('odd', 'even') %>">
     <div class="small-1 column">
       <% if entry[:kind] == :primary %>
-        <%= link_to t('inline_forms.view.restore'),
-          send('revert_' + @object.class.to_s.underscore + "_path",
-            version,
-            :update => "#{@object.class.name.underscore}_#{@object.id}"
-          ),
-          :remote => true,
-          :method => :post
-        %>
+        <% row_id = "#{@object.class.name.underscore}_#{@object.id}" %>
+        <%= link_to t("inline_forms.view.restore"),
+              send("revert_#{@object.class.to_s.underscore}_path", version, update: row_id),
+              inline_forms_turbo_link_data(row_id, method: :post) %>
       <% else %>
         &nbsp;
       <% end %>

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

@@ -0,0 +1,3 @@
+<turbo-frame id="<%= @update_span %>">
+  <%= render partial: "inline_forms/list" %>
+</turbo-frame>

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

@@ -0,0 +1,3 @@
+<turbo-frame id="<%= @update_span %>">
+  <%= render partial: "inline_forms/edit" %>
+</turbo-frame>

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

@@ -0,0 +1,3 @@
+<turbo-frame id="<%= @update_span %>">
+  <%= inline_forms_field_show(@object, @attribute, @form_element, turbo_frame: @turbo_field_show_turbo_frame) %>
+</turbo-frame>

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

@@ -0,0 +1,3 @@
+<turbo-frame id="<%= @update_span %>">
+  <%= render partial: "inline_forms/new" %>
+</turbo-frame>

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

@@ -0,0 +1,5 @@
+<turbo-frame id="<%= @update_span %>">
+  <div class="row odd top-level">
+    <%= render partial: "inline_forms/close" %>
+  </div>
+</turbo-frame>

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

@@ -0,0 +1,9 @@
+<turbo-frame id="<%= @update_span %>">
+  <div class="row odd top-level">
+    <div class="small-12 column">
+      <%= link_to t("inline_forms.view.undo"),
+            send("revert_#{@Klass.to_s.underscore}_path", @undo_version, update: @update_span),
+            inline_forms_turbo_link_data(@update_span, method: :post).merge(class: "button") %>
+    </div>
+  </div>
+</turbo-frame>

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

@@ -0,0 +1,3 @@
+<turbo-frame id="<%= @update_span %>">
+<%= render partial: "inline_forms/show" %>
+</turbo-frame>

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

@@ -0,0 +1,3 @@
+<turbo-frame id="<%= @update_span %>">
+  <%= render partial: "inline_forms/versions_list" %>
+</turbo-frame>

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

@@ -0,0 +1,3 @@
+<turbo-frame id="<%= @update_span %>">
+  <%= render partial: "inline_forms/versions" %>
+</turbo-frame>

data/app/views/layouts/application.html.erb

@@ -24,6 +24,5 @@
       window.Turbo = Turbo
     </script>
     <%= javascript_include_tag "https://unpkg.com/trix@1.3.1/dist/trix.js" %>
-    <%= javascript_include_tag Ckeditor.cdn_url if defined?(Ckeditor) %>
   </body>
 </html>

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

@@ -12,6 +12,16 @@
 
   <body>
     <%= render "inline_forms/header" %>
+    <%# Read string keys (FlashHash stores strings); avoid `notice`/`alert` helpers %>
+    <%# so we do not depend on add_flash_types wiring in every layout context. %>
+    <% flash_notice = flash["notice"] %>
+    <% flash_alert = flash["alert"] %>
+    <% if flash_notice.present? || flash_alert.present? %>
+      <div class="row" id="inline_forms_layout_flash" role="status">
+        <% if flash_notice.present? %><div class="flash notice"><%= flash_notice %></div><% end %>
+        <% if flash_alert.present? %><div class="flash alert"><%= flash_alert %></div><% end %>
+      </div>
+    <% end %>
     <%= render "/inline_forms_tabs" %>
     <div id="outer_container">
       <%= yield %>
@@ -28,6 +38,5 @@
       window.Turbo = Turbo
     </script>
     <%= javascript_include_tag "https://unpkg.com/trix@1.3.1/dist/trix.js" %>
-    <%= javascript_include_tag Ckeditor.cdn_url if defined?(Ckeditor) %>
   </body>
 </html>

data/bin/inline_forms

@@ -110,7 +110,28 @@ module InlineForms
 
 				app_template_file = File.join(File.dirname(__FILE__), 'inline_forms_app_template.rb')
 
-        if ! run("rails new #{app_name} -m #{app_template_file} --skip-bundle --skip-bootsnap --javascript=importmap")
+        # Prefer a Rails 7.0.x generator when one is installed locally: the
+        # generated `Gemfile`/`config/application.rb` pin to `rails ~> 7.0.0`,
+        # so running `rails new` from a newer system Rails (e.g. 8.0.x) emits
+        # 7.1+/8.0-only configuration (`load_defaults 8.0`,
+        # `config.autoload_lib(...)`) that aborts on the next `bundle exec`.
+        # The installer_core also patches `config/application.rb` defensively;
+        # the explicit version selector is the cleaner first line of defense.
+        require 'rubygems'
+        compatible_rails =
+          begin
+            Gem::Specification
+              .find_all_by_name("rails")
+              .map(&:version)
+              .select { |v| v >= Gem::Version.new("7.0") && v < Gem::Version.new("7.1") }
+              .max
+          rescue StandardError
+            nil
+          end
+        rails_invocation = compatible_rails ? "rails _#{compatible_rails}_" : "rails"
+        say "Generating app with: #{rails_invocation} new ...", :green
+
+        if ! run("#{rails_invocation} new #{app_name} -m #{app_template_file} --skip-bundle --skip-bootsnap --javascript=importmap")
           say "Rails could not create the app '#{app_name}', maybe because it is a reserved word...", :red # TODO ROYTJE MAKE ERROR MESSAGE MORE RELEVANT # Rails could not create the app 'MyApp', maybe because it is a reserved word..
           exit 1
         end

data/bin/inline_forms_installer_core.rb

@@ -5,12 +5,33 @@ GENERATOR_PATH = File.dirname(File.expand_path(__FILE__)) +  '/../'
 remove_file 'Gemfile' if File.exist?('Gemfile')
 create_file 'Gemfile', "# created by inline_forms #{ENV['inline_forms_version']} on #{Date.today}\n"
 
+# `rails new` is invoked with whatever the system `rails` binary points at
+# (often Rails 8.x once it lands in the global gemset), so the generated
+# `config/application.rb` may carry Rails 7.1+/8.0 idioms (`load_defaults
+# 8.0`, `config.autoload_lib(...)`). The Gemfile we write below pins
+# `rails ~> 7.0.0`, so Rails 7.0 must be able to interpret application.rb;
+# otherwise the first `bundle exec rails …` aborts with `Unknown version
+# "8.0"` or `NoMethodError: undefined method 'autoload_lib'`.
+if File.exist?('config/application.rb')
+  gsub_file 'config/application.rb',
+            /config\.load_defaults\s+\d+\.\d+/,
+            'config.load_defaults 7.0'
+  # Strip Rails 7.1+ `config.autoload_lib(ignore: ...)` (and any surrounding
+  # explanatory comment block). Not supported on Rails 7.0.
+  gsub_file 'config/application.rb',
+            /^\s*#[^\n]*\n(\s*#[^\n]*\n)*\s*config\.autoload_lib\([^)]*\)\s*\n/,
+            ""
+  gsub_file 'config/application.rb',
+            /^\s*config\.autoload_lib\([^)]*\)\s*\n/,
+            ""
+end
+
 add_source 'https://rubygems.org'
 
 gem 'cancancan'
 gem 'carrierwave', '~> 3.1'
-gem 'devise-i18n', :git => 'https://github.com/acesuares/devise-i18n.git'
-gem 'devise'
+gem 'devise', '~> 5.0'
+gem 'devise-i18n', '~> 1.16'
 gem 'autoprefixer-rails'
 # foundation-rails 6.7+ uses Dart Sass (`sass:math`); sass-rails/sassc removed.
 # Visually tuned against foundation-rails ~> 6.6.2; current pin ~> 6.9 (6.9.0.x).
@@ -443,7 +464,7 @@ sleep 1 # unique migration timestamps per generator
 generate "inline_forms", "InlineFormsKey name:string inline_forms_translations:has_many inline_forms_translations:associated _enabled:yes _presentation:\#{name}"
 sleep 1
 generate "inline_forms", "InlineFormsTranslation inline_forms_key:belongs_to inline_forms_locale:dropdown value:text interpolations:text is_proc:boolean _presentation:\#{value}"
-# TODO: fix text_area into text_area_without_ckeditor
+# Plain long text uses :plain_text; ActionText-backed fields use :rich_text.
 sleep 1 # to get unique migration number
 create_file "db/migrate/" +
   Time.now.utc.strftime("%Y%m%d%H%M%S") +
@@ -756,6 +777,19 @@ if ENV['install_example'] == 'true'
 
   remove_file 'public/index.html'
 
+  say "- Apartment name list demo (field-level inline edit without _show)..."
+  inject_into_file "app/controllers/apartments_controller.rb",
+                   "\n  skip_load_and_authorize_resource only: :name_list\n\n  def name_list\n    authorize! :read, Apartment\n    @apartments = Apartment.accessible_by(current_ability).order(:id).limit(10)\n  end\n",
+                   after: "set_tab :apartment\n"
+
+  example_views_root = File.join(GENERATOR_PATH, "lib/installer_templates/example_app_views")
+  Dir.glob(File.join(example_views_root, "**", "*")).sort.each do |abs|
+    next unless File.file?(abs)
+    rel = abs.delete_prefix(example_views_root + File::SEPARATOR).tr("\\", "/")
+    create_file File.join("app/views", rel), File.read(abs)
+  end
+
+  route 'get "apartments/name_list", to: "apartments#name_list", as: :apartment_name_list'
   route "root :to => 'apartments#index'"
 
   say "- Adding example app regression tests (bundle exec rails test)..."
@@ -768,6 +802,7 @@ if ENV['install_example'] == 'true'
   say "\nDone! Example app (Photo + Apartment) is ready.", :yellow
   say "  bundle exec rails test     # example regression tests", :yellow
   say "  bundle exec rails s        # then http://localhost:3000/apartments", :yellow
+  say "  More menu → Apartment names (first 10)  # /apartments/name_list", :yellow
   say "  Log in: #{ENV["email"]} / #{ENV["password"]}", :yellow
 end
 # done!

data/docs/ujs-to-turbo.md

@@ -0,0 +1,193 @@
+# UJS → Turbo migration checklist
+
+Track progress toward full Turbo integration and removal of jQuery UJS from inline_forms generated apps.
+
+**Current gem version:** see `lib/inline_forms/version.rb` (Step 3 complete in **7.5.0** except tree-related `*.js.erb` — Step 4)
+
+**Architecture today:** almost every inline interaction is `remote: true` → `format.js` → `*.js.erb` doing `$('#<update_span>').html(...)`. Turbo is loaded as an ES module with **`Turbo.session.drive = false`**. One vertical slice (nested has_many list pagination) uses **`<turbo-frame>`**.
+
+**Target end state:** Turbo Frames/Streams for all swaps; **`format.html`** / **`format.turbo_stream`** responses; no `*.js.erb`; no `jquery_ujs` or `jquery.remotipart`; Drive enabled.
+
+---
+
+## Step 1 — Turbo wired (DONE)
+
+- [x] `gem 'turbo-rails'` in installer Gemfile (`bin/inline_forms_installer_core.rb`)
+- [x] Turbo loaded as `<script type="module">` in `layouts/inline_forms.html.erb` and `layouts/application.html.erb`
+- [x] `Turbo.session.drive = false` (UJS still owns navigation)
+- [x] Turbo **not** in Sprockets bundle (`app/assets/javascripts/inline_forms/inline_forms.js`) — ESM parse-error lesson from 7.1.1
+- [x] Smoke test: `lib/installer_templates/example_app_tests/test/integration/example_app_turbo_layout_test.rb`
+
+---
+
+## Step 2 — Nested list + pagination as Turbo Frame (DONE)
+
+- [x] `_list.html.erb`: nested has_many container is `<turbo-frame id="…_list">` when `parent_class` present
+- [x] Nested pagination: no `:remote => true`; `update=` param matches frame id (`…_list` suffix)
+- [x] `InlineFormsController#index`: HTML for nested frame requests; `turbo_rails/frame` vs `inline_forms` layout negotiation
+- [x] **Nested rows (7.4.2):** per-row `<turbo-frame>` + Turbo presentation links; **`row_html_turbo_allowed?`** enables **`format.html`** row open/close for **`not_accessible_through_html?`** models when **`params[:update]`** is a nested associated row id (`apartment_<aid>_photo_<pid>`). Removed row-level **`data-turbo="false"`** (field cancel + pagination use Turbo inside nested frames).
+- [x] Top-level rows must **not** carry `data-turbo="false"` (would poison inner frames — 7.2.3)
+- [x] Smoke test: `example_app_apartment_photos_pagination_test.rb`
+- [x] Example seed data (Konferensha + photos) for pagination assertions
+
+**Explicitly deferred in this step:** top-level index lists (`/apartments`) stay full-page + UJS row open.
+
+---
+
+## Step 3 — Per-row / per-field inline-edit lifecycle
+
+Convert the **`show → edit → update → show_element → close`** cycle without UJS.
+
+### DOM contract (unchanged)
+
+- Every editable region needs a stable id: `{model}_{id}_{attribute}` (e.g. `apartment_5_name`)
+- `params[:update]` must match that id (set by `link_to_inline_edit`, `_edit.html.erb`, etc.)
+
+### Row-level (stock list → full `_show` panel)
+
+- [x] Wrap each top-level list row in `<turbo-frame id="apartment_<id>">` (`_list.html.erb` when `parent_class` is nil)
+- [x] Row title link: GET `show` → HTML `row_show` / `_show` inside frame (no `:remote`; `data-turbo` + `data-turbo-frame`)
+- [x] `InlineFormsController#show` (full record, no `params[:attribute]`): `format.html` → `row_show` / `row_close` + `turbo_rails/frame` when `turbo_frame_request?`, else full `inline_forms` layout; `format.js` still renders `show.js.erb` / `close.js.erb` for legacy callers
+- [x] `close_link` and `_close` presentation link: Turbo when `@inline_forms_turbo_row` (row HTML path)
+- [x] `soft_delete`, `soft_restore`, `destroy`, `revert`: `format.html` → `row_close` / `row_destroyed` (+ Turbo toolbar links); `format.js` kept for non-frame callers only
+- [x] Remove `edit.js.erb`, `update.js.erb`, `show_element.js.erb` (field lifecycle is Turbo HTML only)
+- [ ] Remove `show.js.erb`, `close.js.erb`, `record_destroyed.js.erb`, `show_undo.js.erb` — **`_tree.html.erb` still uses UJS row open (Step 4)**
+- [x] Remove `:remote => true` from nested `_list` / `_close` / toolbar where migrated (row toolbar, versions, nested `+` use Turbo; `_tree` / top-level `new` UJS deferred to Step 4)
+- [x] Remove `data-turbo="false"` from nested rows once nested inline-edit is Turbo-native (7.4.2)
+
+### Nested associated lists (e.g. Apartment → Photo)
+
+- [x] Each nested row is `<turbo-frame id="{parent}_{id}_{assoc}_{child_id}">` with Turbo presentation links (same contract as top-level).
+- [x] **`row_html_turbo_allowed?`:** `format.html` row open/close for **`not_accessible_through_html?`** models when **`params[:update]`** matches a nested associated row id (≥4 underscore segments, trailing numeric id).
+- [x] Scalar field edit/cancel inside nested **`_show`** uses existing **`format.html`** field templates (no **`UnknownFormat`** for Photo).
+
+### Field-level (`link_to_inline_edit` / `*_show` helpers)
+
+- [x] **Stock `_show` scalar fields**: wrapped in `<turbo-frame id="{model}_{id}_{attribute}">`; `@inline_forms_turbo_field = true` so `link_to_inline_edit` omits `:remote => true`
+- [x] **`link_to_inline_edit`**: uses Turbo when `@inline_forms_turbo_field` or explicit `turbo_frame: true`
+- [x] **`InlineFormsController#edit`, `#update`, `#show`** (single attribute): `format.html` + `field_edit` / `field_show` templates when `turbo_frame_request?`
+- [x] **`_edit.html.erb`**: omits `:remote => true` when `@turbo_frame` (Turbo form submit inside frame)
+- [x] Remove `edit.js.erb`, `update.js.erb`, `show_element.js.erb`
+- [x] **`not_accessible_through_html?` models** (e.g. Photo): nested list row **`show` / `close`** use **`format.html`** when **`params[:update]`** is a nested row id (`row_html_turbo_allowed?`); field **`edit` / `update` / `show`** already always register **`format.html`**.
+
+### Widget re-init after swap
+
+- [x] ActionText/Trix: `turbo:frame-load` in `inline_forms.js` attaches Trix editors after frame replace
+- [x] jQuery UI datepicker / timepicker: re-bound on `turbo:frame-load` (autocomplete widgets in field partials still use inline scripts until Step 5)
+
+### Helpers (`app/helpers/inline_forms_helper.rb`)
+
+- [x] `close_link`, `link_to_soft_delete`, `link_to_destroy`, `link_to_new_record`, `link_to_versions_list`, `close_versions_list_link`: Turbo when `turbo_row:` (default **true**); legacy `remote: true` only when `turbo_row: false`
+
+### Tests
+
+- [x] Integration: stock row open/close on `/apartments` (Turbo frame + HTML `show` / `close`): `example_app_apartment_row_turbo_test.rb`
+- [x] Integration: open apartment row → edit text field → save → cancel (field flow: `example_app_apartment_field_turbo_test.rb`)
+- [x] Integration: nested Photo row open/close + name field cancel (`example_app_apartment_photos_pagination_test.rb`)
+- [x] Integration: replace photo image (multipart) inside nested frame (`example_app_apartment_photos_pagination_test.rb`)
+- [x] Integration: custom field-only page (`ApartmentsController#name_list`) — Turbo edit/update/cancel without full `_show`
+- [x] Assert no `406 UnknownFormat` on Turbo field update (name list test)
+- [x] Integration: row destroy + PaperTrail revert via Turbo (`example_app_apartment_row_turbo_test.rb`)
+- [x] Integration: versions panel open/close via Turbo (`example_app_apartment_versions_turbo_test.rb`)
+
+---
+
+## Step 4 — Remaining UJS surfaces
+
+### Lists and create flow
+
+- [ ] Top-level index: wrap in `<turbo-frame id="apartments_list">`; in-frame pagination
+- [ ] `new` / `create`: frame refresh or stream replacing list container
+- [ ] Remove `new.js.erb`, `list.js.erb` (keep `format.js` only until Step 5 if needed for overlap)
+- [ ] `_new.html.erb`: drop `:remote => true`
+
+### Tree
+
+- [ ] `_tree.html.erb`: same frame pattern as `_list`; remote pagination → frame pagination
+- [ ] Remove tree-related UJS if any dedicated `*.js.erb` remain
+
+### Versions panel
+
+- [ ] `VersionsConcern#list_versions`: HTML frame / stream
+- [ ] Remove `versions.js.erb`, `versions_list.js.erb`
+- [ ] `_versions_list.html.erb`: drop `:remote => true` on revert links (or stream revert)
+
+### Geo / misc
+
+- [ ] `geo_code_curacao/list_streets.js.erb`: frame or stream for street dropdown
+
+### Controller cleanup
+
+- [ ] Every action in `InlineFormsController` + `VersionsConcern` has a non-JS response path
+- [ ] Audit `respond_to` blocks: parallel `format.turbo_stream` where stream is cleaner than full frame
+
+### Tests
+
+- [ ] Top-level list pagination in frame
+- [ ] Create apartment → list frame updates
+- [ ] Versions panel open/close
+- [ ] Assert zero `data-remote="true"` in rendered HTML for inline_forms flows
+
+---
+
+## Step 5 — Enable Drive, remove UJS
+
+- [ ] Remove `Turbo.session.drive = false` from both layouts
+- [ ] Delete `//= require jquery_ujs` from `inline_forms.js`
+- [ ] Delete `//= require jquery.remotipart` from `inline_forms.js`
+- [ ] Delete all `app/views/inline_forms/*.js.erb` (and geo `list_streets.js.erb`)
+- [ ] Remove `format.js` branches from controllers (or empty stubs, then delete)
+- [ ] Update `example_app_turbo_layout_test.rb`: Drive enabled (or line removed)
+- [ ] Full `bundle exec rails test` in `--example` app
+
+### jQuery (optional follow-up — not required to drop UJS)
+
+These can remain while UJS is gone; separate migration if desired:
+
+- [ ] `jquery.ui.all` (datepicker)
+- [ ] `jquery.timepicker.js`
+- [ ] `autocomplete-rails`
+- [ ] `foundation` jQuery plugin → consider Foundation JS without jQuery or Stimulus
+
+---
+
+## Reference — UJS inventory
+
+### `*.js.erb` (all to delete by Step 5)
+
+| File | Effect |
+|------|--------|
+| `show.js.erb` | Replace row with `_show` |
+| `edit.js.erb` | Replace `#update_span` with `_edit` form |
+| `update.js.erb` | Replace with `{form_element}_show` |
+| `show_element.js.erb` | Single-field show after update |
+| `new.js.erb` | Replace with `_new` form |
+| `list.js.erb` | Replace list container |
+| `close.js.erb` | Fade + `_close` partial |
+| `record_destroyed.js.erb` | Fade out row |
+| `show_undo.js.erb` | Undo link after destroy |
+| `versions.js.erb` | Fade + versions partial |
+| `versions_list.js.erb` | Versions table |
+| `geo_code_curacao/list_streets.js.erb` | Street list dropdown |
+
+### Controller actions still on `format.js`
+
+`index`, `show`, `edit`, `update`, `new`, `create`, `soft_delete`, `soft_restore`, `destroy`, `revert`, `list_versions`
+
+### Key files
+
+- `app/controllers/inline_forms_controller.rb`
+- `app/controllers/concerns/versions_concern.rb`
+- `app/views/inline_forms/_list.html.erb`, `_show.html.erb`, `_edit.html.erb`, `_new.html.erb`
+- `app/helpers/inline_forms_helper.rb`
+- `app/helpers/form_elements/*.rb` (`*_show` → `link_to_inline_edit`)
+- `app/assets/javascripts/inline_forms/inline_forms.js`
+- `app/views/layouts/inline_forms.html.erb`
+
+---
+
+## Custom field-only pages (helper bypass)
+
+Stock `_show` / `_list` are not required for inline edit. Any page can call form-element helpers (e.g. `text_field_show(apartment, :name)`) inside a container with id `apartment_<id>_name`. Edit/update still hit `ApartmentsController#edit` / `#update` via polymorphic paths.
+
+Example app **`--example` name list** (`GET /apartments/name_list`): custom page using the **same** turbo-field contract as stock `_show` (not a separate code path). Linked from the **More** menu; regression-tested after stock field Turbo lands.

data/lib/generators/USAGE

@@ -3,7 +3,7 @@ Description:
     We will generate a model, a migration, a controller and a route. Please do check the migration and the model because they will likely need tweaking.
 
 Example:
-    rails generate inline_forms Thing name:text_field description:text_area yesno:check_box gender:boolean_with_values
+    rails generate inline_forms Thing name:text_field description:plain_text yesno:check_box gender:boolean_with_values
 
 This will create:
       create  app/models/thing.rb
@@ -19,7 +19,7 @@ This will create:
           def inline_forms_field_list
             [
                 [ :name, 'name', :text_field ],
-                [ :description, 'description', :text_area ],
+                [ :description, 'description', :plain_text ],
                 [ :yesno, 'yesno', :check_box ],
                 [ :gender, 'gender', :boolean_with_values ],
             ]

data/lib/generators/assets/stylesheets/inline_forms.scss

@@ -223,6 +223,14 @@ select:hover, select:focus {
   font-weight: bold;
   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).
+turbo-frame {
+  display: block;
+}
 .list_container {
   .row {
     font-size: 1.2rem;
@@ -258,6 +266,30 @@ select:hover, select:focus {
   }
 }
 
+// Field edit cancel: outer <a> carries Turbo/UJS; inner input[type=button] matches ok height.
+.edit_form .row.collapse {
+  input[type="submit"].postfix.button,
+  a.inline_forms-field-cancel input[type="button"].postfix.button {
+    margin: 2px 0 !important;
+  }
+}
+
+.edit_form a.inline_forms-field-cancel {
+  display: inline-block;
+  padding: 0;
+  border: 0;
+  background: transparent;
+  line-height: 0;
+  vertical-align: top;
+  text-decoration: none;
+
+  input[type="button"] {
+    pointer-events: none;
+    cursor: pointer;
+    width: auto;
+  }
+}
+
 .object_presentation {
   background-color: #B94C32;
   color: white;
@@ -434,22 +466,6 @@ select:hover, select:focus {
   margin-bottom: 0.5em;
 }
 
-.ckeditor_area {
-    position: relative;
-}
-
-.ckeditor_area .glass_plate {
-    position: absolute;
-    top: -1px;
-    width: 98%;
-    height: 232px;
-    border: 0;
-}
-
-.ckeditor_area .cke_top, .ckeditor_area .cke_bottom, .ckeditor_area .cke_border {
-    display: none;
-}
-
 /* jQuery ui Slider 8 */
 .slider {
     width: 300px;

data/lib/inline_forms/version.rb

@@ -1,4 +1,4 @@
 # -*- encoding : utf-8 -*-
 module InlineForms
-  VERSION = "7.2.11"
+  VERSION = "7.5.2"
 end