From f34b131678315e3eb6edd21fc9b7c775e299a156 Mon Sep 17 00:00:00 2001
From: Brian Talbot <btalbot@edx.org>
Date: Tue, 4 Jun 2013 14:44:57 -0400
Subject: [PATCH 1/2] lms - revises the physical height of the edX logo within
 the slim-sized header and vertically aligns all elements

---
 .../course/layout/_courseware_header.scss     | 21 ++++++++++++-------
 1 file changed, 13 insertions(+), 8 deletions(-)

diff --git a/lms/static/sass/course/layout/_courseware_header.scss b/lms/static/sass/course/layout/_courseware_header.scss
index 4d8f000668..9650dabc95 100644
--- a/lms/static/sass/course/layout/_courseware_header.scss
+++ b/lms/static/sass/course/layout/_courseware_header.scss
@@ -62,7 +62,8 @@ nav.course-material {
 
 header.global.slim {
   @include box-shadow(0 1px 2px rgba(0, 0, 0, .1));
-  height: 50px;
+  height: auto;
+  padding: 5px 0 10px 0;
   border-bottom: 1px solid $outer-border-color;
   background: $white;
 
@@ -106,16 +107,15 @@ header.global.slim {
     padding-top: 5px;
   }
 
-  h1.logo {    
-    margin-left: 13px;
-    margin-right: 20px;
+  h1.logo {
+    margin: 0 10px 0 13px;
     padding-right: 20px;
 
     &:before {
       @extend .faded-vertical-divider;
       content: "";
       display: block;
-      height: 40px;
+      height: 35px;
       position: absolute;
       right: 3px;
       top: 0;
@@ -126,12 +126,16 @@ header.global.slim {
       @extend .faded-vertical-divider-light;
       content: "";
       display: block;
-      height: 40px;
+      height: 35px;
       position: absolute;
-      right: 0px;
+      right: 0;
       top: 0;
       width: 1px;
     }
+
+    img {
+      height: 30px;
+    }
   }
 
   .nav-global {
@@ -147,6 +151,7 @@ header.global.slim {
     color: #777;
     letter-spacing: 0;
     margin-top: 9px;
+    margin-bottom: 0;
     text-transform: none;
     text-shadow: 0 1px 0 #fff;
     white-space: nowrap;
@@ -168,4 +173,4 @@ header.global.slim {
     font-weight: bold;
     letter-spacing: 0;
   }
-}
\ No newline at end of file
+}

From 7c4e20f9e5afaf0fe14110515107cd6bb2bedfcb Mon Sep 17 00:00:00 2001
From: Brian Talbot <btalbot@edx.org>
Date: Tue, 4 Jun 2013 14:45:45 -0400
Subject: [PATCH 2/2] lms - adds in alt text attribute to logo/edX home image
 for accessibility purposes

---
 lms/templates/navigation.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lms/templates/navigation.html b/lms/templates/navigation.html
index 82d08f6ca9..13c461173b 100644
--- a/lms/templates/navigation.html
+++ b/lms/templates/navigation.html
@@ -39,7 +39,7 @@ site_status_msg = get_site_status_msg(course_id)
 
   <h1 class="logo">
       <a href="${marketing_link('ROOT')}">
-        <img src="${static.url(branding.get_logo_url(request.META.get('HTTP_HOST')))}"/>
+        <img src="${static.url(branding.get_logo_url(request.META.get('HTTP_HOST')))}" alt="edX home" />
       </a></h1>
 
     % if course: