recourse 4.4.3 → 4.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ba7f8f62f4a4488b30a0ccc90cc550d98e56dc6a717380b368d5f2a51e06f0c6
-  data.tar.gz: 6311ab19960ad5bc92151cfde18dfa4a8610c067ecb08c00021340d19b7e9a2c
+  metadata.gz: 10cc1a031240c0430a970cea2dc0c79460e3b6dfe131ce6e8a854aed364a485c
+  data.tar.gz: 631f40d0bc652b90cdaf960864ea5c47f7baa55ea1808b4a366a3d26ff021220
 SHA512:
-  metadata.gz: 34d609a74f655281b6e4438571a00d4fc554fd431f4e07d3058ba8b84529f800a0e4c798f709ec806989ed9519da2045301658be5314fb0a09317cdcc213ffc9
-  data.tar.gz: 142f6ec366b5e45fcffc6afb92ebcf8cbe84ce6e4918e7d4ee50b33144048263fb35f9b519655ea16beb4816e679ec2e4a1311c286881bf7477fcda8a70fffe7
+  metadata.gz: 43892125e4332125787621108a81ccea1de70de89c66c6cdf686d68f54d7d8eda0c7251e964e2a74df8c5a9492ec8d26b8cf99c5443fa246bc91aebe0de8b13d
+  data.tar.gz: be7fa5f7c3853ba0ccecb21f5b290ab0ac38f00f1165d541ab670ff35decd71f9df154cc09836ecc7432153d3035dad9ccd290e449cba08d36976fbd579c9965

data/CHANGELOG.md

