From bacb0c056a240c2bfac6af4dfefd3f4d07462dcf Mon Sep 17 00:00:00 2001
From: Sunil Mohan Adapa <sunil@medhas.org>
Date: Tue, 15 Jan 2019 14:44:54 -0800
Subject: [PATCH] ui: Underline the logo along with 'Home' text when active

- This works properly in mobile sizes by underlining only the logo.

Signed-off-by: Sunil Mohan Adapa <sunil@medhas.org>
Reviewed-by: Joseph Nuthalapati <njoseph@thoughtworks.com>
---
 plinth/templates/base.html           | 5 +++--
 static/themes/default/css/plinth.css | 1 +
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/plinth/templates/base.html b/plinth/templates/base.html
index 2e6ea7687..cbe9fd60d 100644
--- a/plinth/templates/base.html
+++ b/plinth/templates/base.html
@@ -99,8 +99,9 @@
           <span class="icon-bar"></span>
         </button>
         {% block mainmenu_left %}
-          <a href="{% url 'index' %}"  class="navbar-brand"
-             title="{{ box_name }}">
+          <a href="{% url 'index' %}"  class="navbar-brand
+                   {% if not submenu.url %} menu_link_active {% else %}
+                   menu_link {% endif %}" title="{{ box_name }}">
             <i class="fa fa-freedombox fa-2x fa-inverse" aria-hidden="true"></i>
           </a>
         {% endblock %}
diff --git a/static/themes/default/css/plinth.css b/static/themes/default/css/plinth.css
index d614fec97..1648fa626 100644
--- a/static/themes/default/css/plinth.css
+++ b/static/themes/default/css/plinth.css
@@ -78,6 +78,7 @@ body {
 }
 
 .navbar-brand {
+    height: 53px;
     margin-right: -30px;
 }