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

Nasty list parsing edge case #1180

Closed
vsemozhetbyt opened this issue Mar 28, 2018 · 1 comment
Closed

Nasty list parsing edge case #1180

vsemozhetbyt opened this issue Mar 28, 2018 · 1 comment
Labels
category: lists L2 - annoying Similar to L1 - broken but there is a known workaround available for the issue

Comments

@vsemozhetbyt
Copy link
Contributor

See nodejs/node#19647

The list item number in v0.3.6 is 1, in v0.3.19 is 0, but the erroneous list is added in both cases, unlike the GitHub parser behavior.

@styfle styfle added L2 - annoying Similar to L1 - broken but there is a known workaround available for the issue category: lists labels Aug 20, 2018
@styfle
Copy link
Member

styfle commented Aug 20, 2018

I was able to reproduce with this:

Example list that does not work.

* First one goes here. We can add a new line here as long as we indent
  and the output will render as if there is no new line.
* But if we start the next line with a number followed by a period such as
  0. Then it will not render properly.

Example list that does work.

* First one goes here. We can add a new line here as long as we indent
  and the output will render as if there is no new line.
* Now if we start the next line to end with a number followed by a period such as 0.
  Then it will render properly.

Marked Demo

I haven't looked at the spec but it seems like this is ambiguous behavior since nested lists are possible.

@UziTech UziTech closed this as completed Mar 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: lists L2 - annoying Similar to L1 - broken but there is a known workaround available for the issue
Projects
None yet
Development

No branches or pull requests

3 participants