-
Notifications
You must be signed in to change notification settings - Fork 816
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 support for html underline and videos #1955
Add support for html underline and videos #1955
Conversation
fix: issue where when html2md parse <em> to "_" instead of "*" that won't be detected MarkdownToDelta converter feat(test): added test for DeltaX feat: added config classes to MarkdownToDelta and html2md that allow users configure their own styles
Thank you for your contributions, I will review and test the changes soon. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for adding tests, I haven't tested this change yet.
Just a few things to consider before releasing this change.
dcfc228
to
acf6336
Compare
@CatHood0 Please resolve the conflict. Please do not worry too much about where the best place is to put the code. I believe @ellet0 can always move around code to make sense OOD wise. |
Yeah, i'm trying to resolve the before commit t |
@ellet0 I am going ahead and merging this. Please feel free to patch it after this. |
Description
If you try to use
DeltaX.fromHtml
with a tag<u>
to use it as a underline style, then will be removed and just pass content as plain. Videos has the same result of the before one, if you try to use an<iframe>
will be paste as a plain text or removed (idk really this last one case).Related Issues
Fix #1917
Fix #1945
Improvements
Checklist
CHANGELOG.md
nor the package version inpubspec.yaml
files../scripts/before_push.sh
and it all passed successfullyBreaking Change
Does your PR require developers to manually update their apps to accommodate your change?
!
in the title as explained in Conventional Commits).