Inconsistent hard line break behavior in GFM (GitHub Flavored Markdown) parser about kramdown

[gettalong]

Actually, Github uses both, hard line breaks and the normal Markdown behaviour.
For example, in this comment line breaks are automatically converted to hard
line
breaks.

Whereas in other parts (e.g. conversion of README files) hard line breaks are not used.

Maybe an option to turn off hard line breaks in GFM would be useful?

[plexus]

I based myself on the documentation at https://help.github.com/articles/github-flavored-markdown

But since it's apparently not consitently used everywhere I agree a switch would be useful.

[brandur]

Good call guys. I've opened a basic pull materializing this idea at #83. Thanks.

[gettalong]

@brandur Thanks for the pull request. Now, what should the default value for this be? Hard line breaks as in the documentation link provided by @plexus or default Markdown behaviour?

[plexus]

I would say take the documented behavior as default. That way if people are
already using GFM it doesn't change how their applications behave.
On 26 Oct 2013 09:08, "Thomas Leitner" [email protected] wrote:

@brandur https://github.com/brandur Thanks for the pull request. Now,
what should the default value for this be? Hard line breaks as in the
documentation link provided by @plexus https://github.com/plexus or
default Markdown behaviour?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/82#issuecomment-27141009
.

[brandur]

@gettalong @plexus Thanks for taking a look guys. I was originally trying to make the interface look like Redcarpet's, where a flag has to be set for hard wrapping to come into effect. +1 on making it the documented default though, if only so that the behavior for existing users doesn't change under them. I've updated my pull to toggle the default.

Slightly unrelated, I added another set of tests, but was slightly surprised when they didn't fail because nothing written in Ruby ever works on the first try. Am I doing it right?

[gettalong]

Thanks @brandur for the pull request. I have merged your code now.