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

feat: add aria-label to sidebar notification count

Browse source
This commit is contained in:
Kaushal Shriwas 2026-04-17 00:22:33 +05:30
parent 5f85777c8e
commit 7572e6fe45
2 changed files with 6 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
frappe/public/js/frappe/ui/notifications/notifications.js
View file

@ -402,9 +402,12 @@ class NotificationsView extends BaseNotificationsView {
if (!$suffix?.length) return;
if (count > 0) {
$suffix.text(count > 99 ? "99+" : count).removeClass("hidden");
$suffix
.text(count > 99 ? "99+" : count)
.attr("aria-label", __("{0} unread notifications", [count]))
.removeClass("hidden");
} else {
$suffix.addClass("hidden");
$suffix.removeAttr("aria-label").addClass("hidden");
}
}

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

@ -487,7 +487,7 @@ frappe.ui.Sidebar = class Sidebar {
standard: true,
type: "Button",
class: "sidebar-notification hidden",
suffix: "<span class='sidebar-notification-count hidden'></span>",
suffix: "<span class='sidebar-notification-count hidden' aria-live='polite'></span>",
onClick: () => {
const $dropdown = this.wrapper.find(".dropdown-notifications");
$dropdown.toggleClass("hidden");