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.

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

No closing tag li and the problem with nested lists #62

Open
FJCrux opened this issue Apr 27, 2020 · 7 comments
Open

No closing tag li and the problem with nested lists #62

FJCrux opened this issue Apr 27, 2020 · 7 comments

Comments

@FJCrux
Copy link

FJCrux commented Apr 27, 2020

Here is 2 problems:

  • Only last <li> have </li>
  • Nested list not in <li></li>
@FJCrux FJCrux changed the title Do not close the tag LI and nested list No closing tag li and the problem with nested lists Apr 27, 2020
@Kainkainkain
Copy link

Kainkainkain commented Nov 5, 2020

I have the same issue, quick solution is paste in vscode an replace open tag with </li><li> and then replace <ul></li> with just <ul> and then replace all <li></ul> with </ul>. With ignoring white space.

@pencerw
Copy link

pencerw commented Jul 9, 2021

i have this issue as well. it is indeed easy to fix in a text editor, but it would be great to have it baked into the plugin!

@pencerw
Copy link

pencerw commented Jul 9, 2021

i'm guessing, but is the issue here?

gdc.maybeCloseList = function(el) {

@joshuauyi
Copy link

I experienced this same issue. Only the last li closing tag is rendered.

@evbacher
Copy link
Owner

I believe this is fixed with recent changes. Please let us know if you see evidence of this bug still happening.

@joshuauyi
Copy link

Open and closing tags were rendered when I tested, thanks for fixing this!

NB: the Nested list not in <li></li> mentioned by the original post still exists

@evbacher
Copy link
Owner

Can any of you link a doc (view permission is fine) that illustrates the nested list issue. I'm not following exactly what the problem is (and I don't think I have any test case that shows the problem yet).

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants