activeadmin 3.2.0 → 4.0.0.beta2

data/lib/active_admin/views/components/panel.rb

@@ -8,8 +8,8 @@ module ActiveAdmin
       def build(title, attributes = {})
         super(attributes)
         add_class "panel"
-        @title = h3(title.to_s)
-        @contents = div(class: "panel_contents")
+        @title = h3(title.to_s, class: "panel-title")
+        @contents = div(class: "panel-body")
       end
 
       def add_child(child)
@@ -26,14 +26,6 @@ module ActiveAdmin
       def children?
         @contents.children?
       end
-
-      def header_action(*args)
-        action = args[0]
-
-        @title << div(class: "header_action") do
-          action
-        end
-      end
     end
   end
 end

data/lib/active_admin/views/components/scopes.rb

@@ -1,29 +1,22 @@
 # frozen_string_literal: true
-require "active_admin/helpers/collection"
 require "active_admin/view_helpers/method_or_proc_helper"
 
 module ActiveAdmin
   module Views
-
     # Renders a collection of ActiveAdmin::Scope objects as a
     # simple list with a seperator
     class Scopes < ActiveAdmin::Component
-      builder_method :scopes_renderer
-
       include ActiveAdmin::ScopeChain
-      include ::ActiveAdmin::Helpers::Collection
-
-      def default_class_name
-        "scopes"
-      end
 
       def tag_name
         "div"
       end
 
       def build(scopes, options = {})
+        super({ role: "toolbar" })
+        add_class "scopes"
         scopes.group_by(&:group).each do |group, group_scopes|
-          ul class: "table_tools_segmented_control #{group_class(group)}" do
+          div class: "index-button-group", role: "group", data: { "group": group_name(group) } do
             group_scopes.each do |scope|
               build_scope(scope, options) if call_method_or_exec_proc(scope.display_if_block)
             end
@@ -36,21 +29,19 @@ module ActiveAdmin
       protected
 
       def build_scope(scope, options)
-        li class: classes_for_scope(scope) do
-          params = request.query_parameters.except :page, :scope, :commit, :format
+        params = request.query_parameters.except :page, :scope, :commit, :format
 
-          a href: url_for(scope: scope.id, params: params), class: "table_tools_button" do
-            text_node scope_name(scope)
-            span class: "count" do
-              "(#{get_scope_count(scope)})"
-            end if options[:scope_count] && scope.show_count
+        a href: url_for(scope: scope.id, params: params), class: classes_for_scope(scope) do
+          text_node scope_name(scope)
+          if options[:scope_count] && scope.show_count
+            span get_scope_count(scope), class: "scopes-count"
           end
         end
       end
 
       def classes_for_scope(scope)
-        classes = ["scope", scope.id]
-        classes << "selected" if current_scope?(scope)
+        classes = ["index-button"]
+        classes << "index-button-selected" if current_scope?(scope)
         classes.join(" ")
       end
 
@@ -67,8 +58,8 @@ module ActiveAdmin
         collection_size(scope_chain(scope, collection_before_scope))
       end
 
-      def group_class(group)
-        group.present? ? "scope-group-#{group}" : "scope-default-group"
+      def group_name(group)
+        group.present? ? group : "default"
       end
     end
   end

data/lib/active_admin/views/components/status_tag.rb

@@ -9,12 +9,8 @@ module ActiveAdmin
         "span"
       end
 
-      def default_class_name
-        "status_tag"
-      end
-
       # @overload status_tag(status, options = {})
-      #   @param [String] status the status to display. One of the span classes will be an underscored version of the status.
+      #   @param [String] status the status to display.
       #   @param [Hash] options
       #   @option options [String] :class to override the default class
       #   @option options [String] :id to override the default id
@@ -22,17 +18,30 @@ module ActiveAdmin
       #   @return [ActiveAdmin::Views::StatusTag]
       #
       # @example
