inline_forms 7.2.11 → 7.9.1

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).
@@ -36,7 +57,6 @@ gem 'rails-i18n', '~> 7.0'
 gem 'rails-jquery-autocomplete'
 gem 'rails', '~> 7.0.0'
 gem 'rake'
-gem 'remotipart', '~> 1.0'
 gem 'rvm'
 gem 'dartsass-rails'
 # Rails 7 no longer adds sprockets-rails to the default Gemfile; declare it
@@ -45,10 +65,9 @@ gem 'dartsass-rails'
 gem 'sprockets-rails'
 # Rails 7 default JavaScript tooling: importmap-rails replaces Webpacker.
 gem 'importmap-rails'
-# Hotwire/Turbo. Loaded into the Sprockets bundle by inline_forms.js with
-# Turbo Drive disabled by default so existing UJS-driven links/forms keep
-# working; turbo-rails also registers the `turbo_stream` Mime type and view
-# format so future controllers can opt in to Turbo Stream responses.
+# Hotwire/Turbo. Loaded from layouts as `<script type="module">`; inline flows
+# use `<turbo-frame>` + HTML responses (see docs/ujs-to-turbo.md). Registers the
+# `turbo_stream` MIME type for optional stream responses.
 gem 'turbo-rails'
 gem 'tabs_on_rails', :git => 'https://github.com/acesuares/tabs_on_rails.git', :branch => 'update_remote_before_action'
 gem 'unicorn'
@@ -443,7 +462,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") +
@@ -661,6 +680,73 @@ if ENV['install_example'] == 'true'
   run 'bundle exec rails generate uploader Image'
   run 'bundle exec rails g inline_forms Apartment name:string title:string description:rich_text photos:has_many photos:associated _enabled:yes _presentation:\'#{name}\''
 
+  say "- Apartment name is required..."
+  inject_into_file "app/models/apartment.rb",
+                   "\n  validates :name, presence: true\n",
+                   after: "  has_paper_trail\n"
+
+  # CarrierWave + PaperTrail history.
+  # PaperTrail snapshots the column scalar (the stored filename) on update,
+  # but CarrierWave's default `remove_previously_stored_files_after_update`
+  # deletes the old file on disk and re-uses the same filename, so a
+  # PaperTrail revert restores a filename whose bytes are gone.
+  # We keep every uploaded file on disk and namespace filenames with a
+  # per-upload token so successive uploads do not collide. See
+  # https://stackoverflow.com/questions/9423279/papertrail-and-carrierwave
+  # (Answers 2, 4 and 5).
+  say "- Configuring CarrierWave to keep previously stored files (PaperTrail history)..."
+  create_file "config/initializers/carrierwave.rb", <<-CWINIT.strip_heredoc
+    # Keep previously stored files on disk so PaperTrail-driven restore
+    # actually returns the previous image bytes. See
+    # https://stackoverflow.com/questions/9423279/papertrail-and-carrierwave
+    # The per-uploader overrides in app/uploaders/image_uploader.rb
+    # complement this by giving every upload a unique on-disk filename
+    # and by no-op'ing `remove!` so destroyed records keep their files.
+    CarrierWave.configure do |config|
+      config.remove_previously_stored_files_after_update = false
+    end
+  CWINIT
+
+  inject_into_file "app/uploaders/image_uploader.rb",
+                   after: "class ImageUploader < CarrierWave::Uploader::Base\n" do
+    <<-RUBY.strip_heredoc.gsub(/^/, "  ")
+      # PaperTrail history support. CarrierWave's default behaviour wipes the
+      # previous file on update and reuses the same filename; PaperTrail only
+      # stores the column scalar, so a plain `version.reify; save!` restores a
+      # filename whose bytes are gone. The knobs below preserve every byte:
+      #
+      #   * `remove_previously_stored_files_after_update = false` is set
+      #     globally in config/initializers/carrierwave.rb (covers
+      #     `multi_image_field` uploaders too).
+      #   * `remove!` is a no-op so hard-destroyed records keep their files
+      #     and revert-after-destroy still finds the bytes on disk.
+      #   * `filename` is prefixed with a per-upload UUID so successive
+      #     uploads never collide on disk.
+      #
+      # Trade-off: files accumulate on disk; sweeping is out of scope.
+      # Source: https://stackoverflow.com/questions/9423279/papertrail-and-carrierwave
+      def remove!
+        # no-op: keep the file so PaperTrail revert can restore it.
+      end
+
+      def filename
+        # CarrierWave 3.x calls `filename` again after storing to record the
+        # persisted name; at that point `original_filename` may be nil and we
+        # must still return the memoized name (see
+        # https://github.com/carrierwaveuploader/carrierwave/issues/2708).
+        @name ||= "\#{secure_token}-\#{original_filename}" if original_filename
+        @name
+      end
+
+      private
+
+      def secure_token
+        var = :"@\#{mounted_as}_secure_token"
+        model.instance_variable_get(var) || model.instance_variable_set(var, SecureRandom.uuid)
+      end
+    RUBY
+  end
+
   say "- Lower Photo.per_page so the seeded gallery paginates..."
   # The model template (lib/generators/templates/model.erb) emits
   #   attr_reader :per_page
