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

Markdown: Fixed headers #1557

Merged
merged 7 commits into from
Nov 28, 2018
Merged

Conversation

RunDevelopment
Copy link
Member

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.

@@ -4,6 +4,9 @@ title 1
foobar
======

Überschrift
===========
Copy link
Member

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
-----------

Copy link
Member Author

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.

Copy link
Member

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.

@mAAdhaTTah mAAdhaTTah merged commit c658429 into PrismJS:master Nov 28, 2018
@RunDevelopment RunDevelopment deleted the markdown-header-fix branch November 28, 2018 15:43
ggrossetie pushed a commit to ggrossetie/prism that referenced this pull request Mar 11, 2019
This PR fixed markdown header of the form:

```
Überschrift
-----------
```
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

Successfully merging this pull request may close these issues.

2 participants