+      #   status_tag(true)
+      #   # => <span class="status-tag" data-status="yes">Yes</span>
+      #
+      # @example
+      #   status_tag(false)
+      #   # => <span class="status-tag" data-status="no">No</span>
+      #
+      # @example
+      #   status_tag(nil)
+      #   # => <span class="status-tag" data-status="unset">Unknown</span>
+      #
+      # @example
       #   status_tag('In Progress')
-      #   # => <span class='status_tag in_progress'>In Progress</span>
+      #   # => <span class="status-tag" data-status="in_progress">In Progress</span>
       #
       # @example
-      #   status_tag('active', class: 'important', id: 'status_123', label: 'on')
-      #   # => <span class='status_tag active important' id='status_123'>on</span>
+      #   status_tag('Active', class: 'important', id: 'status_123', label: 'on')
+      #   # => <span class="status-tag important" id="status_123" data-status="active">on</span>
       #
       def build(status, options = {})
         label = options.delete(:label)
         classes = options.delete(:class)
-        status = convert_to_boolean_status(status)
+        boolean_status = convert_to_boolean_status(status)
+        status = boolean_status || status
 
         if status
           content = label || if s = status.to_s and s.present?
@@ -41,8 +50,8 @@ module ActiveAdmin
         end
 
         super(content, options)
-
-        add_class(status_to_class(status)) if status
+        add_class "status-tag"
+        set_attribute("data-status", convert_status(status)) if status
         add_class(classes) if classes
       end
 
@@ -56,19 +65,13 @@ module ActiveAdmin
           "No"
         when nil
           "Unset"
-        else
-          status
         end
       end
 
-      def status_to_class(status)
+      def convert_status(status)
         case status
-        when "Unset"
-          "unset no"
         when String, Symbol
-          status.to_s.titleize.gsub(/\s/, "").underscore
-        else
-          ""
+          status.to_s.titleize.delete(" ").underscore
         end
       end
     end

data/lib/active_admin/views/components/table_for.rb

@@ -20,6 +20,7 @@ module ActiveAdmin
 
         build_table
         super(options)
+        add_class "data-table"
         columns(*attrs)
       end
 
@@ -66,20 +67,26 @@ module ActiveAdmin
       end
 
       def build_table_header(col)
-        classes = Arbre::HTML::ClassList.new
         sort_key = sortable? && col.sortable? && col.sort_key
         params = request.query_parameters.except :page, :order, :commit, :format
 
-        classes << "sortable" if sort_key
-        classes << "sorted-#{current_sort[1]}" if sort_key && current_sort[0] == sort_key
-        classes << col.html_class
+        attributes = {
+          class: col.html_class,
+          "data-column": col.title_id.presence,
+          "data-sortable": (sort_key.present?) ? "" : nil,
+          "data-sort-direction": (sort_key && current_sort[0] == sort_key) ? current_sort[1] : nil
+        }
 
         if sort_key
-          th class: classes do
-            link_to col.pretty_title, params: params, order: "#{sort_key}_#{order_for_sort_key(sort_key)}"
+          th(attributes) do
+            link_to params: params, order: "#{sort_key}_#{order_for_sort_key(sort_key)}" do
+              svg = '<svg class="data-table-sorted-icon" fill="none" viewBox="0 0 10 6"><path stroke="currentColor" stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="m1 1 4 4 4-4"/></svg>'
+
+              (col.pretty_title + svg).html_safe
+            end
           end
         else
-          th col.pretty_title, class: classes
+          th col.pretty_title, attributes
         end
       end
 
@@ -87,19 +94,13 @@ module ActiveAdmin
         @tbody = tbody do
           # Build enough rows for our collection
           @collection.each do |elem|
-            classes = [helpers.cycle("odd", "even")]
-
-            if @row_class
-              classes << @row_class.call(elem)
-            end
-
-            tr(class: classes.flatten.join(" "), id: dom_id_for(elem))
+            tr(id: dom_id_for(elem), class: @row_class&.call(elem))
           end
         end
       end
 
       def build_table_cell(col, resource)
