inline_forms 7.2.10 → 7.5.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: eb08c32173e4c264ca774ebf89cbd8f36a76fa0235604647d7373552843869ff
-  data.tar.gz: 3368aca794cd8fa8a0be1383c53b137402dd011fac1a1153b87d01adfc4e6210
+  metadata.gz: bbf211e9092af732b645bf10e9f9c338ced10c55e3ebf0a533146537669d1a4d
+  data.tar.gz: 90bc8cf55083ca23bbc1d3122a49e835620bf2e4800c8c83a93b871230d54718
 SHA512:
-  metadata.gz: '07290fc7a415f889892f8692f6af1a74647749b7e91503c0120847cc68f5e90e9b791a457a8a8775fd11880d72c89c30e437aa5641a1ee5aefec05c112eddb67'
-  data.tar.gz: 7811db2cfad1170a35e58659d6088f3acb0f9bb551d280db7437995893810262cee4efd9ed2014ee17e368478ebb3e9f59e04d6175d478c48220cc90356c9fba
+  metadata.gz: 5f66995db3089c59e1654f92f1a7c78b9701ccad3ea0a12fb4cf7729ea5ed1bff70003847af80ebcae6347c05e29f7e6fc9e8a9317252ea120a7a09d790c2284
+  data.tar.gz: 2e04d9869aff6e13a1eb6556e9865e640725dee6d9ad5a56887ed23f125bd9378ab679c773fd84cbc72b8df408335dd7026998e113254366e4cc358155c70785

data/CHANGELOG.md

@@ -4,6 +4,193 @@ All notable changes to this project are documented in this file.
 
 ## [Unreleased]
 
