Skip to content

Comments

linkify-user-labels - Disable for viewer member label, enable for collaborator label#8997

Open
SunsetTechuila wants to merge 6 commits intomainfrom
user-labels
Open

linkify-user-labels - Disable for viewer member label, enable for collaborator label#8997
SunsetTechuila wants to merge 6 commits intomainfrom
user-labels

Conversation

@SunsetTechuila
Copy link
Member

@SunsetTechuila SunsetTechuila marked this pull request as ready for review February 20, 2026 17:50

function init(signal: AbortSignal): void {
observe([
'span[data-testid="comment-author-association"][aria-label*="a member of the"]',
Copy link
Member Author

@SunsetTechuila SunsetTechuila Feb 20, 2026

Choose a reason for hiding this comment

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

I don't know what was the reason to use [aria-label*="a member of the"] instead of [aria-label^="This user is a member"].

It was like that from the start: https://github.com/refined-github/refined-github/pull/5765/changes/2c2c77c85ffacb28d1f7ebae7e8140cfc0938765#diff-964f0dc23f2488134d464600f151a7b051cd026e2de58e6899c3b7d27e391f77R11

Copy link
Member Author

@SunsetTechuila SunsetTechuila Feb 20, 2026

Choose a reason for hiding this comment

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

I doubt that the tooltip text used to start with a username

Copy link
Member

Choose a reason for hiding this comment

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

The links in the original PR might have the answer: #5765

Do comments and PRs by bots/apps say "this user"?

Copy link
Member Author

@SunsetTechuila SunsetTechuila Feb 21, 2026

Choose a reason for hiding this comment

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

Do comments and PRs by bots/apps say "this user"?

Yes

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants