A cleaner right side of the navbar with a Github link (#666)

apache · Jun 23, 2016 · f949b88 · f949b88
1 parent ab71ee4
commit f949b88
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 11 deletions.
diff --git a/caravel/templates/appbuilder/navbar.html b/caravel/templates/appbuilder/navbar.html
@@ -21,7 +21,12 @@
         {% include 'appbuilder/navbar_menu.html' %}
       </ul>
       <ul class="nav navbar-nav navbar-right">
-        <li><a href="http://airbnb.io/caravel"><i class="fa fa-book"></i> {{_("Documentation") }}</a></li>
+        <li><a href="https://github.com/airbnb/caravel" title="Caravel's Github">
+          <i class="fa fa-github fa-lg"></i></a>
+        </li>
+        <li><a href="http://airbnb.io/caravel" title="Documentation">
+          <i class="fa fa-book fa-lg"></i> </a>
+        </li>
         {% include 'appbuilder/navbar_right.html' %}
       </ul>
     </div>

diff --git a/caravel/templates/appbuilder/navbar_right.html b/caravel/templates/appbuilder/navbar_right.html
@@ -10,26 +10,28 @@
        </div>
     </a>
     <ul class="dropdown-menu">
-    <li class="dropdown">
+      <li class="dropdown">
         {% for lang in languages %}
             {% if lang != locale %}
                 <a tabindex="-1" href="{{appbuilder.get_url_for_locale(lang)}}">
-                  <div class="f16"><i class="flag {{languages[lang].get('flag')}}"></i> - {{languages[lang].get('name')}}
-                </div></a>
-                {% endif %}
-            {% endfor %}
-        </li>
-        </ul>
+                  <div class="f16">
+                    <i class="flag {{languages[lang].get('flag')}}"></i> - {{languages[lang].get('name')}}
+                  </div>
+                </a>
+            {% endif %}
+        {% endfor %}
+      </li>
+    </ul>
 </li>
 {% endif %}
 
 
 
 {% if not current_user.is_anonymous() %}
     <li class="dropdown">
-        <a class="dropdown-toggle" data-toggle="dropdown" href="#">
-           <span class="fa fa-user"></span> {{g.user.get_full_name()}}<b class="caret"></b>
-        </a>
+      <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>
         <ul class="dropdown-menu">
             <li><a href="{{appbuilder.get_url_for_userinfo}}"><span class="fa fa-fw fa-user"></span>{{_("Profile")}}</a></li>
             <li><a href="{{appbuilder.get_url_for_logout}}"><span class="fa fa-fw fa-sign-out"></span>{{_("Logout")}}</a></li>