-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix more-conversation-filters
subscription link
#6502
Conversation
const subscriptionsLink = commentsLink.cloneNode(true); | ||
subscriptionsLink.lastChild!.textContent = 'Everything you subscribed to'; | ||
subscriptionsLink.setAttribute('aria-checked', 'false'); // #4589 | ||
const subscriptionsLink = select('#filters-select-menu a:last-child')!.cloneNode(true); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why change it? We have the same cloneNode code to create commentsLink. The error suggests that the only issue was the missing input
element, which you're also fixing below
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It should be an external link, or it will act like a bogus filter:
Screen.Recording.2023-04-12.at.22.28.32.mov
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Love it. Worth a comment though. I'd perhaps even just extract the source link to its own variable to clarify what it's targeting:
const searchSyntaxLink = select()
const subBlahBlah = searchSyntaxLink.clone(true)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can be merged after the comment nit
Fix #6501
Test URLs
https://github.com/refined-github/refined-github/issues
Screenshot
I took the chance to make some slight visual changes:
Because previously when pressing back button you see this: