Skip to content

unread-anywhere - Avoid double indicator on profiles#9710

Merged
fregante merged 2 commits into
mainfrom
unread-no-double
Jun 7, 2026
Merged

unread-anywhere - Avoid double indicator on profiles#9710
fregante merged 2 commits into
mainfrom
unread-no-double

Conversation

@fregante

@fregante fregante commented Jun 7, 2026

Copy link
Copy Markdown
Member

@fregante fregante added the bug label Jun 7, 2026

@fregante fregante left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed twice

nativeLink.style.width = '30px'; // Reduce width of native button
button.setAttribute('data-variant', 'invisible'); // Enables hover style
button.classList.add(...classes);
removeNotificationIndicator(button);

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix 1

'#js-pjax-container',
'#repo-content-turbo-frame',
'#repo-content-pjax-container',
'[data-turbo-body]', // User profile

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix 2

@fregante
fregante enabled auto-merge (squash) June 7, 2026 09:04
@fregante
fregante merged commit 494c985 into main Jun 7, 2026
13 checks passed
@fregante
fregante deleted the unread-no-double branch July 10, 2026 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

unread-anywhere double circle for notifications

1 participant