@@ -5,6 +5,28 @@ All notable changes to this project will be documented in this file.
 For more information about changelogs, check [Keep a Changelog](http://keepachangelog.com) and
 [Vandamme](http://tech-angels.github.io/vandamme).
 
+## 4.5.0 - 2026-09-08
+
+* [Fix] A row with a bookmark square is as tall as a row without one: the square no longer claims a button's minimum height
+
+* [Feature] The arrow after a web address opens it in a new tab, while the words still open it in this one
+
+* [Fix] A production rake task boots without Rails 8.2 warning that Action Controller and Active Record were loaded early
+
+  `recourses` defined each missing controller as the routes drew, which loaded the host's
+  `RecoursesController` and everything above it, and asked a nested name for its model the
+  same way. A boot that will not eager load — a `db:migrate` on release — draws its routes
+  before it is done, and Rails 8.2 logs a warning for every component loaded then. The
+  controller now waits for the app's first request or job, and the model is looked up as a
+  constant, without loading it.
+
+## 4.4.4 - 2026-09-08
+
+* [Fix] On a phone the search form keeps its place beside the breadcrumb once a filter is picked
+
+  A form sized by its content widened with the words a picked option put on a toggle
+  and wrapped under the crumbs.
+
 ## 4.4.3 - 2026-09-08
 
 * [Fix] A counted tab reads `1 franchise` again, without the extra space 4.4.0 put between the figure and the word

data/README.md

@@ -33,16 +33,16 @@ end
 Every recourse with `index` gets a sidebar entry (in routes order), a keyboard shortcut on its
 first letter, and a paginated, searchable, sortable, filterable index table:
 
-[image]
+<img width="3824" height="2274" alt="Image" src="https://github.com/user-attachments/assets/f952d3e5-c320-464b-b166-77c7ab8f9071" />
 
 Every recourse with `show` gets a detail page where attributes are displayed with the appropriate
 formatting, or masked if sensitive:
 
-[image]
+<img width="3824" height="2274" alt="Image" src="https://github.com/user-attachments/assets/515f63c4-8a9b-422e-95c4-88d6647911d8" />
 
 Every recourse with `new` or `edit` gets a form with appropriate browser formatting and validation:
 
-[image]
+<img width="3824" height="1220" alt="Image" src="https://github.com/user-attachments/assets/ebe5108b-bbd1-4cd1-a635-89eef0e6fc50" />
 
 Every recourse with `destroy` gets a button with a detailed confirmation message:
 
@@ -50,7 +50,7 @@ Every recourse with `destroy` gets a button with a detailed confirmation message
 
 Every nested recourse gets namespaced after the parent:
 
-[image]
+<img width="3824" height="2274" alt="Image" src="https://github.com/user-attachments/assets/9b242335-a25b-4fa8-8023-422538d235b0" />
 
 
 ## Step 2. Enhance your models
@@ -134,6 +134,9 @@ Recourse.color = :orange
 Recourse.bookmarks = -> { Keepsake.where agent: Current.agent }
 ```
 
+<img width="3824" height="1220" alt="Image" src="https://github.com/user-attachments/assets/d1db4adb-0b5c-4e41-8bb6-cf72a35288f0" />
+
+
 ## Development
 
 ```bash

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

@@ -265,12 +265,19 @@
          next to one, so it takes the line's height and not `.btn`'s own minimum —
          which would leave the row under every table taller than the words in it. */
       .recourse-limit { min-height: 0; }
+      /* The bookmark square is an icon in a row of text, not a control in a form, so it
+         takes the line's height too: `.btn`'s minimum made every row with one taller
+         than the rows without. */
+      .recourse-bookmark { min-height: 0; }
       /* Stacked, each control is a row of its own rather than a fragment of one:
          there is no second control beside it to share the line with. */
       @media (width < 768px) {
-        .recourse-search { max-width: none; }
+        /* The room the breadcrumb leaves, and never more: a form sized by its content
+           widened with the words a picked option put on a toggle, and wrapped under
+           the crumbs. A basis of nothing and leave to shrink hold it in its row. */
+        .recourse-search { flex: 1 1 0; min-width: 0; max-width: none; }
         .recourse-search-box,
-        .recourse-search .combobox-toggle { flex: 1 1 100%; }
+        .recourse-search .combobox-toggle { flex: 1 1 100%; min-width: 0; }
         /* Safari zooms in on a field whose text is under 16px, an absolute figure: the
            small controls are 14px, and so is the body on a phone, so the controls say
            16px themselves and focusing one moves nothing. */

data/config/locales/recourse.en.yml

@@ -45,6 +45,7 @@ en:
     # What a menu with several picks reads as, the first named and the rest counted.
     more: "%{first} + %{count} more"
     new: New %{model}
+    new_tab: Open in a new tab
     no_results: No results found
     none: "No %{models}."
     optional: Optional

data/lib/recourse/controllers.rb

@@ -1,16 +1,43 @@
 module Recourse
   # Defines the controller classes a host app has not written for itself.
   module Controllers
+    @held = []
+    @lock = Mutex.new
+
     # Creates a controller for the named resource unless the host app has one. The
     # name arrives with whatever namespace it was drawn in — `admin/contacts` — so
-    # the constant lands where Rails will look for it.
-    def self.define_missing(path, base = nil)
+    # the constant lands where Rails will look for it. A block names the superclass,
+    # `RecoursesController` by default, and is called only once the class is made.
+    def self.define_missing(path, &base)
       class_name = "#{path.camelize}Controller"
       # const_defined? is true for a Zeitwerk autoload, so files on disk count too.
       return if Object.const_defined? class_name
 
-      namespace(class_name.deconstantize).const_set class_name.demodulize,
-                                                    Class.new(base || RecoursesController)
+      booting? ? hold(class_name, base) : define(class_name, base)
+    end
+
+    # Makes the controllers held back while the app booted. The executor runs this
+    # ahead of the app's first request or job, once every class may be loaded.
+    def self.define_held
+      @lock.synchronize { define(*@held.shift) until @held.empty? }
+    end
+
+    private_class_method def self.define(class_name, base)
+      superclass = base ? base.call : RecoursesController
+      namespace(class_name.deconstantize).const_set class_name.demodulize, Class.new(superclass)
+    end
+
+    # Rails 8.2 warns when Action Controller is loaded during the boot of an app that
+    # will not eager load — a rake task's, which draws its routes before it is done
+    # and never dispatches to a controller — so the class waits for the first run.
+    private_class_method def self.booting?
+      app = Rails.application
+      app.present? && !app.config.eager_load && !app.initialized?
+    end
+
+    private_class_method def self.hold(class_name, base)
+      Rails.application.executor.to_run { Controllers.define_held } if @held.empty?
+      @held << [class_name, base]
     end
 
     # The module a namespaced controller belongs in, made where the host has none of

data/lib/recourse/helpers/bookmarks.rb

@@ -50,13 +50,15 @@ module Recourse
 
       # Kept or not, one square either way: the same path with the verb reversed, so
       # the button toggles by flipping the method Rails already wrote into the form.
+      # `recourse-bookmark` takes back the height `.btn` claims, so the row stays as
+      # tall as one with no square in it.
       def bookmark_button(record)
         kept = bookmarked_ids.include? record.id
         label = t "recourse.#{kept ? 'unbookmark' : 'bookmark'}"
 
         button_to bookmark_icon(kept, label), bookmark_url(record),
                   method: kept ? :delete : :post, form_class: 'd-inline-block',
-                  class: 'btn btn-sm btn-link p-0 border-0 lh-1',
+                  class: 'btn btn-sm btn-link p-0 border-0 lh-1 recourse-bookmark',
                   **bookmark_data(kept, label)
       end
 

data/lib/recourse/helpers/formats.rb

@@ -79,15 +79,21 @@ module Recourse
         value.is_a?(String) && value.match?(WEB_URL)
       end
 
-      # Bootstrap's icon link, saying the value leads somewhere the way text cannot.
-      # What it reads is the host, and an ellipsis where the address goes further --
-      # a path is how a machine finds the page, and the href is already carrying it.
+      # The words lead where the value points, in this tab, the way any link does; the
+      # arrow after them — Bootstrap's icon link, stepping under the cursor — opens the
+      # same address in a new tab. Either reads as the host, and an ellipsis where the
+      # address goes further: a path is how a machine finds the page, and the href has it.
       def url_link(value)
         host, rest = value.match(WEB_URL).captures
         said = rest.delete_suffix('/').empty? ? host : "#{host}/…"
 
-        tag.a safe_join([said, icon_tag(:point_right)], ' '),
-              href: value, class: 'icon-link icon-link-hover'
+        safe_join [tag.a(said, href: value), new_tab_arrow(value)], ' '
+      end
+
+      def new_tab_arrow(value)
+        tag.a icon_tag(:point_right), href: value, target: '_blank', rel: 'noopener',
+                                      class: 'icon-link icon-link-hover',
+                                      aria: { label: t('recourse.new_tab') }
       end
     end
   end

data/lib/recourse/routes/nested.rb

@@ -18,8 +18,9 @@ module Recourse
       # Whether this resource has rows to address one at a time. A bookmark names one
       # row, which means nothing for a name with no class behind it at all — and one was
       # drawn anyway, at `/placeholders/:placeholder_id/bookmark`, where nothing linked
-      # to it and anything reaching it raised.
-      def addressable_rows? = Recourse.model?(parent_resource.name).present?
+      # to it and anything reaching it raised. Asked of the constant rather than the
+      # class: loading a model while the routes draw is what Rails 8.2 warns about.
+      def addressable_rows? = Object.const_defined? Recourse.model_name(parent_resource.name)
 
       # The row a table's bookmark square writes: one record kept by whoever is looking,
       # at `/places/5/bookmark`. Deliberately not recorded through `Recourse.nest` — it
@@ -28,7 +29,7 @@ module Recourse
       # since the gem drew it: one segment under the resource, always.
       def draw_bookmark
         path = [current_module, 'bookmarks'].compact.join '/'
-        Controllers.define_missing path, ::BookmarksController
+        Controllers.define_missing(path) { ::BookmarksController }
         resource :bookmark, only: %i[create destroy]
       end
     end

data/lib/recourse/version.rb

@@ -1,4 +1,4 @@
 module Recourse
   # Version of the gem, read by the gemspec and by hosts checking compatibility.
-  VERSION = '4.4.3'
+  VERSION = '4.5.0'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: recourse
 version: !ruby/object:Gem::Version
-  version: 4.4.3
+  version: 4.5.0
 platform: ruby
 authors:
 - claudiob