From 6974958a7539c5f1d35b7f9f0b999fde4f8e454f Mon Sep 17 00:00:00 2001
From: sokumon <sohamkulkarns9@gmail.com>
Date: Tue, 18 Feb 2025 15:35:08 +0530
Subject: [PATCH] fix: collapse button on mobile visible on scroll

---
 frappe/public/js/frappe/ui/sidebar.js         | 5 +++++
 frappe/public/js/frappe/ui/toolbar/toolbar.js | 1 +
 2 files changed, 6 insertions(+)

diff --git a/frappe/public/js/frappe/ui/sidebar.js b/frappe/public/js/frappe/ui/sidebar.js
index 979e30121f..b2c0e3a332 100644
--- a/frappe/public/js/frappe/ui/sidebar.js
+++ b/frappe/public/js/frappe/ui/sidebar.js
@@ -472,4 +472,9 @@ frappe.ui.Sidebar = class Sidebar {
 			this.setup_pages();
 		});
 	}
+	set_height() {
+		$(".body-sidebar").css("height", window.innerHeight + "px");
+		$(".overlay").css("height", window.innerHeight + "px");
+		document.body.style.overflow = "hidden";
+	}
 };
diff --git a/frappe/public/js/frappe/ui/toolbar/toolbar.js b/frappe/public/js/frappe/ui/toolbar/toolbar.js
index 9a96f1ed21..ba1c163fc7 100644
--- a/frappe/public/js/frappe/ui/toolbar/toolbar.js
+++ b/frappe/public/js/frappe/ui/toolbar/toolbar.js
@@ -29,6 +29,7 @@ frappe.ui.toolbar.Toolbar = class {
 		this.bind_events();
 		$(document).trigger("toolbar_setup");
 		$(".navbar-brand .app-logo").on("click", () => {
+			frappe.app.sidebar.set_height();
 			frappe.app.sidebar.toggle_sidebar();
 		});
 	}