infotable 1.4.0 → 1.5.0

data/app/controllers/infotable/tables_controller.rb

@@ -27,6 +27,10 @@ module Infotable
         format.turbo_stream {
           # For infinite scroll - return turbo stream to append rows and replace pagination frame
           # Following the guide's "Frames + Streams" pattern
+          if @page_out_of_range # nothing left: appending the first page again would repeat its rows
+            return render turbo_stream: turbo_stream.replace("#{params[:table_name]}_pagination", partial: "infotable/tables/end_indicator")
+          end
+
           streams = [
             turbo_stream.append(
               "#{params[:table_name]}_tbody",
@@ -48,7 +52,7 @@ module Infotable
             # No more pages - replace with end indicator
             streams << turbo_stream.replace(
               "#{params[:table_name]}_pagination",
-              '<div class="infotable-end-indicator border-t border-gray-100 bg-white px-6 py-8 text-center"><div class="inline-flex items-center gap-2 text-sm text-gray-500"><svg class="h-4 w-4" fill="none" stroke="currentColor" viewBox="0 0 24 24"><path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M5 13l4 4L19 7"></path></svg><span class="font-medium">All items loaded</span></div></div>'.html_safe
+              partial: "infotable/tables/end_indicator"
             )
           end
 
@@ -139,6 +143,7 @@ module Infotable
       # Apply pagination
       @pagy, @records = paginate(@page)
     rescue Pagy::RangeError # page past the end: show the first page
+      @page_out_of_range = true
       @pagy, @records = paginate(1)
     end
 
@@ -207,7 +212,7 @@ module Infotable
     def add_sort_tiebreaker(query, sort_columns)
       model = query.model
       pk = model.primary_key
-      return query if pk.blank? || sort_columns.map(&:to_s).include?(pk) || query.distinct_value
+      return query if pk.blank? || sort_columns.map(&:to_s).include?(pk) || query.distinct_value || query.group_values.any?
 
       query.order(Arel.sql("#{model.table_name}.#{pk} ASC"))
     end
@@ -228,9 +233,15 @@ module Infotable
       requested.positive? ? [ requested, MAX_PER_PAGE ].min : default
     end
 
-    # Guard against spreadsheet formula injection
+    PLAIN_NUMBER = /\A[+-]?\d+([.,]\d+)?\z/
+
+    # Guard against spreadsheet formula injection. Plain numbers ("-5", "+48123456789") are
+    # not formulas and are left alone.
     def csv_safe(value)
-      value.is_a?(String) && value.match?(/\A[=+\-@\t\r]/) ? "'#{value}" : value
+      return value unless value.is_a?(String) && value.match?(/\A[=+\-@\t\r]/)
+      return value if value.match?(PLAIN_NUMBER)
+
+      "'#{value}"
     end
 
     def render_table_frame

data/app/helpers/infotable/tables_helper.rb

@@ -28,6 +28,17 @@ module Infotable
       render partial: "infotable/tables/table", locals: locals
     end
 
+    # Inline width style for a header/body cell. Data columns are clipped to their width (ellipsis);
+    # action columns hold buttons, so their width is only a minimum and the column grows to fit all
+    # of them (a clipped "Delete" button is worse than a wider column).
+    def infotable_cell_width_style(column, ellipsis: false)
+      if %w[action actions].include?(column[:type].to_s)
+        "width: 1%; min-width: #{column[:width]}; white-space: nowrap;"
+      else
+        "width: #{column[:width]}; max-width: #{column[:width]}; overflow: hidden;#{' text-overflow: ellipsis;' if ellipsis}"
+      end
+    end
+
     # Format cell value based on column definition
     def format_cell_value(record, column, table_class = nil)
       value = (table_class || Infotable::Base).cell_value(record, column)
@@ -78,29 +89,6 @@ module Infotable
       end
     end
 
-    # Build export params with current filters, sort, and visible columns
-    def build_export_params(table_name, table_class)
-      params_hash = {
-        table_name: table_name,
-        sort_column: scalar_request_param(:sort_column),
-        sort_direction: scalar_request_param(:sort_direction),
-        sort: scalar_request_param(:sort)
-      }
-
-      # Add filters if present
-      if params[:filters].present?
-        params_hash[:filters] = params[:filters]
-      end
-
-      # Get visible columns from localStorage via JavaScript or use defaults
-      # For now, we'll pass all visible columns and let the controller handle it
-      # The JavaScript will need to update the link dynamically
-      visible_column_names = table_class.visible_columns.map { |c| c[:name].to_s }
-      params_hash[:visible_columns] = visible_column_names.join(",")
-
-      params_hash.compact
-    end
-
     private
 
     def resolve_table_class(table_name)

data/app/views/infotable/tables/_column_visibility.html.erb

@@ -50,8 +50,8 @@
                     data-pin-direction="left"
                     title="<%= t('infotable.columns.pin.left') %>"
                     class="pin-btn pin-left w-7 h-7 rounded flex items-center justify-center <%= column[:pinned] == :left ? 'bg-indigo-100 dark:bg-indigo-900/40 text-indigo-600 dark:text-indigo-400' : 'text-gray-400 hover:bg-gray-100 dark:hover:bg-gray-700 hover:text-gray-600 dark:hover:text-gray-300' %> transition-colors">
-              <svg class="w-4 h-4" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-                <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M15 19l-7-7 7-7"></path>
+              <svg class="w-4 h-4 -rotate-45" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" viewBox="0 0 24 24">
+                <path d="M12 17v5"/><path d="M9 10.76a2 2 0 0 1-1.11 1.79l-1.78.9A2 2 0 0 0 5 15.24V16a1 1 0 0 0 1 1h12a1 1 0 0 0 1-1v-.76a2 2 0 0 0-1.11-1.79l-1.78-.9A2 2 0 0 1 15 10.76V7a1 1 0 0 1 1-1 2 2 0 0 0 0-4H8a2 2 0 0 0 0 4 1 1 0 0 1 1 1z"/>
               </svg>
             </button>
             <%# Pin Right %>
@@ -61,8 +61,8 @@
                     data-pin-direction="right"
                     title="<%= t('infotable.columns.pin.right') %>"
                     class="pin-btn pin-right w-7 h-7 rounded flex items-center justify-center <%= column[:pinned] == :right ? 'bg-indigo-100 dark:bg-indigo-900/40 text-indigo-600 dark:text-indigo-400' : 'text-gray-400 hover:bg-gray-100 dark:hover:bg-gray-700 hover:text-gray-600 dark:hover:text-gray-300' %> transition-colors">
-              <svg class="w-4 h-4" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-                <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M9 5l7 7-7 7"></path>
+              <svg class="w-4 h-4 rotate-45" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" viewBox="0 0 24 24">
+                <path d="M12 17v5"/><path d="M9 10.76a2 2 0 0 1-1.11 1.79l-1.78.9A2 2 0 0 0 5 15.24V16a1 1 0 0 0 1 1h12a1 1 0 0 0 1-1v-.76a2 2 0 0 0-1.11-1.79l-1.78-.9A2 2 0 0 1 15 10.76V7a1 1 0 0 1 1-1 2 2 0 0 0 0-4H8a2 2 0 0 0 0 4 1 1 0 0 1 1 1z"/>
               </svg>
             </button>
           </div>

data/app/views/infotable/tables/_end_indicator.html.erb

@@ -0,0 +1,8 @@
+<div class="border-t border-gray-100 dark:border-gray-800 bg-gradient-to-b from-gray-50/60 to-white/60 px-6 py-8 text-center">
+  <div class="inline-flex items-center gap-2 text-sm text-gray-500 dark:text-gray-400">
+    <svg class="h-4 w-4" fill="none" stroke="currentColor" viewBox="0 0 24 24">
+      <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M5 13l4 4L19 7"></path>
+    </svg>
+    <span class="font-medium"><%= t('infotable.table.loading.all_loaded') %></span>
+  </div>
+</div>

data/app/views/infotable/tables/_next_page_frame.html.erb

@@ -27,13 +27,5 @@
     </div>
   <% end %>
 <% else %>
-  <%# No more pages - show end indicator %>
-  <div class="border-t border-gray-100 dark:border-gray-800 bg-gradient-to-b from-gray-50/60 to-white/60 px-6 py-8 text-center">
-    <div class="inline-flex items-center gap-2 text-sm text-gray-500 dark:text-gray-400">
-      <svg class="h-4 w-4" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-        <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M5 13l4 4L19 7"></path>
-      </svg>
-      <span class="font-medium"><%= t('infotable.table.loading.all_loaded') %></span>
-    </div>
-  </div>
+  <%= render partial: "infotable/tables/end_indicator" %>
 <% end %>

data/app/views/infotable/tables/_table_rows.html.erb

@@ -14,8 +14,15 @@
       else "text-sm" # default
     end
   %>
-  <tr class="infotable-row relative border-t border-gray-50 dark:border-gray-800 first:border-t-0 hover:shadow-sm hover:translate-x-px transition-all duration-200 ease-out <%= 'cursor-pointer' if row_link_enabled %>"
-      <%= tag.attributes(data: { row_link: row_url, action: "click->infotable--table#handleRowClick" }) if row_link_enabled && row_url %>>
+  <%
+    clickable = table_class.row_clickable?
+    row_dom_id = dom_id(record) if record.respond_to?(:to_key) && record.to_key.present?
+    row_data = table_class.row_data_for(record).transform_keys { |key| key.to_s.dasherize }
+    row_data["row-link"] = row_url if row_url
+    row_data["action"] = "click->infotable--table#handleRowClick" if clickable
+  %>
+  <tr <%= tag.attributes(id: row_dom_id, data: row_data) %>
+      class="infotable-row group relative border-t border-gray-100 dark:border-gray-800 first:border-t-0 hover:bg-gray-50 dark:hover:bg-gray-800 transition-colors duration-150 <%= 'cursor-pointer' if clickable %> <%= table_class.row_classes_for(record) %>">
     <% if table_class.checkbox_enabled? %>
       <td class="<%= cell_padding %> w-10">
         <input type="checkbox"
@@ -29,7 +36,7 @@
     <% table_class.columns.each do |column| %>
       <td data-column-name="<%= column[:name] %>"
           data-default-visible="<%= column[:visible] %>"
-          style="width: <%= column[:width] %>; max-width: <%= column[:width] %>; overflow: hidden; text-overflow: ellipsis;"
+          style="<%= infotable_cell_width_style(column, ellipsis: true) %>"
           class="<%= cell_padding %> whitespace-nowrap <%= text_size %> text-gray-700 dark:text-gray-300">
         <%= format_cell_value(record, column, table_class) %>
       </td>

data/app/views/infotable/tables/show.html.erb

@@ -34,8 +34,8 @@
                     data-column-name="<%= column[:name] %>"
                     data-default-visible="<%= column[:visible] %>"
                     <%= tag.attributes(data: { default_pinned: column[:pinned] }) if column[:pinned] %>
-                    style="width: <%= column[:width] %>; max-width: <%= column[:width] %>; overflow: hidden;"
-                    class="<%= header_padding %> text-left text-xs font-semibold text-gray-900 dark:text-gray-100 tracking-wider select-none bg-gray-50 dark:bg-gray-800 sticky top-0 z-20">
+                    style="<%= infotable_cell_width_style(column) %>"
+                    class="<%= header_padding %> text-left whitespace-nowrap text-xs font-medium tracking-wide text-gray-500 dark:text-gray-400 select-none bg-gray-50 dark:bg-gray-800 sticky top-0 z-20">
                   <% if column[:sortable] %>
                     <% sort_index = current_sorts.index { |name, _| name == column[:name].to_s } %>
                     <% sort_dir = sort_index ? current_sorts[sort_index].last : nil %>
@@ -90,7 +90,7 @@
         </table>
 
       <%# Next page frame for infinite scroll - rendered after table %>
-      <% if @infinite_scroll_enabled && params[:page].blank? %>
+      <% if @infinite_scroll_enabled %>
         <%= render partial: "infotable/tables/next_page_frame" %>
       <% end %>
     <% else %>

data/config/importmap.rb

@@ -2,5 +2,5 @@
 pin "infotable", to: "infotable/index.js"
 pin_all_from Infotable::Engine.root.join("app/assets/javascripts/infotable/controllers"), under: "controllers/infotable", to: "infotable/controllers"
 
-# External dependencies
-pin "sortablejs", to: "https://cdn.jsdelivr.net/npm/sortablejs@1.15.7/modular/sortable.esm.js"
+# External dependencies, served with the gem (no third-party CDN at runtime)
+pin "sortablejs", to: "infotable/vendor/sortable.esm.js"

data/lib/generators/infotable/install/templates/initializer.rb

@@ -14,9 +14,11 @@ Infotable.configure do |config|
   # Drawer width
   config.drawer_width = "26rem" # px, rem, em, vw or %
 
-  # Controller the engine controllers inherit from. Use your app's controller so
-  # authentication, locale, tenant scoping etc. also protect the tables:
-  # config.parent_controller = "ApplicationController"
+  # SECURITY: tables are served at /infotable/tables/:name/{data,export,ids}. The engine
+  # controllers inherit from this controller, so authentication, locale, tenant scoping etc.
+  # of your app also protect the tables. With the default (ActionController::Base) nothing
+  # does: anyone can read and export every table that has no `authorize { ... }` block.
+  config.parent_controller = "ApplicationController"
 
   # Mirror search, filters, sort and page in the address bar (shareable links, Back button)
   # config.url_state = true

data/lib/infotable/base.rb

@@ -1,7 +1,7 @@
 module Infotable
   class Base
     class << self
-      attr_accessor :table_columns, :table_query, :table_query_sql, :pagination_settings, :sort_settings, :height_settings, :infinite_scroll_settings, :row_link_settings, :slim_rows_settings, :row_size_settings, :filters_button_settings, :display_settings, :nil_value_settings, :empty_value_settings, :table_authorize, :checkbox_settings, :batch_action_list, :countless_setting, :url_state_setting, :resizable_columns_setting
+      attr_accessor :table_columns, :table_query, :table_query_sql, :pagination_settings, :sort_settings, :height_settings, :infinite_scroll_settings, :row_link_settings, :row_attr_settings, :slim_rows_settings, :row_size_settings, :filters_button_settings, :display_settings, :nil_value_settings, :empty_value_settings, :table_authorize, :checkbox_settings, :batch_action_list, :countless_setting, :url_state_setting, :resizable_columns_setting
 
       def inherited(subclass)
         super
@@ -19,6 +19,7 @@ module Infotable
         subclass.infinite_scroll_settings = {
           enabled: false
         }
+        subclass.row_attr_settings = { class: nil, data: nil }
         subclass.row_link_settings = {
           enabled: false,
           url: nil
@@ -65,7 +66,9 @@ module Infotable
         subclass.countless_setting = parent.countless_setting
         subclass.url_state_setting = parent.url_state_setting
         subclass.resizable_columns_setting = parent.resizable_columns_setting
-        %i[pagination_settings sort_settings height_settings infinite_scroll_settings row_link_settings
+        subclass.instance_variable_set(:@toolbar_setting, parent.instance_variable_get(:@toolbar_setting))
+        subclass.instance_variable_set(:@export_column_names, parent.instance_variable_get(:@export_column_names)&.dup)
+        %i[pagination_settings sort_settings height_settings infinite_scroll_settings row_link_settings row_attr_settings
            slim_rows_settings row_size_settings filters_button_settings display_settings
            nil_value_settings empty_value_settings].each do |attr|
           subclass.public_send("#{attr}=", parent.public_send(attr).dup)
@@ -134,8 +137,9 @@ module Infotable
       # Plucks when the id method is a plain column of an ungrouped query, loads records otherwise.
       def checkbox_ids(query, limit = nil)
         method = @checkbox_settings[:id_method]
-        scope = query.reorder(nil)
-        scope = scope.limit(limit) if limit
+        # Without a cap the order is irrelevant (and costs a sort); with one, the table's own order
+        # decides which rows make it in, so a capped selection is the first N rows the user sees
+        scope = limit ? query.limit(limit) : query.reorder(nil)
         model = query.model
 
         if model.column_names.include?(method.to_s) && scope.group_values.empty? && scope.having_clause.empty?
@@ -167,8 +171,16 @@ module Infotable
       end
 
       # Define a column
+      COLUMN_OPTIONS = %i[label type sortable searchable filterable visible exportable pinned virtual virtual_sql
+                          sql sort_sql value includes formatter formatter_options actions filter_type filter_options
+                          filter_options_cache filter_options_limit filter_multi_select width].freeze
+
       def column(name, **options)
         @table_columns ||= []
+        unknown = options.keys - COLUMN_OPTIONS
+        if unknown.any? && defined?(Rails) && Rails.logger
+          Rails.logger.warn("[Infotable] #{self.name || 'table'}: column :#{name} ignores unknown option(s) #{unknown.map(&:inspect).join(', ')}")
+        end
         type = options[:type] || :string
 
         # Determine width - use custom if provided and not blank, otherwise use default
@@ -319,6 +331,51 @@ module Infotable
         end
       end
 
+      # Extra CSS classes per row: a String, or a Proc that receives the record (nil/false adds nothing).
+      # Tailwind only generates classes it can see in your source, so write them out in full.
+      # @example
+      #   row_class ->(order) { "bg-red-50" if order.overdue? }
+      def row_class(value = nil, &block)
+        @row_attr_settings[:class] = value || block
+      end
+
+      def row_classes_for(record)
+        setting = @row_attr_settings[:class]
+        (setting.respond_to?(:call) ? setting.call(record) : setting).presence
+      end
+
+      # Extra data-* attributes per row: a Proc that receives the record and returns a Hash
+      # ({ post_id: 1 } becomes data-post-id="1"). Handy for your own Stimulus controllers.
+      # @example
+      #   row_data ->(post) { { post_id: post.id, status: post.status } }
+      def row_data(value = nil, &block)
+        @row_attr_settings[:data] = value || block
+      end
+
+      def row_data_for(record)
+        setting = @row_attr_settings[:data]
+        data = setting.respond_to?(:call) ? setting.call(record) : setting
+        data.is_a?(Hash) ? data : {}
+      end
+
+      # Clicking a row (not a link, button or input inside it) fires a cancelable
+      # `infotable:row-click` event on the row (detail: { id, url, data }). If nothing calls
+      # preventDefault() and the table has a row_link, the click falls back to following it.
+      # @example
+      #   row_click true
+      def row_click(enabled = true)
+        @row_link_settings[:click] = enabled ? true : false
+      end
+
+      def row_click?
+        @row_link_settings[:click] ? true : false
+      end
+
+      # Rows react to clicks (row_link and/or row_click)
+      def row_clickable?
+        row_link_enabled? || row_click?
+      end
+
       # Set row size (padding and text size)
       # @param size [Symbol] - Row size (:slim, :mid, :default)
       # @example

data/lib/infotable/configuration.rb

@@ -31,7 +31,7 @@ module Infotable
       @nil_value = "-" # Default placeholder for nil values
       @empty_value = "-" # Default placeholder for empty strings
       @url_state = true # mirror table state in the address bar
-      @max_select_all = 10_000 # "select all matching" collects at most this many ids (nil = no limit)
+      @max_select_all = 4000 # "select all matching" collects at most this many ids (nil = no limit). Rack accepts 4096 params per request, and every id is sent as ids[]
       @max_filter_options = 500 # select filters render at most this many <option>s (nil = no limit)
       @parent_controller = "ActionController::Base" # Set to "ApplicationController" to inherit host auth/filters
     end

data/lib/infotable/engine.rb

@@ -6,18 +6,11 @@ module Infotable
   class Engine < ::Rails::Engine
     isolate_namespace Infotable
 
-    # Configure engine
-    config.infotable = ActiveSupport::OrderedOptions.new
-
-    # Default configuration values
-    config.infotable.table_min_height = "400px"
-    config.infotable.table_max_height = "600px"
-
     # Rails already adds the engine's app/assets/* directories to the asset path;
     # only the precompile list is needed. Skipped for apps without an asset pipeline.
     initializer "infotable.assets" do |app|
       if app.config.respond_to?(:assets)
-        app.config.assets.precompile += %w[infotable/controllers/*.js]
+        app.config.assets.precompile += %w[infotable/controllers/*.js infotable/vendor/*.js]
       end
     end
 

data/lib/infotable/filter_processor.rb

@@ -110,7 +110,7 @@ module Infotable
       column_name = column[:name]
       is_virtual = having?(column)
       # Use qualified column reference
-      column_ref = qualified_column_ref(column)
+      column_ref = text_ref(column)
 
       result = query
 
@@ -145,6 +145,14 @@ module Infotable
       result
     end
 
+    # PostgreSQL has no LOWER / ILIKE / = '' for numbers, dates, uuids...: a text filter on such a
+    # column (filter_type: :text on an integer) compares its text form.
+    def text_ref(column)
+      ref = qualified_column_ref(column)
+      type = select_column_type(column)
+      type.nil? || %i[string text].include?(type) ? ref : "CAST(#{ref} AS text)"
+    end
+
     # Boolean filter
     def apply_boolean_filter(column, params)
       return query unless params.is_a?(Hash)
@@ -245,6 +253,7 @@ module Infotable
       from = parse_date(scalar(params[:from] || params["from"]))
       to = parse_date(scalar(params[:to] || params["to"]))
       to = to.end_of_day if to && datetime
+      from = from.beginning_of_day if from && datetime # same zone as `to` (Time.zone), not UTC midnight
 
       result = query
       result = result.public_send(filter_method, "#{column_ref} >= ?", from) if from

data/lib/infotable/formatters/boolean_formatter.rb

@@ -10,7 +10,7 @@ module Infotable
         when :badge
           format_as_badge(value, true_text, false_text)
         when :icon
-          format_as_icon(value)
+          format_as_icon(value, true_text, false_text)
         else
           value ? true_text : false_text
         end
@@ -26,11 +26,11 @@ module Infotable
         end
       end
 
-      def format_as_icon(value)
+      def format_as_icon(value, true_text, false_text)
         if value
-          html_safe('<svg class="w-5 h-5 text-green-500" fill="currentColor" viewBox="0 0 20 20"><path fill-rule="evenodd" d="M10 18a8 8 0 100-16 8 8 0 000 16zm3.707-9.293a1 1 0 00-1.414-1.414L9 10.586 7.707 9.293a1 1 0 00-1.414 1.414l2 2a1 1 0 001.414 0l4-4z" clip-rule="evenodd"/></svg>')
+          html_safe('<svg class="w-5 h-5 text-green-500" fill="currentColor" viewBox="0 0 20 20" role="img" aria-label="' + ERB::Util.html_escape(true_text) + '"><path fill-rule="evenodd" d="M10 18a8 8 0 100-16 8 8 0 000 16zm3.707-9.293a1 1 0 00-1.414-1.414L9 10.586 7.707 9.293a1 1 0 00-1.414 1.414l2 2a1 1 0 001.414 0l4-4z" clip-rule="evenodd"/></svg>')
         else
-          html_safe('<svg class="w-5 h-5 text-gray-400" fill="currentColor" viewBox="0 0 20 20"><path fill-rule="evenodd" d="M10 18a8 8 0 100-16 8 8 0 000 16zM8.707 7.293a1 1 0 00-1.414 1.414L8.586 10l-1.293 1.293a1 1 0 101.414 1.414L10 11.414l1.293 1.293a1 1 0 001.414-1.414L11.414 10l1.293-1.293a1 1 0 00-1.414-1.414L10 8.586 8.707 7.293z" clip-rule="evenodd"/></svg>')
+          html_safe('<svg class="w-5 h-5 text-gray-400" fill="currentColor" viewBox="0 0 20 20" role="img" aria-label="' + ERB::Util.html_escape(false_text) + '"><path fill-rule="evenodd" d="M10 18a8 8 0 100-16 8 8 0 000 16zM8.707 7.293a1 1 0 00-1.414 1.414L8.586 10l-1.293 1.293a1 1 0 101.414 1.414L10 11.414l1.293 1.293a1 1 0 001.414-1.414L11.414 10l1.293-1.293a1 1 0 00-1.414-1.414L10 8.586 8.707 7.293z" clip-rule="evenodd"/></svg>')
         end
       end
     end

data/lib/infotable/table_resolver.rb

@@ -22,9 +22,18 @@ module Infotable
       raise NotFound, "Table '#{table_name}' not found. Make sure it's defined in app/infotable/"
     end
 
+    CANDIDATES = Concurrent::Map.new
+
+    # The class names to try are pure string work (up to 2**(n-1) of them): memoized per name
     def candidates(table_name)
       return [] unless table_name.to_s.match?(/\A[a-z0-9]+(_[a-z0-9]+)*\z/i)
 
+      CANDIDATES.compute_if_absent(table_name.to_s) { build_candidates(table_name) }
+    end
+
+    def build_candidates(table_name)
+      return [] unless table_name.to_s.match?(/\A[a-z0-9]+(_[a-z0-9]+)*\z/i)
+
       parts = table_name.to_s.sub(/_table\z/, "").split("_")
       return [] if parts.empty? || parts.size > MAX_PARTS
 

data/lib/infotable/version.rb

@@ -1,3 +1,3 @@
 module Infotable
-  VERSION = "1.4.0"
+  VERSION = "1.5.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: infotable
 version: !ruby/object:Gem::Version
-  version: 1.4.0
+  version: 1.5.0
 platform: ruby
 authors:
 - Jakub Cieślar
@@ -137,6 +137,7 @@ files:
 - app/assets/javascripts/infotable/controllers/drawer_controller.js
 - app/assets/javascripts/infotable/controllers/table_controller.js
 - app/assets/javascripts/infotable/index.js
+- app/assets/javascripts/infotable/vendor/sortable.esm.js
 - app/assets/tailwind/infotable/engine.css
 - app/controllers/infotable/application_controller.rb
 - app/controllers/infotable/tables_controller.rb
@@ -147,6 +148,7 @@ files:
 - app/models/infotable/application_record.rb
 - app/views/infotable/tables/_active_filters.html.erb
 - app/views/infotable/tables/_column_visibility.html.erb
+- app/views/infotable/tables/_end_indicator.html.erb
 - app/views/infotable/tables/_filter_drawer.html.erb
 - app/views/infotable/tables/_next_page_frame.html.erb
 - app/views/infotable/tables/_pagination.html.erb