+## [7.5.2] - 2026-05-16
+
+### Fixed
+
+- **Field re-edit after Turbo update / cancel (Photo image, name, etc.):** `render_turbo_field` now sets **`@inline_forms_turbo_field = true`**, so the link inside the swapped **`<turbo-frame id="photo_<id>_image">`** carries **`data-turbo`** instead of legacy **`data-remote`**. 7.5.1 set the flag only in `_show.html.erb` (the row open template); the bare `field_show` re-render after a field cancel/update lost it, the link fell back to **`remote: true`**, jquery_ujs intercepted as a JS request the controller does not register, and the second click silently failed (no swap, no edit form).
+- **Top-level `+ new` flow (e.g. /apartments):** `link_to_new_record` falls back to **UJS (`remote: true`)** when called without a `parent_class`. The top-level list root stays a **`<div id="apartments_list">`** (a `<turbo-frame>` there collapses inside `position: absolute` `#outer_container` and hides rows under the fixed top bar), so a Turbo target on the **`+`** link is invalid -- 7.5.1 emitted `data-turbo-frame="apartments_list"`, which made cancel/create produce Turbo's "Content missing" or fall back to a full-page navigation. UJS (`new.js.erb` / `list.js.erb`) swap **`#apartments_list`** in place; nested has_many lists keep the Turbo contract unchanged.
+
+### Added
+
+- **Defensive CSS:** `turbo-frame { display: block; }` so any `<turbo-frame>` inside `#outer_container` (custom elements default to `display: inline`) does not collapse and hide its row content.
+- **Regression tests:**
+  - `example_app_apartment_top_level_new_test.rb` -- top-level list root stays `<div>`, **`+`** link is UJS, `new` / `cancel` / `create` go through `format.js` and swap **`#apartments_list`**.
+  - `example_app_apartment_photos_pagination_test.rb` -- after Turbo update / cancel of a Photo image field, `field_show` carries `data-turbo="true"` (not `data-remote="true"`).
+
+### Verified
+
+- **`bundle exec rails test`** -- **60 runs, 302 assertions, 0 failures**.
+- **curl:** field show after cancel/update emits `data-turbo="true" data-turbo-frame="_self"` (no `data-remote`); top-level new returns `format.js` swap of `#apartments_list`.
+- **Browser:** top-level `+ new Apartment` form renders inline + cancel returns to list (no "Content missing"); nested Photo image edit can be reopened immediately after cancel and after replacement.
+
+## [7.5.1] - 2026-05-16
+
+### Fixed
+
+- **Nested associated `+` (New) on `not_accessible_through_html?` models (Photo):** `new` now serves **`format.html`** when `parent_class` and `update` are present (was **406** / empty frame for Photo).
+- **New / Cancel / OK after New:** associated-list frame responses use the **`inline_forms`** layout (styled) instead of bare **`turbo_rails/frame`**; **`create`** restores the list with **`@ul_needed = true`** so the inner **`…_photos_list`** `<turbo-frame>` matches **`_show`** (fixes Turbo **“Content missing”** on cancel and create).
+- **`_new.html.erb`:** Turbo form sets **`data-turbo-frame`** to the parent list frame id.
+
+### Verified
+
+- **`bundle exec rails test`** — **53 runs, 260 assertions, 0 failures**; nested Photo new → cancel → create integration test.
+- **curl:** new **200** + stylesheet; cancel/create **200** with **`apartment_<id>_photos`** + **`…_photos_list`** frames.
+
+## [7.5.0] - 2026-05-16
+
+### Added
+
+- **Step 3 completion (UJS → Turbo row/field lifecycle):** row toolbar **`soft_delete` / `soft_restore` / `destroy` / `revert`** respond with **`format.html`** (`row_close`, `row_destroyed`) inside matching **`<turbo-frame>`**; helpers default to Turbo (`inline_forms_turbo_link_data`, `turbo_row:` on toolbar / versions / nested **`+`** links).
+- **Nested `+new` / `create` / versions panel:** `new_record`, `create_list_frame`, `versions_panel`, `versions_list_panel` HTML templates; associated/has_one/versions regions in **`_show`** wrapped in **`<turbo-frame>`**.
+- **`turbo:frame-load`** in **`inline_forms.js`:** re-init datepicker, timepicker, and Trix after frame swaps.
+- **Regression tests:** row destroy/revert Turbo, versions panel Turbo (`example_app_apartment_row_turbo_test.rb`, `example_app_apartment_versions_turbo_test.rb`).
+
+### Removed
+
+- **`edit.js.erb`**, **`update.js.erb`**, **`show_element.js.erb`** — scalar field edit/update/cancel is Turbo HTML only.
+
+### Changed
+
+- **`docs/ujs-to-turbo.md`:** Step 3 checklist marked done except **`show.js.erb` / `close.js.erb` / …** retained for **`_tree.html.erb`** (Step 4).
+
+### Verified (end-to-end against the `--example` install)
+
+- **`bundle exec rails test`** — **52 runs, 236 assertions, 0 failures, 0 errors, 0 skips** against the generated MyApp.
+- **curl smoke:** versions panel **`GET /apartments/1/list_versions?update=apartment_1_versions`** with **`Turbo-Frame`** → **200** + matching **`<turbo-frame id="apartment_1_versions">`**.
+- **curl + browser** — row toolbar, versions, field edit, nested photos (same contract as 7.4.x).
+
+## [7.4.5] - 2026-05-15
+
+### Added
+
+- **Example app integration test** (`example_app_apartment_photos_pagination_test.rb`): nested Photo **`image`** field — Turbo **`Turbo-Frame`** GET edit (multipart form) and **multipart `PUT`** — asserts **200** HTML with matching **`<turbo-frame id="…_image">`** and no **`UnknownFormat`** / **406** (Step 3 multipart regression guard from the 7.2.0 nested-frame era).
+
+### Changed
+
+- **`docs/ujs-to-turbo.md`:** Step 3 checklist — “replace photo image (multipart) inside nested frame” marked done.
+
+### Verified (end-to-end against the `--example` install)
+
+- **`bundle exec rails test`** — **47 runs, 208 assertions, 0 failures, 0 errors, 0 skips** against the generated MyApp (Apartment + Photo).
+- **curl smoke** (authenticated **`Turbo-Frame`** GETs / **`POST` multipart** with **`_method=put`**): same contract as 7.4.4 plus nested Photo **`/photos/1/edit?attribute=image&form_element=image_field&update=apartment_1_photo_<id>_image`** and image replace **`POST /photos/1?...`** — **200** and matching **`<turbo-frame id="…_image">`** (Devise scope **`/auth/users/sign_in`**; cookie jar must be sent on the multipart step).
+- **Browser** (cursor-ide-browser MCP, dev server): already-signed-in session — open **Konferensha** row (URL stays **`/`**, gallery appears) — **Next page** (gallery shows page-2 filenames) — **`/apartments/name_list`** — edit name — **ok** (read-only link shows new value).
+
+## [7.4.4] - 2026-05-16
+
+### Fixed
+
+- **`inline_forms create … --example` against a Rails 8 system gem:** `bin/inline_forms` now prefers a locally installed Rails **`~> 7.0`** (`rails _7.0.X_ new …`) when one is present, so the generated **`config/application.rb`** matches the **`rails ~> 7.0.0`** pin the installer writes into the **`Gemfile`**. Without this, a system **`rails 8.x`** wrote **`config.load_defaults 8.0`** and **`config.autoload_lib(ignore: …)`** into `application.rb`, both rejected by Rails 7.0 (`rails aborted! Unknown version "8.0"` and `NoMethodError: undefined method 'autoload_lib'` on the first **`bundle exec rails dartsass:install`**), so app generation aborted right after Dart Sass install.
+- **Defensive `application.rb` rewrite (belt + suspenders):** even when the picked generator is Rails 8.x (e.g. no 7.0 gem available), `bin/inline_forms_installer_core.rb` rewrites **`config.load_defaults <N>.<M>`** to **`config.load_defaults 7.0`** and strips **`config.autoload_lib(…)`** post-generation so the bundled Rails 7.0 can boot.
+
+### Changed
+
+- **`docs/ujs-to-turbo.md`:** Step 3 marks the apartment field flow integration test (**`example_app_apartment_field_turbo_test.rb`** — open row → edit text field → save → cancel) as done; the test has been shipping since 7.4.1.
+
+### Verified (end-to-end against the `--example` install)
+
+- **`bundle exec rails test`** — **46 runs, 196 assertions, 0 failures, 0 errors, 0 skips** against the generated MyApp (Apartment + Photo).
+- **curl smoke** with **`Turbo-Frame`** header: top-level row open/close (**`/apartments/1?update=apartment_1[&close=true]`**), scalar field edit/update/cancel (**`/apartments/1/edit?attribute=name&form_element=text_field&update=apartment_1_name`** + **`PUT /apartments/1`**), nested **`/photos?parent_class=Apartment&parent_id=1&update=apartment_1_photos_list`** pagination, and nested Photo row open/close (**`/photos/1?update=apartment_1_photo_1[&close=true]`**) all return **200** with the matching **`<turbo-frame id="…">`** in the body.
+- **Browser** (devtools MCP, Drive disabled, UJS for the legacy paths): sign-in → click apartment row (Turbo open, no full-page nav) → nested photo pagination Next (Turbo) → click name field (Turbo edit-in-place) → save (in-place swap, no reload) → close X (Turbo collapse, list shows updated name). **`/apartments/name_list`** field-edit demo confirmed reachable from the **More** menu.
+
+## [7.4.3] - 2026-05-16
+
+### Added
+
+- **Turbo row open/close on stock index:** each top-level list row is a **`<turbo-frame id="{model}_{id}">`**; presentation links use Turbo (**`row_show.html.erb`** / **`row_close.html.erb`**, **`close_link(..., turbo_row:)`**, **`example_app_apartment_row_turbo_test.rb`**).
+
+### Fixed
+
+- **Nested associated list (e.g. Apartment → Photo):** same per-row **`<turbo-frame>`** + Turbo presentation as top-level; removed **`data-turbo="false"`**, which broke nested **cancel** and other in-frame GETs. **`row_html_turbo_allowed?`** / **`nested_associated_list_row_update?`** serve **`format.html`** row **show** / **close** for **`not_accessible_through_html?`** models when **`params[:update]`** is a nested row id (e.g. **`apartment_1_photo_5`**).
+
+### Changed
+
+- **`docs/ujs-to-turbo.md`:** Step 2 / Step 3 checklist for row + nested Turbo.
+- **`example_app_apartment_photos_pagination_test.rb`:** asserts nested turbo-frame rows and Photo row open/close + field cancel; drops the obsolete requirement that nested rows opt out of Turbo.
+
+## [7.4.1] - 2026-05-15
+
+### Added
+
+- **Turbo field edit (Step 3, partial):** scalar fields in stock **`_show.html.erb`** are wrapped in **`<turbo-frame id="{model}_{id}_{attribute}">`**. Field **edit**, **update**, and **cancel** respond with **`format.html`** via **`field_edit.html.erb`** / **`field_show.html.erb`** and the **`turbo_rails/frame`** layout (no UJS on field links/forms). Row-level show/close remains UJS.
+- **`inline_forms_field_cancel_link`** helper and **`inline_forms_field_show`** helper; **`link_to_inline_edit`** accepts **`turbo_frame:`** and omits **`remote: true`** on the Turbo path.
+- **Regression tests** **`example_app_apartment_field_turbo_test.rb`** (stock panel field turbo-frame edit/update/cancel) and extended **`example_app_apartment_name_list_test.rb`** (turbo-frame contract, no **`UnknownFormat`** on cancel).
+
+### Fixed
+
+- **Field cancel on Turbo path:** cancel no longer triggers full-page navigation or **`UnknownFormat`** — **`format.html`** is always registered for single-attribute show; cancel uses a plain GET link with **`data-turbo-frame="_self"`** (no **`data-method`**, which conflicted with jQuery UJS).
+- **Cancel button height:** restored **`input[type=button]`** inside a thin wrapper link (Turbo/UJS attrs on the **`<a>`**) so cancel matches the **ok** submit height; Foundation **`a.button`** was rendering much taller in collapse rows.
+
+### Changed
+
+- **Example name list (`--example`):** uses the same turbo-field contract as stock **`_show`** ( **`@inline_forms_turbo_field`**, **`<turbo-frame>`** wrappers), not a separate UJS path.
+- **`docs/ujs-to-turbo.md`:** Step 3 field-level checklist items marked done for stock scalar fields and name-list regression.
+
+## [7.4.0] - 2026-05-15
+
+### Added
+
+- **`--example` demo: field-level inline edit without the stock `_show` UI** (`ApartmentsController#name_list`, **`GET /apartments/name_list`**). Lists the first 10 apartments with each **`name`** rendered via **`text_field_show`** inside a wrapper `id="apartment_<id>_name"`, so edit/update use the normal polymorphic paths without opening the full inline-edit panel. Installer copies **`app/views/apartments/name_list.html.erb`**, injects the controller action (with CanCan **`skip_load_and_authorize_resource`** / **`authorize! :read, Apartment`**), and adds the route.
+- **More menu link** in the example app only: installer overrides **`app/views/inline_forms/_header.html.erb`** with an extra item **“Apartment names (first 10)”** pointing at **`apartment_name_list_path`**.
+- **UJS → Turbo migration checklist** at **`docs/ujs-to-turbo.md`** (Steps 1–2 done, 3–5 tracked).
+- **Regression test** **`test/integration/example_app_apartment_name_list_test.rb`**: page render, More menu link, and UJS edit-link **`update=`** contract.
+
+## [7.3.4] - 2026-05-15
+
+### Changed
+
+- **Installer Gemfile**: `devise-i18n` is taken from RubyGems (`~> 1.16`, current release **1.16.0**) instead of the obsolete `https://github.com/acesuares/devise-i18n.git` fork (which matched upstream only through 2018). `devise` is pinned to **`~> 5.0`** so it satisfies `devise-i18n` 1.16’s runtime dependency on Devise 5+.
+
+## [7.3.3] - 2026-05-07
+
+### Fixed
+
+- **Session flash no longer appears inside inline-edit fields**: `_edit.html.erb` was injected into the field span via UJS and iterated the full `flash` hash, so an unconsumed Devise notice (for example after sign-in) could show up inside the Trix/rich-text editor when opening a field. The edit partial no longer renders global flash; `_new.html.erb` only shows `flash.now` keys used by `create` (`header`, `error`, `success`). **`layouts/inline_forms.html.erb`** now renders **`flash["notice"]` / `flash["alert"]`** under the header so redirect flash is shown once and consumed on normal pages.
+- **`InlineFormsApplicationController` default layout**: `layout 'devise' if :devise_controller?` was ineffective because `:devise_controller?` is a Symbol (always truthy), so every controller defaulted to `layouts/devise`. Replaced with `layout ->(c) { c.devise_controller? ? "devise" : "inline_forms" }` so app controllers use the full **`inline_forms`** chrome unless an action overrides `render` layout.
+- **Devise sign-in form** (`app/views/devise/sessions/_form.html.erb`): added `data-turbo="false"` on the form so a Turbo-enabled asset bundle cannot turn the POST into a non-navigational request (which would skip Devise’s `set_flash_message!` for `:signed_in`).
+
+## [7.3.2] - 2026-05-07
+
+### Fixed
+
+- **Legacy `:text_area` alias no longer triggers plain-text column checks**: `:text_area` is treated as the rich-text alias path, while plain-text column enforcement remains limited to `:plain_text`, `:plain_text_area`, and `:text_area_without_ckeditor`. This prevents false `InlineForms::PlainTextColumnMissingError` when switching an ActionText-backed attribute from `:rich_text` to `:text_area`.
+
+## [7.3.1] - 2026-05-07
+
+### Changed
+
+- **Long text form element naming is now explicit**: `:plain_text` is the canonical non-WYSIWYG textarea form element (backed by a DB `text` column), while `:rich_text` remains the ActionText/Trix element.
+- **Default mapping for migration type `:text` now emits form element `:plain_text`** (instead of `:text_area`) so newly generated models use the explicit name.
+- **Legacy aliases remain supported**: `:text_area_without_ckeditor` and `:plain_text_area` delegate to `:plain_text`; legacy `:text_area` and `:ckeditor` delegate to `:rich_text`.
+
+### Fixed
+
+- **Misconfigured `plain_text` attributes now fail fast with a clear error**: inline_forms now raises `InlineForms::PlainTextColumnMissingError` when a `plain_text`-style form element targets an attribute without a DB column (for example, an ActionText-only attribute such as `description` after switching from `:rich_text` to `:text_area`/`:plain_text` without adding a column).
+- **Checks run both during app boot/reload (for loaded models) and at request runtime** (`getKlass`, `create`, `update`) to prevent late `ActiveModel::MissingAttributeError` failures.
+- **Generated example-app tests now cover rich_text/plain_text edge cases**, including the failure path above and the reverse direction (`plain_text` -> `rich_text`) staying non-failing from inline_forms' perspective.
+
+## [7.3.0] - 2026-05-07
+
+### Removed
+
+- **CKEditor**: no CDN script tags in engine layouts, no `cktext_area_tag` / `CKEDITOR` usage, no `.ckeditor_area` styles, and no `inline_forms/ckeditor/config.js` asset precompile entry. Long text uses the same plain `<textarea>` path as before when the CKEditor gem was absent.
+
+### Changed
+
+- **`:text_area`** always renders a plain multiline field (equivalent to the old non-CKEditor path and to **`:text_area_without_ckeditor`**).
+- **`:ckeditor`** remains a valid generator/model type name but now delegates to **`:text_area`** behavior (plain text); migrate to **`:text_area`** or **`:rich_text`** when convenient.
+
+## [7.2.11] - 2026-05-07
+
+### Changed
+
+- **Release status notice added to user-facing docs and gem metadata**: `README.rdoc` now includes a top-level notice that versions after `6.2.14` are currently broken and that a follow-up notice will be posted when the gem is good again. `inline_forms.gemspec` now mirrors this status in both `summary` and `description` so the same warning is visible on RubyGems.
+
+### Fixed
+
+- **Generated apps now always source `inline_forms` from the exact generator path** (`gem 'inline_forms', path: generator_repo`) so local unreleased builds are used consistently during `inline_forms create`. To support that safely outside git checkouts, `inline_forms.gemspec` now falls back to globbed file lists when `.git` is absent (instead of shelling out to `git ls-files` unconditionally), while keeping the warning cleanup (`required_ruby_version`, bounded runtime deps, deprecated `rubyforge_project` removal, and executables limited to `inline_forms`).
+- **Dart Sass jQuery UI imports are now deterministic**: vendored jQuery UI SCSS import chains now target explicit `*.css.scss` files (`jquery.ui.all.css.scss` -> `jquery.ui.base.css.scss`/`jquery.ui.theme.css.scss`, plus explicit component/theme-base imports), and `inline_forms.scss` imports the vendored bundle directly. This fixes the case where `dartsass:build` exited 0 but wrote an error CSS payload (`Can't find stylesheet to import`) into `app/assets/builds/inline_forms/inline_forms.css`, leaving logged-in pages effectively unstyled.
+
 ## [7.2.10] - 2026-05-06
 
 ### Changed

