Add blank lines around lists in rendered Markdwon.




 


  Review Request #4942
  —
  Created Nov. 12, 2013 and submitted 

 















 
  
  
 




 
  

   Information

  

 
 

  

   
   
chipx86


  


  

   
   
Review Board


  


  

   
   
master


  


  

   
   



  


  

   
   



  


 



 
  

   Reviewers

  

 
 

  

   
   
reviewboard


  


  

   
   



  


 











 

  
  


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.




 




 

  
  


Tested with ordered and unordered lists, with sub-lists, and with multiple

paragraphs within lists. It was far more readable than before.




 














 

 

  
  


   

    

     

     

     

    

   


   

    

     

     

     

    

   


   

  

 















  

 








 














 





 

  

   




david




   

   





  

 



 

  


   

   

    
 
david


    
   


  

  

  



    




  1. 
 
    
  
    
   
    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
      • 

    
  
    
 
    

 
    
  


    
 
    
 
      

  


    1. 
 
 
      
 
      
  
 
      
 
      
  
      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.
      
 
      

      2. 


 
    


    

 
    

    2. 















  2. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    3. 




  

 
















 



 

  

   




chipx86




   

   

  

 



 















 





 

  

   




david




   

   







  

 



 

  


   

   

    
 
david


    
   


  

  

  



    




  1. 
 
    
  
    
   
    Ship It!
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    2. 















  2. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    3. 




  

 
















 



 

  

   




chipx86




   

   

  

 



 

  


   

   

    
 Review request changed

    
   


  

  

  






 

  Status:
  

  Closed (submitted)

  
 







 


 










  

 









     

    

   

  

  

   
   Loading...