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.

Sign up for GitHub

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

Jump to bottom

"Loose" Lists #1

Closed
chjj opened this issue Aug 20, 2011 · 0 comments
Closed

"Loose" Lists #1

chjj opened this issue Aug 20, 2011 · 0 comments

Comments

@chjj
Copy link
Member

chjj commented Aug 20, 2011

When "loose" lists, as they're called (lists with their items separated by 2 line feeds), get implemented, it introduces some ambiguity to parsing. This is a bit of a problem with the markdown grammar itself.

* List 1
  Text
* List 1
  Text

* Loose List 1
  Text

* Loose List 1
  Text

What is that? Is it 2 separate lists? One tight, one loose? Is it three lists? One with 2 items, and 2 with one item? Is it a single list with the items spaced differently? The only solution I see, is to actually always separate consecutive lists by 3 line feeds:

* List 1
  Text
* List 1
  Text


* Loose List 1
  Text

* Loose List 1
  Text

Which would be rendered as 2 lists: one tight, one loose. This is probably what will happen once I merge the experimental branch.
@chjj chjj closed this as completed Aug 23, 2011
vinnymac pushed a commit to vinnymac/marked that referenced this issue Jul 20, 2016
@Guichaguri
Merge pull request markedjs#1 from szego/disable-elements
f62674c 
Disable elements in InlineLexer as well
matt- added a commit that referenced this issue Jul 29, 2016
@matt-
Merge pull request #1 from rsp/fix/xss_html_entities_semicolon
0fa05b6 
Optional semicolon in html entities regex
gibson042 pushed a commit to gibson042/marked that referenced this issue Oct 20, 2017
@joshbruce
Merge pull request markedjs#1 from UziTech/patch-1
77df081 
trim code after regexp
@UziTech UziTech mentioned this issue Jul 3, 2018
Merged
6 tasks
UziTech pushed a commit that referenced this issue Feb 17, 2019
@remyhunt
Merge pull request #1 from UziTech/demo-versions
02d90aa 
Demo versions
@UziTech UziTech mentioned this issue Dec 8, 2019
Merged
12 tasks
UziTech pushed a commit that referenced this issue Jul 13, 2020
@calculuschild
Merge pull request #1 from UziTech/mask-reflinks
56b6f5e 
mask reflinks in a separate function
@calculuschild calculuschild mentioned this issue May 13, 2021
Merged
5 tasks
zhenalexfan pushed a commit to zhenalexfan/MarkdownHan that referenced this issue Nov 8, 2021
@remyhunt
Merge pull request markedjs#1 from UziTech/demo-versions
a591eb3 
Demo versions
@metaskills metaskills mentioned this issue Aug 12, 2023
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@chjj