-        td class: col.html_class do
+        td class: col.html_class, "data-column": col.title_id.presence do
           html = helpers.format_attribute(resource, col.data)
           # Don't add the same Arbre twice, while still allowing format_attribute to call status_tag
           current_arbre_element << html unless current_arbre_element.children.include? html
@@ -139,19 +140,13 @@ module ActiveAdmin
 
       class Column
 
-        attr_accessor :title, :data, :html_class
+        attr_accessor :title, :title_id, :data, :html_class
 
         def initialize(*args, &block)
           @options = args.extract_options!
-
           @title = args[0]
-          html_classes = [:col]
-          if @options.has_key?(:class)
-            html_classes << @options.delete(:class)
-          elsif @title.present?
-            html_classes << "col-#{@title.to_s.parameterize(separator: "_")}"
-          end
-          @html_class = html_classes.join(" ")
+          @title_id = @title.to_s.parameterize(separator: "_") if @title.present? && !title.is_a?(Arbre::Element)
+          @html_class = @options.delete(:class)
           @data = args[1] || args[0]
           @data = block if block
           @resource_class = args[2]

data/lib/active_admin/views/components/tabs.rb

@@ -10,31 +10,30 @@ module ActiveAdmin
         @tabs_content << build_content_item(title, options, &block)
       end
 
-      def build(&block)
-        @menu = ul(class: "nav nav-tabs", role: "tablist")
-        @tabs_content = div(class: "tab-content")
+      def build(attributes = {}, &block)
+        super(attributes)
+        add_class "tabs"
+        @menu = nav(class: "tabs-nav", role: "tablist", "data-tabs-toggle": "#tabs-container-#{object_id}")
+        @tabs_content = div(class: "tabs-content", id: "tabs-container-#{object_id}")
       end
 
       def build_menu_item(title, options, &block)
         fragment = options.fetch(:id, fragmentize(title))
-
-        html_options = options.fetch(:html_options, {})
-        li html_options do
-          link_to title, "##{fragment}"
+        html_options = options.fetch(:html_options, {}).merge("data-tabs-target": "##{fragment}", role: "tab", "aria-controls": fragment)
+        button html_options do
+          title
         end
       end
 
       def build_content_item(title, options, &block)
-        options = options.reverse_merge(id: fragmentize(title))
+        options = options.reverse_merge(id: fragmentize(title), class: "hidden", role: "tabpanel", "aria-labelledby": "#{title}-tab")
         div(options, &block)
       end
 
       private
 
       def fragmentize(string)
-        result = string.parameterize
-        result = CGI.escape(string) if result.blank?
-        result
+        "tabs-#{string.parameterize}-#{object_id}"
       end
     end
   end

data/lib/active_admin/views/index_as_table.rb

@@ -72,7 +72,7 @@ module ActiveAdmin
     #   selectable_column
     #   column :title
     #   actions do |post|
-    #     item "Preview", admin_preview_post_path(post), class: "member_link"
+    #     item "Preview", admin_preview_post_path(post), class: "preview-link"
     #   end
     # end
     # ```
@@ -99,32 +99,6 @@ module ActiveAdmin
     # end
     # ```
     #
-    # In case you prefer to list actions links in a dropdown menu:
-    #
-    # ```ruby
-    # index do
-    #   selectable_column
-    #   column :title
-    #   actions dropdown: true do |post|
-    #     item "Preview", admin_preview_post_path(post)
-    #   end
-    # end
-    # ```
-    #
-    # In addition, you can insert the position of the row in the greater
-    # collection by using the index_column special command:
-    #
-    # ```ruby
-    # index do
-    #   selectable_column
-    #   index_column
-    #   column :title
-    # end
-    # ```
-    #
-    # index_column take an optional offset parameter to allow a developer to set
-    # the starting number for the index (default is 1).
-    #
     # ## Sorting
     #
     # When a column is generated from an Active Record attribute, the table is
