-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Markdown: Fixed headers #1557
Markdown: Fixed headers #1557
Conversation
@@ -4,6 +4,9 @@ title 1 | |||
foobar | |||
====== | |||
|
|||
Überschrift | |||
=========== |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we add a test for the example in your PR comment?
Überschrift
-----------
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That, I cannot do because of the way .test
files are parsed, so I added a similar test.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fair enough. Looks like we had a similar test already but no harm in another.
This PR fixed markdown header of the form: ``` Überschrift ----------- ```
This PR fixed markdown header of the form
Überschrift -----------
(Example from the German Markdown Wikipedia page.)
This however also embraces the ambiguity of horizontal lines and headers. But Since a single empty line resolves the ambiguity (that is built into markdown! ), I think it's ok.