vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Merge pull request #38896 from iamejaaz/small-ui-ux-fix

Browse source 
fix(sidebar): notification panel and icon hover state
This commit is contained in:
Ejaaz Khan 2026-04-25 16:27:45 +05:30 • committed by GitHub
commit 5b4ddb2236
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 3 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
frappe/public/js/frappe/ui/sidebar/sidebar_item.js
View file

@ -111,7 +111,7 @@ frappe.ui.sidebar_item.TypeLink = class SidebarItem {
if (frappe.utils.is_mac()) {
shortcut = shortcut.replace("Ctrl+", "⌘");
}
return `<span class="sidebar-item-suffix keyboard-shortcut">${shortcut}</span>`;
return `<span class="keyboard-shortcut">${shortcut}</span>`;
}
setup_editing_controls() {
this.menu_items = this.get_menu_items();

8
frappe/public/scss/desk/sidebar.scss
View file

@ -159,7 +159,7 @@
.standard-items-sections {
.dropdown-notifications {
.notifications-list {
left: 3.4%;
left: 100%;
}
}
}
@ -170,11 +170,6 @@
// make it an overlay on hover
position: absolute;
width: var(--sidebar-width);
.dropdown-notifications {
.notifications-list {
left: 100%;
}
}
.collapse-sidebar-link {
text-decoration: none;
@ -364,6 +359,7 @@
}
.sidebar-item-suffix {
margin-left: auto;
padding: 7px;
.keyboard-shortcut {
color: var(--ink-gray-4, #999999);
font-size: var(--text-sm);