Skip to content

show-names - Exclude username in nested item#8661

Merged
fregante merged 6 commits intorefined-github:mainfrom
kovsu:fix-show-names
Oct 2, 2025
Merged

show-names - Exclude username in nested item#8661
fregante merged 6 commits intorefined-github:mainfrom
kovsu:fix-show-names

Conversation

@kovsu
Copy link
Member

@kovsu kovsu commented Sep 5, 2025

Test URLs

https://github.com/dashboard

Screenshot

Before After
CleanShot 2025-09-05 at 11 01 34@2x CleanShot 2025-09-05 at 10 58 19@2x

@kovsu kovsu added the bug label Sep 5, 2025
@kidonng
Copy link
Member

kidonng commented Sep 6, 2025

Fix the selector instead, or update it?

// On dashboard
// `.Link--primary` excludes avatars
// [aria-label="card content"] excludes links in cards #6530 #6915
'#dashboard a.Link--primary[data-hovercard-type="user"]:not([aria-label="card content"] *)',

@kovsu
Copy link
Member Author

kovsu commented Sep 8, 2025

Better do this with js not selector i think.

@fregante fregante merged commit a9fc7c1 into refined-github:main Oct 2, 2025
9 checks passed
@kovsu kovsu deleted the fix-show-names branch October 4, 2025 15:59
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.

4 participants