Skip to content
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.

Sign up for GitHub

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

Jump to bottom

linkifyHtml: issue with specific html-escaped links #350

Closed
phoneyman opened this issue Sep 27, 2021 · 1 comment · Fixed by #354
Closed

linkifyHtml: issue with specific html-escaped links #350

phoneyman opened this issue Sep 27, 2021 · 1 comment · Fixed by #354
Assignees
@nfrasser
Labels
bug html Related to HTML parsing pending-merge
Milestone
4.0

Comments

@phoneyman
Copy link

Using linkifyHtml on an escaped string produces incorrect output. EG: linkifyHtml() called on the string '"http://google.com"' produces the following output: '"http://google.com"' which renders as "http://google.com";

The call in the code is:
var options = {/* */};
final = linkifyHtml('"http://google.com"', options);
@nfrasser nfrasser added the bug label Oct 7, 2021
@nfrasser nfrasser self-assigned this Oct 7, 2021
@nfrasser nfrasser added the html Related to HTML parsing label Oct 7, 2021
This was referenced Oct 11, 2021
Merged
Merged
@nfrasser nfrasser added this to the 4.0 milestone Oct 14, 2021
@ajbura ajbura mentioned this issue Jan 25, 2022
Closed
@kfiven kfiven mentioned this issue Feb 9, 2022
Closed
@nfrasser
Copy link
Owner

Fixed in the latest v4 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nfrasser nfrasser

Labels
bug html Related to HTML parsing pending-merge
Projects
None yet
Milestone
4.0
Development

Successfully merging a pull request may close this issue.

v4.0 nfrasser/linkifyjs
2 participants
@nfrasser @phoneyman