From 34d0dd9d6e074e41968c39247d66e19fd551c57b Mon Sep 17 00:00:00 2001
From: Alanna Scott <alanna.scott@airbnb.com>
Date: Wed, 7 Dec 2016 11:24:54 -0800
Subject: [PATCH] adjust header nav links so they are all aligned on the base
 line (#1786)

---
 .../assets/stylesheets/less/cosmo/bootswatch.less    |  4 ++++
 superset/templates/appbuilder/navbar.html            | 12 ++++++++----
 superset/templates/appbuilder/navbar_right.html      |  9 +++++++--
 3 files changed, 19 insertions(+), 6 deletions(-)

diff --git a/superset/assets/stylesheets/less/cosmo/bootswatch.less b/superset/assets/stylesheets/less/cosmo/bootswatch.less
index 20ee0f4ed913c..feea7ce413263 100644
--- a/superset/assets/stylesheets/less/cosmo/bootswatch.less
+++ b/superset/assets/stylesheets/less/cosmo/bootswatch.less
@@ -40,6 +40,10 @@
   background: transparent;
 }
 
+.navbar-nav > li > a {
+  padding-top: 18px;
+}
+
 // Buttons ====================================================================
 
 .btn-default:hover {
diff --git a/superset/templates/appbuilder/navbar.html b/superset/templates/appbuilder/navbar.html
index 78d1823e0bf1a..38bb5037d1b17 100644
--- a/superset/templates/appbuilder/navbar.html
+++ b/superset/templates/appbuilder/navbar.html
@@ -21,11 +21,15 @@
         {% include 'appbuilder/navbar_menu.html' %}
       </ul>
       <ul class="nav navbar-nav navbar-right">
-        <li><a href="https://github.com/airbnb/superset" title="Superset's Github">
-          <i class="fa fa-github fa-lg"></i></a>
+        <li>
+          <a href="https://github.com/airbnb/superset" title="Superset's Github">
+            <i class="fa fa-github"></i> &nbsp;
+          </a>
         </li>
-        <li><a href="http://airbnb.io/superset" title="Documentation">
-          <i class="fa fa-book fa-lg"></i> </a>
+        <li>
+          <a href="http://airbnb.io/superset" title="Documentation">
+            <i class="fa fa-book"></i> &nbsp;
+          </a>
         </li>
         {% include 'appbuilder/navbar_right.html' %}
       </ul>
diff --git a/superset/templates/appbuilder/navbar_right.html b/superset/templates/appbuilder/navbar_right.html
index 7a07f25cb863f..d6d844dad6aae 100644
--- a/superset/templates/appbuilder/navbar_right.html
+++ b/superset/templates/appbuilder/navbar_right.html
@@ -29,8 +29,13 @@
 
 {% if not current_user.is_anonymous() %}
     <li class="dropdown">
-      <a class="dropdown-toggle" data-toggle="dropdown" title="{{g.user.get_full_name()}}" href="#">
-        <span class="fa fa-user fa-lg" title=""></span><b class="caret"></b>
+      <a
+        class="dropdown-toggle"
+        data-toggle="dropdown"
+        title="{{g.user.get_full_name()}}"
+        href="javascript:void(0)"
+      >
+        <i class="fa fa-user"></i>&nbsp;<b class="caret"></b>
       </a>
         <ul class="dropdown-menu">
             <li><a href="{{appbuilder.get_url_for_userinfo}}"><span class="fa fa-fw fa-user"></span>{{_("Profile")}}</a></li>