-
Notifications
You must be signed in to change notification settings - Fork 30
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
Incorrectly detects text as URL #39
Comments
Related to this. Why is the default fallback reg-exp this one?
I find it problematic in that the following text is detected as a URL when clearly it only contains an URL as part of some text:
This is inconvinient, as it prevents me from simply pasting larger bodies of text containing URLs. Pasting it becomes this:
Notice the extra However, if I use this reg-exp instead - which essentially just adds a
then the above copy-statement simply works. (Edit: Fixed ^) What speaks against using this new reg-exp? I think it's very common, that users of this plugin will copy larger bodies with contained URLs - and in this case I think the standard behavior should be, that it is not detected as an URL. What do you think? |
@HaleTom To answer the original question: The < and > are Markdown syntax to specify auto links. @GollyTicker Makes sense. However, it should be a |
Thanks for the correction. Could we adapt this reg exp in the plugin to change the default behavior? Adding test cases would also be nice, as I see that there are not testcases currently. I could offer help, if you want. |
The Regex here for catching links is very buggy. Both the primary method to catch URIs and the One circumvention of these bugs is to, for the setting |
I'm pasting my public key signature:
What gets inserted:
I'm curious as to why the
<
and>
are in there.My config:
The text was updated successfully, but these errors were encountered: