Skip to content

new-tab-links - Improve detection#9640

Merged
fregante merged 5 commits into
mainfrom
new-modal
May 30, 2026
Merged

new-tab-links - Improve detection#9640
fregante merged 5 commits into
mainfrom
new-modal

Conversation

@fregante

@fregante fregante commented May 30, 2026

Copy link
Copy Markdown
Member

The modal links only use the pathname, but sort-conversations-by-update-time swoops in and adds the query to these dead links.

Test URLs

https://github.com/refined-github/refined-github/issues?q=sort%3Aupdated-desc+is%3Aissue+is%3Aopen+

Screenshot

(notice no link in status bar)

Screenshot 13

@fregante fregante added the bug label May 30, 2026
@fregante fregante marked this pull request as ready for review May 30, 2026 08:32
@fregante fregante changed the title sort-conversations-by-update-time new-tab-links Fix conflict new-tab-links - Improve detection May 30, 2026
@fregante fregante linked an issue May 30, 2026 that may be closed by this pull request
1 task
@fregante

Copy link
Copy Markdown
Member Author

Went ahead and fixed #9641 as well.

@fregante fregante enabled auto-merge (squash) May 30, 2026 08:39
@fregante fregante merged commit b27be0a into main May 30, 2026
12 checks passed
@fregante fregante deleted the new-modal branch June 1, 2026 06: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.

new-tab-links only works until the next Ajax navigation

1 participant