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

Newlines converted to BR tags #63

Closed
barraponto opened this issue Nov 13, 2013 · 3 comments
Closed

Newlines converted to BR tags #63

barraponto opened this issue Nov 13, 2013 · 3 comments

Comments

@barraponto
Copy link

@barraponto barraponto commented Nov 13, 2013

Gist.io is currently converting newlines in markdown to BR tags, but that's not the behavior that Github uses in http://gist.github.com

Examples: https://gist.github.com/barraponto/7450314/ and http://gist.io/7450314

@barraponto
Copy link
Author

@barraponto barraponto commented Nov 14, 2013

This issue stems from the commit 0252440 which starts rendering stuff as GithubFormattedMarkdown. As the documentation says:

In GFM mode, hard line breaks are always taken into account, and issue and user mentions are linked accordingly.

However, I don't see a reason to use such mode as the default. Neither does the Github API, which defaults to regular markdown. Of course, having a context to reference issues, users and commits would be great, provided there was some metadata to inform the Markdown API. Otherwise, it'd be best to stick to regular Markdown.

@barraponto
Copy link
Author

@barraponto barraponto commented Nov 14, 2013

Oh and I have to reinforce that regular line breaks in markdown were always ignored, which is great for the 80-chracter-line geeks (like myself). It might make sense for Github textfields, but not for text-editor/code-editor environments.

@Potherca
Copy link

@Potherca Potherca commented Jun 1, 2019

@idan ✔️ Issue can be closed.

Looking at https://gist.io/@barraponto/7450314/ everything looks like it is rendered as expected (i.e. newlines are ignored as in the Gist).

@barraponto barraponto closed this Jun 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.