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

in inline html, attribute values are incorrectly parsed as markdown #98

Open
modernserf opened this issue Feb 17, 2021 · 4 comments · May be fixed by #99
Open

in inline html, attribute values are incorrectly parsed as markdown #98

modernserf opened this issue Feb 17, 2021 · 4 comments · May be fixed by #99

Comments

@modernserf
Copy link

Test case:

snarkdown(`<a href="/foo_bar_baz.html">link</a>`)

JSFiddle link

expected result:

<a href="/foo_bar_baz.html">link</a>

actual result:

<a href="/foo<em>bar</em>baz.html">link</a>
nikku added a commit to nikku/snarkdown that referenced this issue Jun 29, 2021
@nikku
fix: do not parse Markdown inside HTML tags
374f310 
Closes developit#98
@nikku nikku linked a pull request Jun 29, 2021 that will close this issue
Open
@nikku
Copy link

nikku commented Jun 29, 2021

Should be adressed via #99.

This was referenced Jun 29, 2021
Closed
Merged
Merged
nikku added a commit to bpmn-io/snarkdown that referenced this issue Jun 30, 2021
@nikku
fix: do not parse Markdown inside HTML tags
52acf02 
Closes developit#98
@alexxlagutin
Copy link

Is there a chance that this fix will be added to npm?

@jdolderer
Copy link

It'd be great if this fix would make it to npm!

@nikku
Copy link

nikku commented Nov 18, 2021

Fixed, among other things in @bpmn-io/snarkdown.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: do not parse Markdown inside HTML tags nikku/snarkdown
4 participants
@nikku @modernserf @alexxlagutin @jdolderer