@@ -234,35 +208,22 @@ module ActiveAdmin
     # ```
     #
     class IndexAsTable < ActiveAdmin::Component
-
       def build(page_presenter, collection)
+        add_class "index-as-table"
         table_options = {
           id: "index_table_#{active_admin_config.resource_name.plural}",
           sortable: true,
-          class: "index_table index",
           i18n: active_admin_config.resource_class,
           paginator: page_presenter[:paginator] != false,
           row_class: page_presenter[:row_class]
         }
 
-        table_for collection, table_options do |t|
-          table_config_block = page_presenter.block || default_table
-          instance_exec(t, &table_config_block)
-        end
-      end
-
-      def table_for(*args, &block)
-        insert_tag IndexTableFor, *args, &block
-      end
-
-      def default_table
-        proc do
-          selectable_column
-          id_column if resource_class.primary_key
-          active_admin_config.resource_columns.each do |attribute|
-            column attribute
+        if page_presenter.block
+          insert_tag(IndexTableFor, collection, table_options) do |t|
+            instance_exec(t, &page_presenter.block)
           end
-          actions
+        else
+          render "index_as_table_default", table_options: table_options
         end
       end
 
@@ -275,29 +236,22 @@ module ActiveAdmin
       # methods for quickly displaying items on the index page
       #
       class IndexTableFor < ::ActiveAdmin::Views::TableFor
-
         # Display a column for checkbox
-        def selectable_column
+        def selectable_column(**options)
           return unless active_admin_config.batch_actions.any?
-          column resource_selection_toggle_cell, class: "col-selectable", sortable: false do |resource|
+          column resource_selection_toggle_cell, class: options[:class], sortable: false do |resource|
             resource_selection_cell resource
           end
         end
 
-        def index_column(start_value = 1)
-          column "#", class: "col-index", sortable: false do |resource|
-            @collection.offset_value + @collection.index(resource) + start_value
-          end
-        end
-
         # Display a column for the id
         def id_column
           raise "#{resource_class.name} has no primary_key!" unless resource_class.primary_key
           column(resource_class.human_attribute_name(resource_class.primary_key), sortable: resource_class.primary_key) do |resource|
             if controller.action_methods.include?("show")
-              link_to resource.id, resource_path(resource), class: "resource_id_link"
+              link_to resource.id, resource_path(resource)
             elsif controller.action_methods.include?("edit")
-              link_to resource.id, edit_resource_path(resource), class: "resource_id_link"
+              link_to resource.id, edit_resource_path(resource)
             else
               resource.id
             end
@@ -329,38 +283,17 @@ module ActiveAdmin
         #   item 'Grant Admin', grant_admin_admin_user_path(admin_user)
         # end
         #
-        # # Append some actions onto the end of the default actions displayed in a Dropdown Menu
-        # actions dropdown: true do |admin_user|
-        #   item 'Grant Admin', grant_admin_admin_user_path(admin_user)
-        # end
-        #
-        # # Custom actions without the defaults displayed in a Dropdown Menu.
-        # actions defaults: false, dropdown: true, dropdown_name: 'Additional actions' do |admin_user|
-        #   item 'Grant Admin', grant_admin_admin_user_path(admin_user)
-        # end
-        #
         # ```
         def actions(options = {}, &block)
           name = options.delete(:name) { "" }
           defaults = options.delete(:defaults) { true }
-          dropdown = options.delete(:dropdown) { false }
-          dropdown_name = options.delete(:dropdown_name) { I18n.t "active_admin.dropdown_actions.button_label", default: "Actions" }
-
-          options[:class] ||= "col-actions"
 
           column name, options do |resource|