data/README.rdoc

@@ -2,6 +2,12 @@
 
 Inline Forms is almost a complete admin application. You can try it out easily.
 
+= Release status notice
+
+All gem versions after +6.2.14+ are currently broken.
+
+We will post a notice when the gem is good again.
+
 = Usage
 
  gem install inline_forms
@@ -24,9 +30,9 @@ You can install the example application manually if you like:
 
  inline_forms create MyApp
  cd MyApp
- rails g inline_forms Picture name:string caption:string image:image_field description:text apartment:belongs_to _presentation:'#{name}'
+ rails g inline_forms Picture name:string caption:string image:image_field description:plain_text apartment:belongs_to _presentation:'#{name}'
  rails generate uploader Image
- rails g inline_forms Apartment name:string title:string description:text pictures:has_many pictures:associated _enabled:yes _presentation:'#{name}'
+ rails g inline_forms Apartment name:string title:string description:rich_text pictures:has_many pictures:associated _enabled:yes _presentation:'#{name}'
  bundle exec rake db:migrate
  rails s
 
@@ -42,6 +48,8 @@ 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.
 
+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.
 
 = Build a vagrant virtualbox box for easier development

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

@@ -37,3 +37,26 @@ $(function(){ $(document).foundation(); });
       $(this).attr('title', '');
     });
   });
