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

Add option emptyLines #396

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

zentby
Copy link

@zentby zentby commented Sep 2, 2020

Solves #242

It introduces a new option emptyLines for default and full version.

When it is enabled, it will preserve the empty lines and convert them into <p></p>.

e.g.
Markdown

line1


line2

Before:

<p>line1</p>
<p>line2</p>

After:

<p>line1</p>
<p></p>
<p>line2</p>

Note there will be an empty line not being preserved, to be compatible for existing line terminator.

@coveralls
Copy link

coveralls commented Sep 2, 2020

Coverage Status

Coverage increased (+0.02%) to 98.569% when pulling fcc4a2f on zentby:empty_lines into 58b6945 on jonschlinkert:master.

@Dagur
Copy link

Dagur commented Sep 14, 2023

A copule of comments:

  • Doesn't work if you use "commonmark" instead of "full"
  • README needs to be updated

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.

3 participants