You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
For emphasis, using asterisks to bold and italicize the middle of a word doesn't work. #557
Can I please get some help on this ? I was looking at options on Parser but don't see any thing that can help.
Note that it works with https://github.com/commonmark/commonmark-java
Activity
arjun2 commentedon Feb 28, 2023
Can I please get some help on this ? I was looking at options on Parser but don't see any thing that can help.
Note that it works with https://github.com/commonmark/commonmark-java
david-waltermire commentedon Jun 10, 2023
@arjun2 I am seeing the same. This is example 415 from the CommonMark specification.
A parser creates the following AST:
Notice the missing StrongEmphasis and Emphasis nodes.
This seems like a bug in Flexmark.
david-waltermire commentedon Jun 10, 2023
Tracing this, I believe the problem relates to the code at:
https://github.com/vsch/flexmark-java/blame/cc3a2f59ba6e532833f4805f8134b4dc966ff837/flexmark/src/main/java/com/vladsch/flexmark/parser/internal/InlineParserImpl.java#L1418
It looks like the loop here is rewinding the stack to find the outer most opener and closer. Then processing the result.
This code is a bit complex, so I am not fully sure what is happening, but this seems to result in the delimiters getting ignored.
david-waltermire commentedon Jun 10, 2023
@vsch Is there a possibility of prioritizing a fix? I am not sure I am able to fully debug this to create a PR or I would.
Adjusted the use of delimiters to allow the triple case to work per C…
Removed unneeded CommonMark unit test fixes. All but 1 of the CommonM…
Completed work on a CommonMark-based markdown unit test suite.
david-waltermire commentedon Jun 24, 2023
I was able to create a PR after all. Let me know if this works or needs adjustments.
Completed work on a CommonMark-based markdown unit test suite.
Expand support for Markdown to HTML conversion (#159)