From 897815425c0beabf804716e65a847ac4d525df63 Mon Sep 17 00:00:00 2001 From: KerollesFathy Date: Sat, 27 Dec 2025 19:02:15 +0000 Subject: [PATCH 1/3] fix: search kayboard shortcut --- frappe/public/js/frappe/ui/sidebar/sidebar.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/frappe/public/js/frappe/ui/sidebar/sidebar.js b/frappe/public/js/frappe/ui/sidebar/sidebar.js index 1485c92062..2eec78471f 100644 --- a/frappe/public/js/frappe/ui/sidebar/sidebar.js +++ b/frappe/public/js/frappe/ui/sidebar/sidebar.js @@ -248,7 +248,7 @@ frappe.ui.Sidebar = class Sidebar { type: "Button", id: "navbar-modal-search", suffix: { - keyboard_shortcut: "CtrlK", + keyboard_shortcut: "Ctrl+K", }, class: "navbar-search-bar hidden", }); From c98cfd0c49d12e703f552ae1ab46ff8fb90a6caf Mon Sep 17 00:00:00 2001 From: KerollesFathy Date: Sat, 27 Dec 2025 19:09:16 +0000 Subject: [PATCH 2/3] fix: add space --- frappe/public/js/frappe/ui/sidebar/sidebar.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/frappe/public/js/frappe/ui/sidebar/sidebar.js b/frappe/public/js/frappe/ui/sidebar/sidebar.js index 2eec78471f..1d574c0420 100644 --- a/frappe/public/js/frappe/ui/sidebar/sidebar.js +++ b/frappe/public/js/frappe/ui/sidebar/sidebar.js @@ -248,7 +248,7 @@ frappe.ui.Sidebar = class Sidebar { type: "Button", id: "navbar-modal-search", suffix: { - keyboard_shortcut: "Ctrl+K", + keyboard_shortcut: "Ctrl K", }, class: "navbar-search-bar hidden", }); From 26d4d738fabfa128defada66c8e244d553d83e46 Mon Sep 17 00:00:00 2001 From: sokumon Date: Sun, 28 Dec 2025 13:56:03 +0530 Subject: [PATCH 3/3] fix: use correct shortcut label in sidebar according to OS --- frappe/public/js/frappe/ui/sidebar/sidebar.js | 2 +- frappe/public/js/frappe/ui/sidebar/sidebar_item.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/frappe/public/js/frappe/ui/sidebar/sidebar.js b/frappe/public/js/frappe/ui/sidebar/sidebar.js index 1d574c0420..2eec78471f 100644 --- a/frappe/public/js/frappe/ui/sidebar/sidebar.js +++ b/frappe/public/js/frappe/ui/sidebar/sidebar.js @@ -248,7 +248,7 @@ frappe.ui.Sidebar = class Sidebar { type: "Button", id: "navbar-modal-search", suffix: { - keyboard_shortcut: "Ctrl K", + keyboard_shortcut: "Ctrl+K", }, class: "navbar-search-bar hidden", }); diff --git a/frappe/public/js/frappe/ui/sidebar/sidebar_item.js b/frappe/public/js/frappe/ui/sidebar/sidebar_item.js index 3b8d7fb038..837466bb60 100644 --- a/frappe/public/js/frappe/ui/sidebar/sidebar_item.js +++ b/frappe/public/js/frappe/ui/sidebar/sidebar_item.js @@ -84,7 +84,7 @@ frappe.ui.sidebar_item.TypeLink = class SidebarItem { } get_shortcut_html(shortcut) { if (frappe.utils.is_mac()) { - shortcut = shortcut.replace("Ctrl", "⌘"); + shortcut = shortcut.replace("Ctrl+", "⌘"); } return `${shortcut}`; }