From 842921c5879f9c795f5af254ad8a57edd34db626 Mon Sep 17 00:00:00 2001
From: KerollesFathy <kerollesfhabib@gmail.com>
Date: Mon, 6 Apr 2026 15:18:15 +0000
Subject: [PATCH] fix(ux): add confirm before logout

Note:
This not a new fix it's already merged before in #37880
But removed by mistake in this #38192 so this for make it come back :)
---
 frappe/public/js/frappe/desk.js | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/frappe/public/js/frappe/desk.js b/frappe/public/js/frappe/desk.js
index e27968fac2..1aea6a02a9 100644
--- a/frappe/public/js/frappe/desk.js
+++ b/frappe/public/js/frappe/desk.js
@@ -380,15 +380,17 @@ frappe.Application = class Application {
 	logout() {
 		var me = this;
 		me.logged_out = true;
-		return frappe.call({
-			method: "logout",
-			callback: function (r) {
-				if (r.exc) {
-					return;
-				}
+		frappe.confirm(__("Are you sure you want to log out?"), function () {
+			return frappe.call({
+				method: "logout",
+				callback: function (r) {
+					if (r.exc) {
+						return;
+					}
 
-				me.redirect_to_login();
-			},
+					me.redirect_to_login();
+				},
+			});
 		});
 	}
 	handle_session_expired() {