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

hoedown -> pulldown regression: missing whitespace in multi-line bullet points #41162

Closed
colin-kiegel opened this issue Apr 8, 2017 · 3 comments

Comments

@colin-kiegel
Copy link

@steveklabnik this seems to be a hoedown -> pulldown regression #40912

IN

//! * lorem ipsum
//!   dolor sit
//!   amet
//! * lorem ipsum dolor sit amet

OUT

1.18.0-nightly (2017-04-06)

* lorem ipsumdolor sitamet
* lorem ipsum dolor sit amet

1.15.0 ... 1.17.0-beta.3 (2017-04-05)

* lorem ipsum dolor sit amet
* lorem ipsum dolor sit amet
@colin-kiegel
Copy link
Author

Interestingly this does not seem to be a bug in pulldown-cmark, but maybe something post-processing related?

The doc-output of the nightly compiler is

<ul><li>lorem ipsumdolor sitamet</li><li>lorem ipsum dolor sit amet</li></ul>

While pulldown-cmark itself outputs:

<ul>
<li>lorem ipsum
dolor sit
amet</li>
<li>lorem ipsum dolor sit amet</li>
</ul>
Example Code
//! * lorem ipsum
//!   dolor sit
//!   amet
//! * lorem ipsum dolor sit amet
extern crate pulldown_cmark;
use pulldown_cmark::{html, Parser};
const MARKDOWN: &'static str = r##"
* lorem ipsum
dolor sit
amet
* lorem ipsum dolor sit amet
"##;
fn main() {mut buf = String::new();let parser = Parser::new(MARKDOWN);
  html::push_html(&mut buf, parser);println!("{}", buf);
}

@ollie27
Copy link
Member

ollie27 commented Apr 8, 2017

This was fixed by #41112. You'll need to update to the latest nightly.

@colin-kiegel
Copy link
Author

Right. That was a quick fix - thanks. :-)

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

No branches or pull requests

2 participants