-            if dropdown
-              dropdown_menu dropdown_name do
-                defaults(resource) if defaults
-                instance_exec(resource, &block) if block_given?
-              end
-            else
-              table_actions do
-                defaults(resource, css_class: :member_link) if defaults
-                if block_given?
-                  block_result = instance_exec(resource, &block)
-                  text_node block_result unless block_result.is_a? Arbre::Element
-                end
+            insert_tag(TableActions, class: "data-table-resource-actions") do
+              render "index_table_actions_default", defaults_data(resource) if defaults
+              if block
+                block_result = instance_exec(resource, &block)
+                text_node block_result unless block_result.is_a? Arbre::Element
               end
             end
           end
@@ -368,29 +301,23 @@ module ActiveAdmin
 
         private
 
-        def defaults(resource, options = {})
+        def defaults_data(resource)
           localizer = ActiveAdmin::Localizers.resource(active_admin_config)
-          if controller.action_methods.include?("show") && authorized?(ActiveAdmin::Auth::READ, resource)
-            item localizer.t(:view), resource_path(resource), class: "view_link #{options[:css_class]}", title: localizer.t(:view)
-          end
-          if controller.action_methods.include?("edit") && authorized?(ActiveAdmin::Auth::EDIT, resource)
-            item localizer.t(:edit), edit_resource_path(resource), class: "edit_link #{options[:css_class]}", title: localizer.t(:edit)
-          end
-          if controller.action_methods.include?("destroy") && authorized?(ActiveAdmin::Auth::DESTROY, resource)
-            item localizer.t(:delete), resource_path(resource), class: "delete_link #{options[:css_class]}", title: localizer.t(:delete),
-                                                                method: :delete, data: { confirm: localizer.t(:delete_confirmation) }
-          end
+          {
+            resource: resource,
+            view_label: localizer.t(:view),
+            edit_label: localizer.t(:edit),
+            delete_label: localizer.t(:delete),
+            delete_confirmation_text: localizer.t(:delete_confirmation)
+          }
         end
 
         class TableActions < ActiveAdmin::Component
-          builder_method :table_actions
-
           def item *args, **kwargs
             text_node link_to(*args, **kwargs)
           end
         end
       end # IndexTableFor
-
     end
   end
 end

data/lib/active_admin.rb

@@ -7,21 +7,16 @@ require "kaminari"
 require "formtastic"
 require "formtastic_i18n"
 require "inherited_resources"
-require "jquery-rails"
 require "arbre"
 
-require "active_admin/helpers/i18n"
-
 module ActiveAdmin
 
   autoload :VERSION, "active_admin/version"
   autoload :Application, "active_admin/application"
-  autoload :AssetRegistration, "active_admin/asset_registration"
   autoload :Authorization, "active_admin/authorization_adapter"
   autoload :AuthorizationAdapter, "active_admin/authorization_adapter"
   autoload :Callbacks, "active_admin/callbacks"
   autoload :Component, "active_admin/component"
-  autoload :BaseController, "active_admin/base_controller"
   autoload :CanCanAdapter, "active_admin/cancan_adapter"
   autoload :ControllerAction, "active_admin/controller_action"
   autoload :CSVBuilder, "active_admin/csv_builder"
@@ -38,30 +33,32 @@ module ActiveAdmin
   autoload :OrderClause, "active_admin/order_clause"
   autoload :Page, "active_admin/page"
   autoload :PagePresenter, "active_admin/page_presenter"
-  autoload :PageController, "active_admin/page_controller"
   autoload :PageDSL, "active_admin/page_dsl"
   autoload :PunditAdapter, "active_admin/pundit_adapter"
   autoload :Resource, "active_admin/resource"
-  autoload :ResourceController, "active_admin/resource_controller"
   autoload :ResourceDSL, "active_admin/resource_dsl"
   autoload :Scope, "active_admin/scope"
   autoload :ScopeChain, "active_admin/helpers/scope_chain"
   autoload :SidebarSection, "active_admin/sidebar_section"
   autoload :TableBuilder, "active_admin/table_builder"