@@ -756,6 +842,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 +867,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,192 @@
+# 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 5** — Turbo Drive on, jquery-ujs / remotipart removed in **7.8.0**)
+
+**Architecture today:** inline interactions use **`<turbo-frame>`** + **`format.html`** (and optional **`format.turbo_stream`**). Turbo loads as an ES module from the layouts; **Turbo Drive** uses the library default (**enabled**). No `*.js.erb` in active engine views; no `jquery_ujs` or `jquery.remotipart` in the Sprockets bundle.
+
+**Target end state:** Stock flows use Turbo Frames + HTML; optional `turbo_stream` where it simplifies a response. jQuery UJS and `*.js.erb` are gone from the engine.
+
+---
+
+## 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` uses the **default (enabled)** — **7.8.0** (was disabled while UJS coexisted)
+- [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
+- [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)
+- [x] Remove `edit.js.erb`, `update.js.erb`, `show_element.js.erb` (field lifecycle is Turbo HTML only)
+- [x] Remove `show.js.erb`, `close.js.erb`, `record_destroyed.js.erb`, `show_undo.js.erb` (7.7.0; tree migrated)
+- [x] Remove `:remote => true` from nested `_list` / `_close` / toolbar where migrated (row toolbar, versions, nested `+` use Turbo; top-level `new` was 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 frame attrs only (**7.8.0**; `turbo_row:` retained for API compat)
+
+### 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
+
+- [x] Top-level index: wrap in `<turbo-frame id="apartments_list">`; in-frame pagination (7.7.0)
+- [x] `new` / `create`: frame refresh replacing list container (7.7.0)
+- [x] Remove `new.js.erb`, `list.js.erb` (7.7.0)
+- [x] `_new.html.erb`: Turbo path when `@turbo_frame` (no `:remote` on cancel)
+
+### Tree
+
+- [x] ~~`:tree` / `_tree.html.erb`~~ — **archived in 7.7.0** (`archived/form_elements/tree/`); needs host tree gem/APIs. Turbo-ready partial kept in archive.
+
+### Versions panel
+
+- [x] `VersionsConcern#list_versions`: HTML frame (`versions_panel` / `versions_list_panel`, `turbo_rails/frame` layout) — **7.7.3** drops `format.js`
+- [x] Remove `versions.js.erb`, `versions_list.js.erb` (**7.7.3**)
+- [x] `_versions_list.html.erb`: revert uses `inline_forms_turbo_link_data` (row frame), not `:remote => true`
+
+### Geo / misc
+
+- [x] ~~`geo_code_curacao`~~ — **archived in 7.6.0** (`archived/form_elements/geo_code_curacao/`). If restored, migrate `list_streets.js.erb` to frame or stream.
+
+### Controller cleanup
+
+- [x] Every action in `InlineFormsController` + `VersionsConcern` has a non-JS response path (**7.8.0** audit)
+- [ ] Audit `respond_to` blocks: parallel `format.turbo_stream` where stream is cleaner than full frame (optional)
+
+### Tests
+
+- [x] Top-level list pagination in frame (`example_app_apartment_top_level_pagination_test.rb`)
+- [x] Create apartment → list frame updates (`example_app_apartment_top_level_new_test.rb`)
+- [x] Versions panel open/close + revert from list (`example_app_apartment_versions_turbo_test.rb`, **7.7.3**)
+- [x] Assert zero `data-remote="true"` in rendered HTML for inline_forms flows (helpers + `_new` / `_close` no longer fall back to UJS — **7.8.0**)
+
+---
+
+## Step 5 — Enable Drive, remove UJS (DONE in **7.8.0**)
+
+- [x] Remove `Turbo.session.drive = false` from both layouts (Drive default **on**)
+- [x] Delete `//= require jquery_ujs` from `inline_forms.js`
+- [x] Delete `//= require jquery.remotipart` from `inline_forms.js`
+- [x] Delete all `app/views/inline_forms/*.js.erb` (completed by **7.7.3**)
+- [x] Remove `format.js` branches from controllers (none remain on stock actions)
+- [x] Update `example_app_turbo_layout_test.rb`: refute `Turbo.session.drive = false`
+- [x] Full `bundle exec rails test` in `--example` app (before release)
+
+### 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`~~ | Removed **7.7.3** — versions panel HTML frame |
+| ~~`versions_list.js.erb`~~ | Removed **7.7.3** — versions list HTML frame |
+| ~~`geo_code_curacao/list_streets.js.erb`~~ | Archived 7.6.0 — street autocomplete JSON |
+
+### Controller actions still on `format.js`
+
+None on `InlineFormsController` / `VersionsConcern` stock actions (**7.8.0**). Host app controllers may still use `format.js`.
+
+### 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,15 @@ select:hover, select:focus {
   font-weight: bold;
   font-size: 110%;
 }
+// Custom elements default to `display: inline`, which collapses the row layout
+// 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;
@@ -258,6 +267,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 +467,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/archived_form_elements.rb

@@ -0,0 +1,70 @@
+# -*- encoding : utf-8 -*-
+module InlineForms
+  class ArchivedFormElementError < StandardError; end
+
+  # Retired form-element symbols. Full source for entries with +archive_path+ lives
+  # under archived/form_elements/<name>/ — see archived/README.md.
+  ARCHIVED_FORM_ELEMENTS = {
+    geo_code_curacao: {
+      archived_in_version: "7.6.0",
+      archive_path: "archived/form_elements/geo_code_curacao",
+      summary: "Curaçao street geocode (MySQL Zones/Buurten/Straatcode, jQuery autocomplete, UJS list_streets).",
+    },
+    chicas_photo_list: {
+      archived_in_version: "7.6.0",
+      archive_path: "archived/form_elements/chicas",
+      summary: "Chicas app read-only member photo gallery (show-only).",
+    },
+    chicas_family_photo_list: {
+      archived_in_version: "7.6.0",
+      archive_path: "archived/form_elements/chicas",
+      summary: "Chicas app read-only family member photo gallery (show-only).",
+    },
+    chicas_dropdown_with_family_members: {
+      archived_in_version: "7.6.0",
+      archive_path: "archived/form_elements/chicas",
+      summary: "Chicas client picker via family.clients; moves CarrierWave upload dir on update.",
+    },
+    kansen_slider: {
+      archived_in_version: "7.6.0",
+      archive_path: "archived/form_elements/kansen_slider",
+      summary: "jQuery UI slider for integer-coded chance scale; uses model attribute_values.",
+    },
+    tree: {
+      archived_in_version: "7.7.0",
+      archive_path: "archived/form_elements/tree",
+      summary: "Self-referential children list via parent.children; requires host tree APIs (see README).",
+    },
+    move: {
+      archived_in_version: "7.7.0",
+      archive_path: "archived/form_elements/tree",
+      summary: "Reparent via hash_tree_to_collection + add_child (host must implement; pairs with :tree).",
+    },
+    absence_list: {
+      removed_in_version: "6.3.0",
+      archive_path: nil,
+      summary: "Project-specific absence list UI; removed without a copy in this repo (see CHANGELOG 6.3.0).",
+    },
+  }.freeze
+
+  def self.validate_no_archived_form_elements_for!(klass)
+    return unless klass.instance_methods.include?(:inline_forms_attribute_list)
+
+    klass.new.inline_forms_attribute_list.each do |attribute, _label, form_element|
+      key = form_element.to_sym
+      next unless ARCHIVED_FORM_ELEMENTS.key?(key)
+
+      meta = ARCHIVED_FORM_ELEMENTS[key]
+      version = meta[:archived_in_version] || meta[:removed_in_version]
+      hint = if meta[:archive_path]
+        "Restore from #{meta[:archive_path]}/README.md or vendor into your app."
+      else
+        "See archived/README.md and CHANGELOG #{version}."
+      end
+
+      raise ArchivedFormElementError,
+        "#{klass.name} inline_forms_attribute_list declares #{attribute}:#{form_element}, " \
+        "which was retired in inline_forms #{version} (#{meta[:summary]}). #{hint}"
+    end
+  end
+end

data/lib/inline_forms/version.rb

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

data/lib/inline_forms.rb

@@ -1,9 +1,12 @@
 # -*- encoding : utf-8 -*-
 require ('inline_forms/version.rb')
 require_relative ('inline_forms/form_element_from_callee')
+require_relative ('inline_forms/archived_form_elements')
 # InlineForms is a Rails Engine that let you setup an admin interface quick and
 # easy. Please install it as a gem or include it in your Gemfile.
 module InlineForms
+  class PlainTextColumnMissingError < StandardError; end
+
   # DEFAULT_COLUMN_TYPES holds the standard ActiveRecord::Migration column types.
   # This list provides compatability with the standard types, but we add our own
   # later in 'Special Column Types'.
@@ -62,7 +65,7 @@ module InlineForms
   #
   DEFAULT_FORM_ELEMENTS = {
     :string     => :text_field,
-    :text       => :text_area,
+    :text       => :plain_text,
     :integer    => :text_field,
     :float      => :text_field,
     :decimal    => :text_field,
@@ -93,6 +96,45 @@ module InlineForms
     :associated => :no_migration
   }
 
+  PLAIN_TEXT_FORM_ELEMENTS = %i[
+    plain_text
+    plain_text_area
+    text_area_without_ckeditor
+  ].freeze
+
+  def self.plain_text_form_element?(form_element)
+    PLAIN_TEXT_FORM_ELEMENTS.include?(form_element.to_sym)
+  rescue NoMethodError
+    false
+  end
+
+  def self.assert_plain_text_column!(object:, attribute:, form_element:)
+    return unless plain_text_form_element?(form_element)
+    return if object.class.column_names.include?(attribute.to_s)
+
+    raise PlainTextColumnMissingError,
+      "#{object.class.name}##{attribute} uses #{form_element} but has no DB column `#{attribute}`. " \
+      "Use :rich_text for ActionText-backed attributes, or add a text column for :plain_text."
+  end
+
+  def self.validate_plain_text_configuration_for!(klass)
+    return unless klass.respond_to?(:table_exists?) &&
+                  klass.respond_to?(:column_names) &&
+                  klass.instance_methods.include?(:inline_forms_attribute_list)
+    return unless klass.table_exists?
+
+    attributes = klass.new.inline_forms_attribute_list
+    attributes.each do |attribute, _label, form_element|
+      next unless plain_text_form_element?(form_element)
+      next if klass.column_names.include?(attribute.to_s)
+
+      raise PlainTextColumnMissingError,
+        "#{klass.name} inline_forms_attribute_list declares #{attribute}:#{form_element}, " \
+        "but table `#{klass.table_name}` has no `#{attribute}` column. " \
+        "Use :rich_text for ActionText-backed attributes."
+    end
+  end
+
   # RELATIONS defines a mapping between AR::Migrations columns and the Model.
   #
   # When a column has the type of :references or :belongs_to, then
@@ -144,7 +186,6 @@ module InlineForms
         inline_forms/inline_forms.css
         inline_forms/devise.css
         inline_forms/inline_forms.js
-        inline_forms/ckeditor/config.js
         inline_forms/glass_plate.gif
       )
     end
@@ -159,6 +200,23 @@ module InlineForms
       app.config.assets.paths << path unless app.config.assets.paths.include?(path)
     end
 
+    config.to_prepare do
+      next unless defined?(ActiveRecord::Base)
+
+      ActiveRecord::Base.descendants.each do |klass|
+        begin
+          InlineForms.validate_no_archived_form_elements_for!(klass)
+          InlineForms.validate_plain_text_configuration_for!(klass)
+        rescue InlineForms::PlainTextColumnMissingError, InlineForms::ArchivedFormElementError
+          raise
+        rescue StandardError
+          # Some descendants might be abstract or temporarily unresolved while
+          # the app boots/reloads; runtime checks in controllers still enforce
+          # plain_text column presence for active resources.
+        end
+      end
+    end
+
     I18n.load_path << Dir[File.join(File.expand_path(File.dirname(__FILE__) + '/locales'), '*.yml')]
     I18n.load_path.flatten!
   end