Allow configuration of Navbar css class

mshibuya · mshibuya · commit 126f7ac79b2c · 2022-04-25T18:41:58.000+09:00
Refs. #3507
diff --git a/.rubocop.yml b/.rubocop.yml
@@ -118,7 +118,7 @@ Metrics/MethodLength:
   Max: 29 # TODO: Lower to 15
 
 Metrics/ModuleLength:
-  Max: 217 # TODO: Lower to 100
+  Max: 219 # TODO: Lower to 100
 
 Metrics/ParameterLists:
   Max: 8 # TODO: Lower to 4
diff --git a/app/views/layouts/rails_admin/application.html.erb b/app/views/layouts/rails_admin/application.html.erb
@@ -5,7 +5,7 @@
   </head>
   <body class="rails_admin">
     <div data-i18n-options="<%= I18n.t("admin.js").to_json %>" id="admin-js"></div>
-    <nav class="navbar navbar-expand-md fixed-top navbar-dark bg-primary border-bottom">
+    <nav class="navbar navbar-expand-md fixed-top <%= RailsAdmin::Config.navbar_css_classes.join(' ') %>">
       <%= render "layouts/rails_admin/navigation" %>
     </nav>
     <div class="container-fluid">
diff --git a/lib/rails_admin/config.rb b/lib/rails_admin/config.rb
@@ -75,6 +75,9 @@ class << self
       # @see RailsAdmin.config
       attr_reader :registry
 
+      # Bootstrap CSS classes used for Navigation bar
+      attr_accessor :navbar_css_classes
+
       # show Gravatar in Navigation bar
       attr_accessor :show_gravatar
 
@@ -336,6 +339,7 @@ def reset
         @label_methods = %i[name title]
         @main_app_name = proc { [Rails.application.engine_name.titleize.chomp(' Application'), 'Admin'] }
         @registry = {}
+        @navbar_css_classes = %w[navbar-dark bg-primary border-bottom]
         @show_gravatar = true
         @navigation_static_links = {}
         @navigation_static_label = nil
diff --git a/spec/integration/rails_admin_spec.rb b/spec/integration/rails_admin_spec.rb
@@ -61,6 +61,20 @@
     end
   end
 
+  describe 'navbar css class' do
+    it 'is set by default' do
+      expect(RailsAdmin.config.navbar_css_classes).to eq(%w[navbar-dark bg-primary border-bottom])
+    end
+
+    it 'can be configured' do
+      RailsAdmin.config do |config|
+        config.navbar_css_classes = %w[navbar-light border-bottom]
+      end
+      visit dashboard_path
+      is_expected.to have_css('nav.navbar.navbar-light.border-bottom')
+    end
+  end
+
   describe '_current_user' do # https://github.com/railsadminteam/rails_admin/issues/549
     it 'is accessible from the list view' do
       RailsAdmin.config Player do