-  autoload :ViewFactory, "active_admin/view_factory"
   autoload :ViewHelpers, "active_admin/view_helpers"
   autoload :Views, "active_admin/views"
 
   class << self
 
-    attr_accessor :application
+    attr_accessor :application, :importmap
 
     def application
       @application ||= ::ActiveAdmin::Application.new
     end
 
     def deprecator
-      @deprecator ||= ActiveSupport::Deprecation.new("4.0", "active-admin")
+      @deprecator ||= ActiveSupport::Deprecation.new("4.1", "active-admin")
+    end
+
+    def importmap
+      require "importmap-rails"
+      @importmap ||= Importmap::Map.new
     end
 
     # Gets called within the initializer
@@ -132,4 +129,3 @@ require "active_admin/filters"
 
 # Require ORM-specific plugins
 require "active_admin/orm/active_record" if defined? ActiveRecord
-require "active_admin/orm/mongoid" if defined? Mongoid

data/lib/generators/active_admin/assets/assets_generator.rb

@@ -2,14 +2,14 @@
 module ActiveAdmin
   module Generators
     class AssetsGenerator < Rails::Generators::Base
-
       source_root File.expand_path("templates", __dir__)
 
       def install_assets
-        template "active_admin.js", "app/assets/javascripts/active_admin.js"
-        template "active_admin.scss", "app/assets/stylesheets/active_admin.scss"
+        remove_file "app/assets/stylesheets/active_admin.scss"
+        remove_file "app/assets/javascripts/active_admin.js"
+        template "active_admin.css", "app/assets/stylesheets/active_admin.css"
+        template "tailwind.config.js", "tailwind-active_admin.config.js"
       end
-
     end
   end
 end

data/lib/generators/active_admin/assets/templates/active_admin.css

@@ -0,0 +1,3 @@
+@tailwind base;
+@tailwind components;
+@tailwind utilities;

data/lib/generators/active_admin/assets/templates/tailwind.config.js

@@ -0,0 +1,18 @@
+const execSync = require('child_process').execSync;
+const activeAdminPath = execSync('bundle show activeadmin', { encoding: 'utf-8' }).trim();
+
+module.exports = {
+  content: [
+    `${activeAdminPath}/vendor/javascript/flowbite.js`,
+    `${activeAdminPath}/plugin.js`,
+    `${activeAdminPath}/app/views/**/*.{arb,erb,html,rb}`,
+    './app/admin/**/*.{arb,erb,html,rb}',
+    './app/views/active_admin/**/*.{arb,erb,html,rb}',
+    './app/views/admin/**/*.{arb,erb,html,rb}',
+    './app/javascript/**/*.js'
+  ],
+  darkMode: "class",
+  plugins: [
+    require(`@activeadmin/activeadmin/plugin`)
+  ]
+}

data/lib/generators/active_admin/install/install_generator.rb

@@ -9,15 +9,13 @@ module ActiveAdmin
 
       hook_for :users, default: "devise", desc: "Admin user generator to run. Skip with --skip-users"
       class_option :skip_comments, type: :boolean, default: false, desc: "Skip installation of comments"
-      class_option :use_webpacker, type: :boolean, default: false, desc: "Use Webpacker assets instead of Sprockets"
 
       source_root File.expand_path("templates", __dir__)
 
       def copy_initializer
-        @underscored_user_name = name.underscore.gsub("/", "_")
+        @underscored_user_name = name.underscore.tr("/", "_")
         @use_authentication_method = options[:users].present?
         @skip_comments = options[:skip_comments]
-        @use_webpacker = options[:use_webpacker]
         template "active_admin.rb.erb", "config/initializers/active_admin.rb"
       end
 
@@ -39,11 +37,7 @@ module ActiveAdmin
       end
 
       def create_assets
-        if options[:use_webpacker]
-          generate "active_admin:webpacker"
-        else
-          generate "active_admin:assets"
-        end
+        generate "active_admin:assets"
       end
 
       def create_migrations