-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Unexpected paragraph in unordered list #474
Comments
According to the Demo, the original parser produces almost identical results. |
I didn't expect a set of there, but perhaps a 2nd list? With the original parser there's another set of for the next list element. But I'm not sure if that's better? |
I think this borders more on expected behaviour? The markdown given is conventionally interpreted as a single list, where the third item contains some extra spacing. The commonmark reference parser produces <ul>
<li>
<p>abc</p>
</li>
<li>
<p>def</p>
</li>
<li>
<p>ghi</p>
</li>
<li>
<p>jkl</p>
</li>
<li>
<p>mno</p>
</li>
<li>
<p>pqr</p>
</li>
</ul>
<p>abc</p> Taking a look at the appropriate part of the spec
So it looks like the break between * ghi
* jkl should cause the list to become "loose", and so
I think this is expected too, the line break following "pqr" acts to "interrupt" the list, and since the interrupt is followed by some text that isn't sufficiently indented to become part of the list, the line break isn't considered to be part of the text for the final item (so no |
Thanks for providing the link to the commonmark specs. Yes, that's a behaviour I'd expect more than just one element being made paragraph. (It also showed me what I should do to separate my two lists correctly - by using a different marker.) |
Using the following Markdown:
I get paragraph tags in the "ghi" list element that I don't think should be there. Interestingly it does not get added to the 2nd list "pqr" if a paragraph follows.
The text was updated successfully, but these errors were encountered: