daniellmb / markdownsharp

Automatically exported from code.google.com/p/markdownsharp
1 stars 0 forks source link

Invalid auto-newlines in multi-line ordered lists #38

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
Transform the following text with  AutoNewlines option set to true:

1. Line1
   Line2

   Line3
   Line4

2. Suspendisse id sem consectetuer libero luctus adipiscing

Expected output:
<ol>
<li><p>Line1<br>
Line2</p>

<p>Line3<br>
Line4</p></li>
</ol>

Actual output:
<ol>
<li><p>Line1<br
Line2</p>

<p>Line3<br
Line4</p></li>
</ol>

Original issue reported on code.google.com by eug.bara...@gmail.com on 22 Dec 2010 at 7:18

GoogleCodeExporter commented 9 years ago
Oh, just noticed that this is not a defect but rather lacking of good default 
value for  MarkdownOptions.EmptyElementSuffix

Original comment by eug.bara...@gmail.com on 22 Dec 2010 at 7:27

GoogleCodeExporter commented 9 years ago
Here's a patch that modifies the constructor to only read EmptyElementSuffix 
value from the MarkdownOptions object if MarkdownOptions.EmptyElementSuffix is 
not null or empty.

Original comment by cagdaste...@gmail.com on 4 Feb 2011 at 2:59

Attachments:

GoogleCodeExporter commented 9 years ago
nice, good catch, will add that

Original comment by wump...@gmail.com on 28 Feb 2011 at 5:01

GoogleCodeExporter commented 9 years ago
Issue 41 has been merged into this issue.

Original comment by wump...@gmail.com on 28 Feb 2011 at 5:03