From ca685cb11fc01822c180326113c746727e304121 Mon Sep 17 00:00:00 2001
From: Ejaaz Khan <ejaaz@frappe.io>
Date: Thu, 29 Jan 2026 15:50:18 +0530
Subject: [PATCH] fix(navbar): remove extra navbar on impersonate

---
 frappe/public/js/frappe/ui/page.html | 24 ++++++++++++------------
 frappe/public/js/frappe/ui/page.js   |  2 --
 frappe/public/scss/desk/page.scss    |  3 ---
 3 files changed, 12 insertions(+), 17 deletions(-)

diff --git a/frappe/public/js/frappe/ui/page.html b/frappe/public/js/frappe/ui/page.html
index 039f564f03..4f42763731 100644
--- a/frappe/public/js/frappe/ui/page.html
+++ b/frappe/public/js/frappe/ui/page.html
@@ -1,4 +1,4 @@
-<div class="page-head flex {% if frappe.boot.read_only || frappe.boot.user.impersonated_by %}show-navbar{% endif %}">
+<div class="page-head flex">
 	<div class="container">
 		<div class="row flex-nowrap align-center page-head-content justify-between">
 			<div class="page-title">
@@ -39,6 +39,17 @@
 				{% endif %}
 				<div class="flex col page-actions justify-content-end {% if frappe.is_mobile() %} pl-0 {% endif %}">
 					<!-- buttons -->
+					{% if (frappe.boot.read_only) { %}
+					<span class="mr-3 indicator-pill yellow no-indicator-dot read-only-banner" title="{%= __(" Your site is undergoing
+						maintenance or being updated.") %}">
+						{%= __("Read Only Mode") %}
+					</span>
+					{% } %}
+					{% if (frappe.boot.user.impersonated_by) { %}
+					<span class="mr-3 indicator-pill red no-indicator-dot" title="{%= __(" You are impersonating as another user.") %}">
+						{%= __("Impersonating {0}", [frappe.boot.user.name]) %}
+					</span>
+					{% } %}
 					<div class="filters flex"></div>
 					<div class="custom-actions hide hidden-xs hidden-md"></div>
 					<div class="standard-actions flex">
@@ -72,17 +83,6 @@
 						</div>
 						<button class="btn btn-primary btn-sm hide primary-action"></button>
 					</div>
-					{% if (frappe.boot.read_only) { %}
-					<span class="ml-3 indicator-pill yellow no-indicator-dot read-only-banner" title="{%= __(" Your site is undergoing
-						maintenance or being updated.") %}">
-						{%= __("Read Only Mode") %}
-					</span>
-					{% } %}
-					{% if (frappe.boot.user.impersonated_by) { %}
-					<span class="ml-3 indicator-pill red no-indicator-dot" title="{%= __(" You are impersonating as another user.") %}">
-						{%= __("Impersonating {0}", [frappe.boot.user.name]) %}
-					</span>
-					{% } %}
 				</div>
 			</div>
 		</div>
diff --git a/frappe/public/js/frappe/ui/page.js b/frappe/public/js/frappe/ui/page.js
index 9b8ba5b319..2902825fa9 100644
--- a/frappe/public/js/frappe/ui/page.js
+++ b/frappe/public/js/frappe/ui/page.js
@@ -67,8 +67,6 @@ frappe.ui.Page = class Page {
 					(frappe.boot.read_only || frappe.boot.user.impersonated_by)
 				) {
 					$(".page-head").css("top", "-15px");
-				} else if (frappe.boot.read_only || frappe.boot.user.impersonated_by) {
-					$(".page-head").css("top", "var(--navbar-height)");
 				}
 				last_scroll = current_scroll;
 			}, 500)
diff --git a/frappe/public/scss/desk/page.scss b/frappe/public/scss/desk/page.scss
index 3f0591442a..0a904d2da8 100644
--- a/frappe/public/scss/desk/page.scss
+++ b/frappe/public/scss/desk/page.scss
@@ -111,9 +111,6 @@ body[data-route^="Form"] {
 	border-bottom: 1px solid var(--border-color);
 	transition: 0.5s top;
 	top: 0;
-	&.show-navbar {
-		top: var(--navbar-height);
-	}
 
 	.page-head-content {
 		height: var(--page-head-height);