From 48cfc6b5ec56ab0d6d926198ffe1206d9c91f98d Mon Sep 17 00:00:00 2001
From: Faris Ansari <netchamp.faris@gmail.com>
Date: Sat, 26 Jan 2019 16:09:11 +0530
Subject: [PATCH] fix: Cleanup navbar login

---
 .../includes/navbar/navbar_login.html         | 44 +++++++++++--------
 1 file changed, 25 insertions(+), 19 deletions(-)

diff --git a/frappe/templates/includes/navbar/navbar_login.html b/frappe/templates/includes/navbar/navbar_login.html
index 95f79c9c1a..2a58efe039 100644
--- a/frappe/templates/includes/navbar/navbar_login.html
+++ b/frappe/templates/includes/navbar/navbar_login.html
@@ -1,23 +1,29 @@
 <!-- post login tools -->
 {% if not only_static %}
-<li class="dropdown logged-in" id="website-post-login" data-label="website-post-login" style="display: none">
-	<a href="#" class="nav-item nav-link dropdown-toggle" data-toggle="dropdown">
-		<span class="user-image-wrapper"></span>
-		<span class="full-name"></span>
-		<b class="caret"></b>
-	</a>
-	<ul class="dropdown-menu dropdown-menu-right" role="menu">
-		{%- for child in post_login -%}
-		{%- if child.url -%}
-		<a class="dropdown-item" href="{{ child.url | abs_url }}" {{ child.target or '' }} rel="nofollow">
-			{{ child.label }}
+
+	{% if frappe.session.user != 'Guest' %}
+	<li class="nav-item dropdown logged-in" id="website-post-login" data-label="website-post-login" style="display: none">
+		<a href="#" class="nav-link dropdown-toggle" data-toggle="dropdown">
+			<span class="user-image-wrapper"></span>
+			<span class="full-name"></span>
+			<b class="caret"></b>
 		</a>
-		{%- endif -%}
-		{%- endfor -%}
-		<a class="dropdown-item switch-to-desk hidden" href="/desk">{{ _('Switch To Desk') }}</a>
-	</ul>
-</li>
-{% if not hide_login %}
-<a class="dropdown-item btn-login-area" href="/login">{{ _("Login") }}</a>
-{% endif %}
+		<ul class="dropdown-menu dropdown-menu-right" role="menu">
+			{%- for child in post_login -%}
+			{%- if child.url -%}
+			<a class="dropdown-item" href="{{ child.url | abs_url }}" {{ child.target or '' }} rel="nofollow">
+				{{ child.label }}
+			</a>
+			{%- endif -%}
+			{%- endfor -%}
+			<a class="dropdown-item switch-to-desk hidden" href="/desk">{{ _('Switch To Desk') }}</a>
+		</ul>
+	</li>
+	{% endif %}
+
+	{% if not hide_login %}
+	<li class="nav-item">
+		<a class="nav-link btn-login-area" href="/login">{{ _("Login") }}</a>
+	</li>
+	{% endif %}
 {% endif %}
\ No newline at end of file