Skip to content

Commit

Permalink
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix: some notifications are not shown
Browse files Browse the repository at this point in the history
tjx666 committed Aug 25, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
1 parent 6bff18d commit 45d0c46
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions source/github-notifications-preview.jsx
Original file line number Diff line number Diff line change
@@ -68,8 +68,10 @@ function updateNotificationsIndicatorStatus(show) {

async function updateUnreadCount() {
const latestStatusElement = $('.notification-indicator .mail-status', await notifications.dom);
const latestCount = $('.js-notification-inboxes .selected .count', await notifications.dom)?.textContent ?? '';
const rghCount = getRefinedGitHubUnreadCount();
const latestCount =
$('.js-notification-inboxes .selected .count', await notifications.dom)?.textContent ??
$('.js-notification-inboxes .Counter', await notifications.dom)?.textContent ??
''; const rghCount = getRefinedGitHubUnreadCount();
updateNotificationsIndicatorStatus(latestCount && (Number(latestCount) + rghCount > 0));

for (const statusElement of $$('.notification-indicator .mail-status')) {

0 comments on commit 45d0c46

Please sign in to comment.