Skip to content

Add getSemanticHTML options & Add preserveWhitespace option to getSem… #4598

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

MaciejDabek
Copy link

As many people, I was also using quill for it HTML content, so it can be displayed somewhere else.

The addition of conversion from whitespace to   was needed for consecutive copying and pasting, so my proposition is to add options object that can be extended for other cases. For now, I added preserveWhitespace that preserve all whitespaces if true, otherwise all whitespaces are changed to   (default: false).

I hope that this can be a middle ground between pushing Quill forward and providing compatibility for all people using it for a long time.

@jankapunkt
Copy link

I think this is the best solution for the problem and is future proof due to extensibility.

@PieterJanCl
Copy link

Would love to see this PR approved, just like many others we are having troubles with the &nbsp chars.

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