fletcher / MultiMarkdown-4

This project is now deprecated. Please use MultiMarkdown-6 instead!
https://github.com/fletcher/MultiMarkdown-5
Other
307 stars 59 forks source link

<pre markdown="1"> works incorrectly. #48

Closed van-de-bugger closed 10 years ago

van-de-bugger commented 10 years ago

Fedora 20, x86_64, multimarkdown built from fresh git sources.

$ multimarkdown --version

MultiMarkdown version 4.5
Copyright (c) 2013 Fletcher T. Penney.

portions based on peg-markdown - Copyright (c) 2008-2009 John MacFarlane.
peg-markdown is Licensed under either the GPLv2+ or MIT.
portions Copyright (c) 2011 Daniel Jalkut, MIT licensed.

This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

$ cat foo.mmd 
<pre>
*a*
</pre>
<pre markdown="1">
*a*
</pre>

$ multimarkdown foo.mmd 
<pre>
*a*
</pre>
<p re >
<em>a</em>
</pre>

Note the incorrect "<p re >" tag.

fletcher commented 10 years ago

Fixed. Thanks!

van-de-bugger commented 10 years ago

? Current result:

$ multimarkdown foo.mmd 
<pre>
*a*
</pre>

<pre markdown="1">
*a*
</pre>

It is not expected result. Attribute markdown="1" should not appear in output, but content of the second <pre> element is expected to be processed to <em>a</em>.

fletcher commented 10 years ago

Separate issues now fixed -- have to check for "pre" instead of "p"