Summary

Add blank lines around lists in rendered Markdwon.

Review Request #4942 — Created Nov. 12, 2013 and submitted Nov. 12, 2013, 5:13 p.m.

Information

Owner

chipx86

Repository

Review Board

Branch

master

Bugs

Depends On

Reviewers

Groups

reviewboard

People

Description

Add blank lines around lists in rendered Markdwon.

This improves the readability of Markdown-rendered lists by adding
some spacing. There's now spacing between paragraphs within a list (in
the case of a list item with multiple paragraphs), and spacing between
list items. These help prevent things from getting too bunched up.

Testing Done

Tested with ordered and unordered lists, with sub-lists, and with multiple
paragraphs within lists. It was far more readable than before.

Files

I think that this is an improvement when list items have multiple paragraphs, but for short lists like this, I think it'll be way too big:

Item 1
Item 2
Item 3

chipx86 Nov. 12, 2013, 2:02 p.m.

Agreed. Got a new change coming out that improves this quite a bit.

Turns out if you do:

    * Item 1
    * Item 2
    * Item 3
    
It'll output:

    <ul>
     <li>Item 1</li>
     <li>Item 2</li>
     <li>Item 3</li>
    </ul>
    
But if you do something more complex, like:

    * Item 1
    
    * Item 2
    
or:

    * Item 1
    
      Paragraph
      
    * Item 2

It'll render with `<p>` tags inside the `<li>`. So, we have enough information to style better.

My update will make use of that.

Change Summary:

Kept simple lists (Ones with no paragraphs or inner content, without blank lines between bullet points) simple and bunched up next to each other. No margins.
More complex lists (with sub-items, multiple paragraphs) have the new spacing.

This does a much better job of matching what the user types.