Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fixes issue #845 #888

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.

Sign up for GitHub

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

Jump to bottom
Merged
xoofx merged 2 commits into from
Sep 11, 2025
Merged

Fixes issue #845 #888

xoofx merged 2 commits into from
Sep 11, 2025
+24 −0

Conversation

@prozolic
Copy link
Contributor

@prozolic prozolic commented Sep 6, 2025

This PR fixes the bug reported in issue #845.
With this fix, in the following case:

var result = Markdown.ToHtml("-\n\n-\n\n  foo");

This result will be produced:

<ul>
<li></li>
<li></li>
</ul>
<p>foo</p>

@xoofx
Copy link
Owner

xoofx commented Sep 7, 2025

Thanks for the fix! Could you add a unit test?

@xoofx xoofx added the bug label Sep 7, 2025
@prozolic
Copy link
Contributor Author

prozolic commented Sep 8, 2025

Ok, I added the test.

@xoofx xoofx merged commit 1b04599 into xoofx:master Sep 11, 2025
1 check passed
@prozolic prozolic deleted the pullreq branch September 11, 2025 10:06
@MihaZupan MihaZupan mentioned this pull request Oct 20, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

bug

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@prozolic @xoofx