+
+// Re-bind jQuery UI widgets and Trix after Turbo Frame swaps (Step 3).
+document.addEventListener("turbo:frame-load", function(event) {
+  var root = event.target;
+  if (!root || !root.querySelectorAll) { return; }
+
+  $(root).find("input.datepicker").each(function() {
+    var $el = $(this);
+    if (!$el.hasClass("hasDatepicker")) { $el.datepicker(); }
+  });
+
+  $(root).find("input.timepicker").each(function() {
+    var $el = $(this);
+    if (!$el.data("timepicker")) { $el.timepicker(); }
+  });
+
+  $(root).find("trix-editor").each(function() {
+    if (window.Trix && this.editor) { return; }
+    if (window.Trix && typeof Trix.Editor === "function") {
+      new Trix.Editor(this);
+    }
+  });
+});

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

@@ -1,8 +1,8 @@
 // inline_forms
 @import url(http://fonts.googleapis.com/css?family=Open+Sans:400,400italic);
 @import 'inline_forms/foundation_icons';
-@import 'themes/jquery.ui.sunny';
-@import 'jquery.ui.all';
+@import '../jquery_ui/themes/jquery.ui.sunny.css.scss';
+@import '../jquery_ui/jquery.ui.all.css.scss';
 @import 'foundation_and_overrides';
 
 // Foundation 6 makes `.row` a flex container (`display: flex`); a nested
@@ -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/app/assets/stylesheets/jquery_ui/jquery.ui.all.css.scss

@@ -9,6 +9,6 @@
  * http://docs.jquery.com/UI/Theming
  */
 
-@import 'jquery.ui.base';
-@import 'jquery.ui.theme';
+@import 'jquery.ui.base.css.scss';
+@import 'jquery.ui.theme.css.scss';
 

data/app/assets/stylesheets/jquery_ui/jquery.ui.base.css.scss

@@ -9,19 +9,19 @@
  * http://docs.jquery.com/UI/Theming
  */
 
-@import 'jquery.ui.core';
+@import 'jquery.ui.core.css.scss';
 
-@import 'jquery.ui.accordion';
-@import 'jquery.ui.autocomplete';
-@import 'jquery.ui.button';
-@import 'jquery.ui.datepicker';
-@import 'jquery.ui.dialog';
-@import 'jquery.ui.menu';
-@import 'jquery.ui.progressbar';
-@import 'jquery.ui.resizable';
-@import 'jquery.ui.selectable';
-@import 'jquery.ui.slider';
-@import 'jquery.ui.spinner';
-@import 'jquery.ui.tabs';
-@import 'jquery.ui.tooltip';
+@import 'jquery.ui.accordion.css.scss';
+@import 'jquery.ui.autocomplete.css.scss';
+@import 'jquery.ui.button.css.scss';
+@import 'jquery.ui.datepicker.css.scss';
+@import 'jquery.ui.dialog.css.scss';
+@import 'jquery.ui.menu.css.scss';
+@import 'jquery.ui.progressbar.css.scss';
+@import 'jquery.ui.resizable.css.scss';
+@import 'jquery.ui.selectable.css.scss';
+@import 'jquery.ui.slider.css.scss';
+@import 'jquery.ui.spinner.css.scss';
+@import 'jquery.ui.tabs.css.scss';
+@import 'jquery.ui.tooltip.css.scss';
 

data/app/assets/stylesheets/jquery_ui/jquery.ui.theme.css.scss

@@ -10,7 +10,7 @@
  *
  * To view and modify this theme, visit http://jqueryui.com/themeroller/
  */
-@import 'themes/jquery.ui.base';
+@import 'themes/jquery.ui.base.css.scss';
 
 
 

data/app/controllers/concerns/versions_concern.rb

@@ -11,11 +11,12 @@ module VersionsConcern
     close = params[:close] || false
     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 { } unless @Klass.not_accessible_through_html?
+        format.html { render "inline_forms/versions_list_panel", layout: "turbo_rails/frame" }
         format.js { render :versions_list }
       end
     end

data/app/controllers/inline_forms_application_controller.rb

@@ -1,7 +1,11 @@
 # -*- encoding : utf-8 -*-
 class InlineFormsApplicationController < ActionController::Base
   protect_from_forgery
-  layout 'devise' if :devise_controller?
+  # `layout 'devise' if :devise_controller?` was wrong: `:devise_controller?` is a
+  # Symbol (always truthy), so every controller used the Devise layout. Use a
+  # callable so only Devise controllers get `layouts/devise`; everything else
+  # defaults to `layouts/inline_forms` (actions may still override via `render`).
+  layout ->(controller) { controller.devise_controller? ? "devise" : "inline_forms" }
 
   # limit available locales by setting this. Override in applicaton_controller.
   I18n.available_locales = [ :en, :nl, :pp ]