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

Putting slighlty misaligned blockquote in a list #62

Open
BigBadaboom opened this Issue Jul 22, 2015 · 5 comments

Comments

Projects
None yet
2 participants
@BigBadaboom
Copy link

commented Jul 22, 2015

Is the following a bug in commonmark.js? The behaviour seems to disobey rule 4 of List Items.

 > Blockquote
> continued here.


1.  > Blockquote
   > continued here.

Dingus here.

@jgm

This comment has been minimized.

Copy link
Member

commented Jul 23, 2015

Rule 4 says: " If a sequence of lines Ls constitutes a list item according to rule #1, #2, or #3, then the result of indenting each line of Ls by 1-3 spaces (the same for each line) also constitutes a list item with the same contents and attributes." I don't see how this applies here. Your list item is not indented.

@BigBadaboom

This comment has been minimized.

Copy link
Author

commented Jul 24, 2015

Sorry. Please close this. It was the result of a late night brain fail on my part.

@jgm jgm closed this Jul 24, 2015

@jgm jgm reopened this Jul 25, 2015

@jgm

This comment has been minimized.

Copy link
Member

commented Jul 25, 2015

As far as I can see, the current behavior respects all of the spec's list rules. However, it does violate an invariant I'd hoped to achieve, the Principle of Uniformity. So it might be worth thinking about how the spec might be adjusted to allow this case to have its more natural interpretation.

@jgm

This comment has been minimized.

Copy link
Member

commented Jul 25, 2015

Here is another, similar violation of the principle of uniformity:
http://spec.commonmark.org/dingus/?text=%20%20oo%0A--%0A%0A-%20%20%20oo%0A%20%20--%0A

@jgm

This comment has been minimized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.