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

Fix issue #87 #88

Merged
merged 2 commits into from
Apr 5, 2024
Merged

Fix issue #87 #88

merged 2 commits into from
Apr 5, 2024

Conversation

shuedo
Copy link
Contributor

@shuedo shuedo commented Apr 5, 2024

This pull request addresses the issue described in #87, where a line break preceding a list containing inline attributes results in improper HTML formatting.

this closes #87

@nadar
Copy link
Owner

nadar commented Apr 5, 2024

Looks great! Could you please add a changelog entrie? otherwise i will do it for you. next patch version would be 3.4.2

@shuedo
Copy link
Contributor Author

shuedo commented Apr 5, 2024

Looks great! Could you please add a changelog entrie? otherwise i will do it for you. next patch version would be 3.4.2

Done!

@nadar nadar merged commit c8fbe3d into nadar:master Apr 5, 2024
5 checks passed
@nadar
Copy link
Owner

nadar commented Apr 5, 2024

Thank you so much @shuedo! Perfect issue description, perfect error finding, perfect pull requesting with testings and changelog! If every pull request would be this way, the world would be a better place for sure! thanks 🌏

https://github.com/nadar/quill-delta-parser/releases/tag/3.4.2

@shuedo
Copy link
Contributor Author

shuedo commented Apr 5, 2024

Thanks, @nadar! Really appreciate the encouragement. I'm excited to keep contributing in the future! 😄

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.

<p> without close tag is added when list has bold text and a previous linebreak
2 participants