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

Improve nested list support. #14

Merged
merged 5 commits into from
Oct 6, 2014
Merged

Conversation

dmitshur
Copy link
Member

This is a work in progress.

It fixes shurcooL-legacy/atom-markdown-format#7.

This has no effect on rendered HTML.
However, it is a change that will allow for improved and correct
support of multiple nested lists with rich content inside.
In the end, this is a formatting change that "makes the most sense"
when considering all of Markdown features and how they interact.
It also more closely matches the original nested lists examples in
Markdown description (see
http://daringfireball.net/projects/markdown/syntax#list).
Do not include $ in installation steps.
Use new WIP list item formatting.
…pport.

Also better support for large spacing (via paragraphs) between list
items.
The implementation is still quite messy because the rules for Markdown
lists and the parser output is quite messy.
@dmitshur
Copy link
Member Author

dmitshur commented Oct 5, 2014

The PR is done now and should be ready to merge. It fixes the issue as far as I can tell.

All tests pass.

If anyone wants to help, please test it and report any inputs on which it fails to produce sane results.

You can try it out in your browser here: https://dl.dropboxusercontent.com/u/8554242/dmitri/projects/live-markdownfmt/live-markdownfmt.html

dmitshur added a commit that referenced this pull request Oct 6, 2014
@dmitshur dmitshur merged commit ce1ca34 into master Oct 6, 2014
@dmitshur dmitshur deleted the improve-nested-list-support branch October 6, 2014 05:54
bwplotka pushed a commit to bwplotka/markdownfmt that referenced this pull request Jun 16, 2021
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

Successfully merging this pull request may close these issues.

Can't handle "blockquote", "code blocks" and "table" within a list.
1 participant