Made the header image idiot-proof style wise, also added a recommended image size. (Hey that rhymed!). Closes activeadmin#560

Author: mattvague

app/assets/stylesheets/active_admin/_header.css.scss

@@ -3,15 +3,19 @@
   @include primary-gradient;
   @include shadow;
   @include text-shadow(#000);
+  height: 20px;
   overflow: visible;
+  position: relative;
   padding: 9px $horizontal-page-margin;
   z-index: 900;
 
   h1 { 
     display: inline-block; 
     color: #cdcdcd;
+    float: left;
     margin-right: 20px; 
     margin-bottom: 0px;
+    padding-top: 3px;
     font-size: 1.3em; 
     font-weight: normal;
 
@@ -21,17 +25,28 @@
       &:hover {
         color: #fff;
       }
-      
     } 
+    
+    img {
+      position: relative;
+      top: -2px;
+    }
   }
 
   a, a:link { color: #cdcdcd; }
 
+  .header-item {
+    float: left;
+    top: 2px;
+    position: relative;
+    height: 20px
+  }
+
   ul#tabs {
-    margin: 0; 
-    padding: 0; 
     display: inline-block; 
     height: 100%;
+    margin: 0; 
+    padding: 0;
 
     & > li { 
       display: inline-block; 
@@ -108,8 +123,11 @@
   }
 
   #utility_nav { 
-    position: absolute; right: 25px; top: 10px; 
     color: #aaa;
+    float: right;
+    display: inline-block;
+    margin: 0;
+    padding: 0;
     span, a { margin-left: 10px; }
 
     a { text-decoration: none; }

lib/active_admin/views/header_renderer.rb

@@ -43,11 +43,11 @@ def title_text
       #
       # It uses the ActiveAdmin.tabs_renderer option
       def global_navigation
-        render view_factory.global_navigation, current_menu
+        render view_factory.global_navigation, current_menu, :class => 'header-item' 
       end
 
       def utility_navigation
-        content_tag 'p', :id => "utility_nav" do
+        content_tag 'p', :id => "utility_nav", :class => 'header-item' do
           if current_active_admin_user?
             html = content_tag(:span, display_name(current_active_admin_user), :class => "current_user")
 

lib/generators/active_admin/install/templates/active_admin.rb.erb

@@ -15,6 +15,8 @@ ActiveAdmin.setup do |config|
   # Set an optional image to be displayed for the header
   # instead of a string (overrides :site_title)
   #
+  # Note: Recommended image height is 21px to properly fit in the header
+  #
   # config.site_title_image = "/images/logo.png"
 
   # == Default Namespace