-
Notifications
You must be signed in to change notification settings - Fork 59
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
Only the Last List-Item Closes #15
Comments
What do you mean both lists? Also, when I convert this to HTML, I get this: <h1>List testing</h1>
<ul>
<li>Thing 1
<li>thing 2
<li>Thingy 3</li></ul> which is actually still valid HTML5, and it renders fine. I'm working on cleaning up some other issues with HTML list-processing, and that may clean up this inconsistency as well. |
I had two bullet lists, one with 3 items, and one with 2.
But had </li> only on the last item.
…On Wed, Nov 22, 2017 at 12:29 PM, Ed Bacher ***@***.***> wrote:
What do you mean both lists?
Also, when I convert this to HTML, I get this:
<h1>List testing</h1>
<ul>
<li>Thing 1
<li>thing 2
<li>Thingy 3</li></ul>
which is actually still valid HTML5, and it renders fine. I'm working on
cleaning up some other issues with HTML list-processing, and that may clean
up this inconsistency as well.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#15 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AgUVgiICDolkiEnq0sfKWQwdrRSZUAh8ks5s5IQigaJpZM4Qm2Vj>
.
|
Ah, got it. And thanks for reporting this edge case -- let me know if you notice any others. |
this is still present, and really my only complaint. I love this addon but the list item bug is really show stopping. |
Agreed - I work with a lot of documents with nested lists and this bug is a showstopper for me as well. |
I will try to spend some time on this, though Markdown is the primary target. |
I believe this is fixed with recent changes. Please let us know if you see evidence of this bug still happening. |
Happened on both lists.
The text was updated successfully, but these errors were encountered: