Merge pull request #3 from workgena/sidebar_classes

Separate the logic for sidebar main_content_classes

Author: Fivell

lib/active_admin_sidebar/activeadmin_views_pages_base.rb

@@ -2,27 +2,13 @@ class ActiveAdmin::Views::Pages::Base < Arbre::HTML::Document
 
   def build_page_content
     build_flash_messages
-    classes = Arbre::HTML::ClassList.new
-
-    if skip_sidebar?
-      classes << 'without_sidebar'
-    else
-      classes << 'with_sidebar'
-      classes << 'left_sidebar' if left_sidebar?
-    end
-
-
-
-    div :id => "active_admin_content", :class => classes do
-
+    div id: "active_admin_content", class: main_content_classes do
       build_sidebar unless skip_sidebar? || right_sidebar?
       build_main_content_wrapper
       build_sidebar unless skip_sidebar? || left_sidebar?
-
     end
   end
 
-
   def left_sidebar?
     assigns[:sidebar_position] == :left
   end
@@ -31,6 +17,15 @@ def right_sidebar?
      !left_sidebar?
   end
 
+  def main_content_classes
+    classes = Arbre::HTML::ClassList.new
+    if skip_sidebar?
+      classes << 'without_sidebar'
+    else
+      classes << 'with_sidebar'
+      classes << 'left_sidebar' if left_sidebar?
+    end
+    classes
+  end
 
-
-end
+end