Skip to content

HR preceeded by ***word*** doesn't render as HR #897

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

Closed
kutetapolu opened this issue Jan 24, 2020 · 1 comment · Fixed by #995
Closed

HR preceeded by ***word*** doesn't render as HR #897

kutetapolu opened this issue Jan 24, 2020 · 1 comment · Fixed by #995
Labels
bug Bug report. confirmed Confirmed bug report or approved feature request. core Related to the core parser code.

Comments

@kutetapolu
Copy link

Input:

***test***
***

Output (second line will be rendered as a literal ***):

<p><strong><em>test</em></strong>
***</p>

Expected output according to Markdown.pl:

<p><strong><em>test</em></strong>
</p>
<hr />

The same problem occurs with --- and ___ as HR.

I think it can be fixed by changing RE in HRProcessor:

497c497
<     RE = r'^[ ]{0,3}((-+[ ]{0,2}){3,}|(_+[ ]{0,2}){3,}|(\*+[ ]{0,2}){3,})[ ]*'
---
>     RE = r'^[ ]{0,3}((-+[ ]{0,2}){3,}|(_+[ ]{0,2}){3,}|(\*+[ ]{0,2}){3,})[ ]*$'
@waylan
Copy link
Member

waylan commented Jan 24, 2020

Here's what Babelmark shows. Looks like a bug.

@waylan waylan added bug Bug report. confirmed Confirmed bug report or approved feature request. core Related to the core parser code. labels Jan 24, 2020
waylan added a commit to waylan/markdown that referenced this issue Jul 1, 2020
waylan added a commit to waylan/markdown that referenced this issue Jul 1, 2020
waylan added a commit that referenced this issue Jul 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug report. confirmed Confirmed bug report or approved feature request. core Related to the core parser code.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants