uncorrect parsing result with very rare case #335

Closed
opened 2026-01-29 14:34:04 +00:00 by claunia · 4 comments
Owner

Originally created by @kflili on GitHub (Oct 24, 2019).

Two cases:
Hello eudev@**microsoft.**com
Hello benefit of **$**225
image

Please the test link in below.
https://babelmark.github.io/?text=Hello+eudev%40microsoft.com%0A%0A+%0AHello+benefit+of+%24225

Originally created by @kflili on GitHub (Oct 24, 2019). Two cases: Hello eudev@**microsoft.**com Hello benefit of **$**225 ![image](https://user-images.githubusercontent.com/19175478/67444899-bb36cc00-f5bf-11e9-9ec6-88b5be2f484a.png) Please the test link in below. https://babelmark.github.io/?text=Hello+eudev%40**microsoft.**com%0A%0A+%0AHello+benefit+of+**%24**225
Author
Owner

@xoofx commented on GitHub (Oct 24, 2019):

It is correct for all CommonMark implementations and part of the specifications.

@xoofx commented on GitHub (Oct 24, 2019): It is correct for all CommonMark implementations and part of the specifications.
Author
Owner

@kflili commented on GitHub (Oct 24, 2019):

CommonMark implementations and part of the specifications

@xoofx Thanks for the reply. Does that mean it meet the markdown standard, and should render as such? Our user want to get parsing upper results, not the lower results in the screenshot.

Any work around for this special case?

Thanks very much!

@kflili commented on GitHub (Oct 24, 2019): > CommonMark implementations and part of the specifications @xoofx Thanks for the reply. Does that mean it meet the markdown standard, and should render as such? Our user want to get parsing upper results, not the lower results in the screenshot. Any work around for this special case? Thanks very much!
Author
Owner

@xoofx commented on GitHub (Oct 24, 2019):

Yes, it is the standard, so we can't change it. You can see also that on GitHub it is rendered like Markdig (because now GitHub is using a CommonMark compliant parser)

Hello eudev@**microsoft.**com
Hello benefit of **$**225

Any work around for this special case?

Apart developing a new emphasis parser by forking the default extension and replacing with your own, there is no easy workaround

@xoofx commented on GitHub (Oct 24, 2019): Yes, it is the standard, so we can't change it. You can see also that on GitHub it is rendered like Markdig (because now GitHub is using a CommonMark compliant parser) Hello eudev@**microsoft.**com Hello benefit of **$**225 > Any work around for this special case? Apart developing a new emphasis parser by forking the default extension and replacing with your own, there is no easy workaround
Author
Owner

@kflili commented on GitHub (Oct 24, 2019):

Thanks @xoofx for the clear answer!

@kflili commented on GitHub (Oct 24, 2019): Thanks @xoofx for the clear answer!
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/markdig#335