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

FR: <br> as option for Two Spaces Between Lines with Content #933

Closed
HemanthJabalpuri opened this issue Oct 31, 2023 · 4 comments · Fixed by #1058
Closed

FR: <br> as option for Two Spaces Between Lines with Content #933

HemanthJabalpuri opened this issue Oct 31, 2023 · 4 comments · Fixed by #1058
Assignees
Labels
markdown General Markdown or Markdown related issue or feature resolution/update-made A change has been made that should resolve this issue or request rule suggestion Suggestion to add or edit a rule

Comments

@HemanthJabalpuri
Copy link

Is Your Feature Request Related to a Problem? Please Describe.

It is not a problem, but <br> can be used for better readability.

Describe the Solution You'd Like

A option to select two spaces or <br>

line1  
line2  
line1<br>
line2<br>
@HemanthJabalpuri HemanthJabalpuri added the rule suggestion Suggestion to add or edit a rule label Oct 31, 2023
@pjkaufman
Copy link
Collaborator

pjkaufman commented Nov 8, 2023

Hey @HemanthJabalpuri . This is something that would be feasible by adding an option for what to use as the line break. It would require a code change to add the option and a change to the following logic here.

Were you thinking that this would have the following options for the two spaces between lines with content?

  • <br> -- correct, but not where xml is concerned
  • <br/> -- the more correct option for the html element

@HemanthJabalpuri
Copy link
Author

Yep, that's what I thought

@pjkaufman
Copy link
Collaborator

I went ahead and merged the PR for these changes. I plan to make a release later this week, but we will see if that pans out.

Please let us know if there is an issue with this on master or in the next release.

@pjkaufman pjkaufman self-assigned this Apr 24, 2024
@pjkaufman pjkaufman added markdown General Markdown or Markdown related issue or feature resolution/update-made A change has been made that should resolve this issue or request labels Apr 24, 2024
@pjkaufman
Copy link
Collaborator

I went ahead and merged the PR for these changes. I plan to make a release later this week, but we will see if that pans out.

Please let us know if there is an issue with this on master or in the next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
markdown General Markdown or Markdown related issue or feature resolution/update-made A change has been made that should resolve this issue or request rule suggestion Suggestion to add or edit a rule
Projects
Status: Released
2 participants