activeadmin_favorites 0.1.7 → 0.1.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 87ee6c5a7e74841630d69de986dcc7dfef4f799e094f696529b3fe51b6c53552
-  data.tar.gz: b9381534a09f71b745c9d3a317d4890c0a21391c27be544e421be5b76cda43c2
+  metadata.gz: 1c6a1ca67243ef7bcf12defce94d288b0afca716511a3a04f6326ecce641e784
+  data.tar.gz: 0150e3932226fe8890a22dac1889d57f8e74fb868928ec8027763a8e07acfc0c
 SHA512:
-  metadata.gz: 422903c24269839ed1142a83e6c8fcc1f5aff493eba16a6d91574869eaaf8c64dcdde3b650c8ce4a90a29f56c4bdc2f656881204e44bb6a60b135eb7ece956e7
-  data.tar.gz: f9bf4dabb5f05f413a031b90694af82929117f14c5b3855c05be0c057f39d66441a83e68d755826ef4ff180a3e367f4f04d18a66a9e57c8d23c29f5965a8e893
+  metadata.gz: 520cd1dec8f6f362b90163d65f8a3d3231f603481a7f95354600cd455cde6fbe37eaaae3bd5b3f26e90cae6bd784075b4dfb4432b4f43747cde70f25ff2ba7fe
+  data.tar.gz: 510d6d7d6b0bc26017bca1d6ff576ba3e835f95215dba7a299ca810d40ea44d541f88e21526449a575f27b41535b1fb1e1e76a5902d9947223a132d3d5984d04

data/CHANGELOG.md

@@ -1,5 +1,10 @@
 # CHANGELOG
 
+## 0.1.8 (2026-10-29)
+
+- Fix Active Admin view-path detection so sibling gems such as `activeadmin-command_palette` are not treated as Active Admin. Prefer `ActiveAdmin::Engine.root`, with a fallback that requires a digit version segment after `activeadmin-`.
+- Remove the gem `active_admin/_html_head` override. Hosts must link `activeadmin_favorites/personalization_menu` from their own Active Admin head.
+
 ## 0.1.7 (2026-10-29)
 
 - Fix engine view path order so favorites sits immediately before ActiveAdmin and host `app/views` stays ahead of the gem `_html_head`. Rails engines prepend view paths; the previous reorder left favorites first whenever it was already ahead of ActiveAdmin.

data/README.md

@@ -61,9 +61,7 @@ When using ActionPolicy, add `save_lens_favorite?` and `reset?` to `ActiveAdmin:
 
 ### Stylesheet
 
-The engine ships an `active_admin/_html_head` that links `activeadmin_favorites/personalization_menu` so the cogwheel dropdown can scroll the catalog while keeping action buttons in view. The file is plain CSS served through Propshaft; it does not need Tailwind processing. Engine view paths sit immediately before ActiveAdmin's, so a host `app/views/active_admin/_html_head` still wins.
-
-If the host overrides that partial, add the link there:
+Link `activeadmin_favorites/personalization_menu` from the host Active Admin HTML head so the cogwheel dropdown can scroll the catalog while keeping action buttons in view. The file is plain CSS served through Propshaft; it does not need Tailwind processing. The gem does not ship `active_admin/_html_head`, so customized host heads keep their stylesheet names and extra tags.
 
 ```erb
 <%= stylesheet_link_tag "activeadmin_favorites/personalization_menu" %>

data/lib/activeadmin/favorites/version.rb

@@ -2,6 +2,6 @@
 
 module ActiveAdmin
   module Favorites
-    VERSION = "0.1.7"
+    VERSION = "0.1.8"
   end
 end

data/lib/activeadmin/favorites/view_paths.rb

@@ -4,8 +4,7 @@ module ActiveAdmin
   module Favorites
     # Place this engine's views immediately before ActiveAdmin's so resource
     # index/show overrides (catalog injection) win. Rails engines prepend their
-    # view paths, so favorites can otherwise sit ahead of the host app/views and
-    # shadow customized templates such as active_admin/_html_head.
+    # view paths, so favorites can otherwise sit ahead of the host app/views.
     module ViewPaths
       module_function
 
@@ -35,7 +34,16 @@ module ActiveAdmin
       end
 
       def active_admin_views_root(path_strings)
-        path_strings.find { |path| path.match?(%r{/gems/activeadmin-[^/]+/app/views\z}) }
+        engine_root = active_admin_engine_views_root
+        return engine_root if engine_root && path_strings.include?(engine_root)
+
+        path_strings.find { |path| path.match?(ACTIVE_ADMIN_GEM_VIEWS) }
+      end
+
+      def active_admin_engine_views_root
+        return unless defined?(::ActiveAdmin::Engine)
+
+        ::ActiveAdmin::Engine.root.join("app/views").to_s
       end
 
       def already_before_active_admin?(path_strings, favorites_root, active_admin_root)
@@ -49,10 +57,16 @@ module ActiveAdmin
         moved
       end
 
+      # Versioned Active Admin gem only (`activeadmin-4...`), not siblings such as
+      # `activeadmin-command_palette-...`.
+      ACTIVE_ADMIN_GEM_VIEWS = %r{/gems/activeadmin-\d[^/]*/app/views\z}
+
       module_function :reorder_path_strings, :active_admin_views_root,
-        :already_before_active_admin?, :place_before_active_admin
+        :active_admin_engine_views_root, :already_before_active_admin?,
+        :place_before_active_admin
       private_class_method :reorder_path_strings, :active_admin_views_root,
-        :already_before_active_admin?, :place_before_active_admin
+        :active_admin_engine_views_root, :already_before_active_admin?,
+        :place_before_active_admin
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: activeadmin_favorites
 version: !ruby/object:Gem::Version
-  version: 0.1.7
+  version: 0.1.8
 platform: ruby
 authors:
 - Andrei Makarov
@@ -355,7 +355,6 @@ files:
 - app/models/active_admin/favorites/lens_favorite.rb
 - app/models/active_admin/favorites/view_lens/layout.rb
 - app/models/active_admin/favorites/view_lens_default.rb
-- app/views/active_admin/_html_head.html.erb
 - app/views/active_admin/favorites/_lens_catalog_data.html.erb
 - app/views/active_admin/favorites/_macro_fields.html.erb
 - app/views/active_admin/favorites/_personalization_trigger.html.erb

data/app/views/active_admin/_html_head.html.erb

@@ -1,14 +0,0 @@
-<%= stylesheet_link_tag "active_admin" %>
-<%= stylesheet_link_tag ActiveAdmin::Favorites::PERSONALIZATION_MENU_STYLESHEET %>
-<meta name="viewport" content="width=device-width,initial-scale=1">
-<%= csrf_meta_tags %>
-<%= csp_meta_tag %>
-<% # On page load or when changing themes, best to add inline in `head` to avoid FOUC %>
-<%= javascript_tag nonce: true do %>
-  if (localStorage.theme === 'dark' || (!('theme' in localStorage) && window.matchMedia('(prefers-color-scheme: dark)').matches)) {
-    document.documentElement.classList.add('dark')
-  } else {
-    document.documentElement.classList.remove('dark')
-  }
-<% end %>
-<%= javascript_importmap_tags "active_admin", importmap: ActiveAdmin.importmap %>