-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
linkify-code - Restore on React-based code views and diffs
#8878
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
base: main
Are you sure you want to change the base?
Conversation
ef7505b to
dea7c85
Compare
linkify-code - Restore on React-based code views and diffs
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as resolved.
This comment was marked as resolved.
|
Does this PR also fix |
This comment was marked as outdated.
This comment was marked as outdated.
|
It seems like it does actually. I'm confused |
| ':is(.snippet-clipboard-content, .highlight) > pre.notranslate', // Code blocks in comments. May be wrapped twice | ||
| '.comment-body code:not(a code, pre code)', // Inline code in comments | ||
| '.diff-text-inner', | ||
| '.react-code-text:not(.react-line-number)', |
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.
That's because I've updated codeElementsSelector
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.
@fregante should I update it in a separate PR?
fixes #6336
Test URLs
https://github.com/refined-github/refined-github/blob/be756e68dd4bf24d53e802e1fec126506979fa80/test/linkify-urls-in-code.js
https://github.com/refined-github/refined-github/pull/546/changes
Screenshot