avo 4.0.13 → 4.0.15

data/app/assets/stylesheets/css/focus.css

@@ -16,7 +16,7 @@
     outline: none;
   }
 
-  a[href]:empty:not([aria-label]) {
+  a[href]:empty:not([aria-label]):not([aria-labelledby]) {
     display: none;
   }
 

data/app/components/avo/fields/index_component.rb

@@ -8,7 +8,7 @@ class Avo::Fields::IndexComponent < Avo::BaseComponent
   attr_reader :parent_resource
   attr_reader :view
 
-  def initialize(field: nil, resource: nil, reflection: nil, index: 0, parent_record: nil, parent_resource: nil, **kwargs)
+  def initialize(field: nil, resource: nil, reflection: nil, index: 0, parent_record: nil, parent_resource: nil, density: nil, field_wrapper: {}, **kwargs)
     @field = field
     @resource = resource
     @index = index
@@ -17,6 +17,8 @@ class Avo::Fields::IndexComponent < Avo::BaseComponent
     @parent_resource = parent_resource
     @view = Avo::ViewInquirer.new("index")
     @reflection = reflection
+    @density = density
+    @field_wrapper_options = field_wrapper
   end
 
   def resource_view_path
@@ -35,7 +37,9 @@ class Avo::Fields::IndexComponent < Avo::BaseComponent
   def field_wrapper_args
     {
       field: @field,
-      resource: @resource
+      resource: @resource,
+      density: @density,
+      **@field_wrapper_options
     }
   end
 end

data/app/components/avo/index/field_wrapper_component.html.erb

@@ -1,10 +1,10 @@
-<%= content_tag :td,
+<%= content_tag @html_tag,
   class: class_names(
     "group/clipboard px-2 leading-none whitespace-nowrap h-full text-content text-sm",
     @classes,
     @field.get_html(:classes, view: @view, element: :wrapper),
     {
-      "py-4": !@flush
+      density_padding_class => !@flush
     }
   ),
   data: {

data/app/components/avo/index/field_wrapper_component.rb

@@ -1,11 +1,23 @@
 # frozen_string_literal: true
 
 class Avo::Index::FieldWrapperComponent < Avo::BaseComponent
+  # Vertical padding for each density step. `nil` density falls back to
+  # `Avo.configuration.density`.
+  DENSITY_PADDING = {
+    tight: "py-2",
+    normal: "py-4",
+    relaxed: "py-6"
+  }.freeze
+
   prop :field
   prop :resource
   prop :dash_if_blank, default: true
   prop :center_content, default: false
   prop :flush, default: false
+  prop :density, default: nil
+  # The wrapper element. Defaults to a table cell; non-table hosts (e.g. the
+  # dashboards list card) render the same field components inside a :div.
+  prop :html_tag, default: :td
   prop :args, kind: :**, default: {}.freeze
 
   def after_initialize
@@ -31,4 +43,8 @@ class Avo::Index::FieldWrapperComponent < Avo::BaseComponent
   def render_dash?
     @field.value.blank? && @dash_if_blank
   end
+
+  def density_padding_class
+    DENSITY_PADDING[(@density || Avo.configuration.density)&.to_sym] || DENSITY_PADDING[:normal]
+  end
 end

data/app/components/avo/u_i/count_component.html.erb

@@ -1 +1 @@
-<%= content_tag :span, @count, class: class_names("count", @classes), data: @data %>
+<%= content_tag :span, @count, class: class_names("count", @classes), data: @data, "aria-label": @label %>

data/app/components/avo/u_i/count_component.rb

@@ -8,4 +8,7 @@ class Avo::UI::CountComponent < Avo::BaseComponent
   prop :count
   prop :classes, default: nil
   prop :data, default: -> { {} }
+  # Optional accessible name (e.g. "1,234 records") when the visible +count+ is
+  # abbreviated or purely numeric. Rendered as aria-label; omitted when nil.
+  prop :label, default: nil
 end

data/app/components/avo/u_i/tabs/tab_component.html.erb

@@ -22,5 +22,6 @@
   ) do %>
     <%= svg @icon, class: "tabs__item-icon", "aria-hidden": "true" if @icon.present? %>
     <%= tag.span @label, class: "tabs__item-label" if @label.present? %>
+    <%= render_badge %>
   <% end %>
 <% end %>

data/app/components/avo/u_i/tabs/tab_component.rb

@@ -19,6 +19,17 @@ module Avo
         prop :data, default: {}.freeze
         prop :classes
         prop :title
+        prop :badge, default: nil
+
+        private
+
+        # The badge may be a component instance (eager count pill) or a
+        # pre-rendered safe string (deferred counter frame); render accordingly.
+        def render_badge
+          return unless @badge.present?
+
+          @badge.respond_to?(:render_in) ? render(@badge) : @badge
+        end
       end
     end
   end

data/app/javascript/js/controllers/table_row_controller.js

@@ -17,8 +17,8 @@ export default class extends Controller {
   anchor = null
 
   mouseEntered(event) {
-    const row = event.target.closest('tr')
-    const url = row.dataset.visitPath
+    const row = event.target.closest('[data-visit-path]')
+    const url = row?.dataset?.visitPath
 
     if (url) {
       this.anchor = this.createAnchor(url)
@@ -64,14 +64,14 @@ export default class extends Controller {
       return // Don't navigate if a link or button is clicked
     }
 
-    const row = event.target.closest('tr')
-    const url = row.dataset.visitPath
+    const row = event.target.closest('[data-visit-path]')
+    const url = row?.dataset?.visitPath
 
     if (!row || !url) {
       return
     }
 
-    if (event.metaKey || event.ctrlKey) {
+    if (event.metaKey || event.ctrlKey || row.dataset.visitTarget === '_blank') {
       this.#visitInNewTab(url)
     } else {
       this.#visitInSameTab(url)

data/lib/avo/configuration.rb

@@ -51,6 +51,7 @@ module Avo
     attr_accessor :resource_parent_controller
     attr_accessor :default_url_options
     attr_accessor :click_row_to_view_record
+    attr_accessor :density
     attr_accessor :alert_dismiss_time
     attr_accessor :is_admin_method
     attr_accessor :is_developer_method
@@ -182,6 +183,7 @@ module Avo
       @default_url_options = []
       @pagination = {}
       @click_row_to_view_record = true
+      @density = :normal
       @alert_dismiss_time = 5000
       @is_admin_method = :is_admin?
       @is_developer_method = :is_developer?

data/lib/avo/plugin_manager.rb

@@ -1,18 +1,47 @@
 module Avo
   class PluginManager
     attr_reader :plugins
-    attr_accessor :engines
+    attr_reader :engines
 
     alias_method :all, :plugins
 
     def initialize
       @plugins = []
       @engines = []
+      @building_plugins = []
+      @building_engines = []
     end
 
-    def reset
-      @plugins = []
-      @engines = []
+    # Starts a re-registration cycle without touching the currently published
+    # @plugins/@engines, so concurrent readers (e.g. mount_avo's route-drawing
+    # loop) keep seeing the complete pre-reload list until #commit_reload
+    # publishes the new one. Called by Avo.boot inside its Mutex, so only one
+    # reload is ever building at a time.
+    def begin_reload
+      @building_plugins = []
+      @building_engines = []
+    end
+
+    # Publishes the registrations collected since #begin_reload. Each of the
+    # two reassignments below is individually an atomic pointer swap, so a
+    # reader of .engines alone (or .plugins alone) always sees the complete
+    # old list or the complete new one, never a partially rebuilt one. The
+    # two fields are not published jointly -- a reader combining both in one
+    # operation could observe them from different reload generations. No
+    # current reader does that (mount_avo reads only .engines; installed?
+    # reads only .plugins).
+    def commit_reload
+      @plugins = @building_plugins
+      @engines = @building_engines
+      # Reset to fresh arrays rather than nil: register/mount_engine must
+      # stay callable even outside a begin_reload/commit_reload window --
+      # e.g. avo-permissions calls Avo.plugin_manager.register at Rails
+      # initializer time, before Avo.boot ever runs. Such a call is simply
+      # discarded by the next #begin_reload rather than raising, matching
+      # the pre-atomic-publish behavior where register/mount_engine never
+      # crashed regardless of timing.
+      @building_plugins = []
+      @building_engines = []
     end
 
     def register(name, priority: 10)
@@ -21,7 +50,7 @@ module Avo
       # (e.g. `:rhino` for `avo-rhino_field`), so the name alone isn't enough.
       registered_from = caller_locations(1, 1)&.first&.path
 
-      @plugins << Plugin.new(name:, priority: priority, registered_from: registered_from)
+      @building_plugins << Plugin.new(name:, priority: priority, registered_from: registered_from)
     end
 
     def register_view_type(name, component:, icon:, active_icon:, translation_key: nil)
@@ -80,7 +109,11 @@ module Avo
     end
 
     def mount_engine(klass, **options)
-      @engines << {klass:, options:}
+      # Dedup by class so a plugin hook that (by bug) calls mount_engine
+      # twice for the same engine within one boot can't leave a duplicate
+      # entry for #commit_reload to publish.
+      @building_engines.delete_if { |engine| engine[:klass] == klass }
+      @building_engines << {klass:, options:}
     end
   end
 

data/lib/avo/version.rb

@@ -1,3 +1,3 @@
 module Avo
-  VERSION = "4.0.13" unless const_defined?(:VERSION)
+  VERSION = "4.0.15" unless const_defined?(:VERSION)
 end

data/lib/avo.rb

@@ -55,6 +55,11 @@ module Avo
 
   class ViewTypeComponentNotFoundError < StandardError; end
 
+  # Serializes Avo.boot so concurrent code reloads (config.to_prepare fires on
+  # every reload, from every request thread) can't interleave the plugin
+  # registry reset with its repopulation. See PluginManager#begin_reload.
+  @boot_mutex = Mutex.new
+
   # Exception raised when a resource is missing
   class MissingResourceError < StandardError
     def initialize(model_class, field)
@@ -92,16 +97,19 @@ module Avo
 
     # Runs when the app boots up
     def boot
-      Turbo::Streams::TagBuilder.prepend(Avo::TurboStreamActionsHelper)
-      @logger = Avo.configuration.logger
-      @field_manager = Avo::Fields::FieldManager.build
-      @view_type_manager = nil # force re-init with defaults on next access
-      @cache_store = Avo.configuration.cache_store
-      Avo.plugin_manager.reset
-      # Run load hooks for plugins to include them in the app.
-      # This is useful for plugins that need to include modules in the app that will be used on avo_boot hook.
-      ActiveSupport.run_load_hooks(:avo_plugin_include, self)
-      ActiveSupport.run_load_hooks(:avo_boot, self)
+      @boot_mutex.synchronize do
+        Turbo::Streams::TagBuilder.prepend(Avo::TurboStreamActionsHelper)
+        @logger = Avo.configuration.logger
+        @field_manager = Avo::Fields::FieldManager.build
+        @view_type_manager = nil # force re-init with defaults on next access
+        @cache_store = Avo.configuration.cache_store
+        Avo.plugin_manager.begin_reload
+        # Run load hooks for plugins to include them in the app.
+        # This is useful for plugins that need to include modules in the app that will be used on avo_boot hook.
+        ActiveSupport.run_load_hooks(:avo_plugin_include, self)
+        ActiveSupport.run_load_hooks(:avo_boot, self)
+        Avo.plugin_manager.commit_reload
+      end
       eager_load_actions
     end
 

data/lib/generators/avo/templates/initializer/avo.tt

@@ -118,6 +118,7 @@ Avo.configure do |config|
 
   ## == Customization ==
   config.click_row_to_view_record = true
+  # config.density = :normal # :tight, :normal, :relaxed
   # config.app_name = 'Avocadelicious'
   # config.timezone = 'UTC'
   # config.currency = 'USD'

metadata

@@ -1,16 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: avo
 version: !ruby/object:Gem::Version
-  version: 4.0.13
+  version: 4.0.15
 platform: ruby
 authors:
 - Adrian Marin
 - Mihai Marin
 - Paul Bob
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-07-16 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -1199,8 +1198,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.22
-signing_key:
+rubygems_version: 4.0.10
 specification_version: 4
 summary: Admin panel framework and Content Management System for Ruby on Rails.
 test_files: []