-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
All spaces converted to in quill-editor #4535
Comments
Duplicated of #4509 |
Would like to see this fixed as well |
Any updates? |
We are reverting to 2.0.2 because this breaks our use cases. I can't imagine anyone wanting this change. |
Glad someone else saw this - we're upgrading from Quill 1 to 2 and I saw this and was very confused. At least there's a minor version back that doesn't have the issue 🙃 In our own codebase we do HTML validation where we confirm certain behaviors end up creating a certain HTML - I wonder if Quill needs something like that |
We reverted to v 2.0.2 to fix this issue. |
Could this be something they added as Quill zaps all multiple spaces otherwise in v2.0.2? We have an issue where if you have two tags within each other, for example EDIT: Whoa! I just upgraded to 2.0.3 and while this actually fixed our problem I see it introduced another where the text will no longer break if being rendered! |
you can just add word-break: break-all; in tag. |
Just updated and have to revert back again. This breaks the layout on a lot of pages. Sure this can be averted by word-break: break-all, but bad work arounds tend to break other stuff. Not worth it. |
We are using quill-editor in our application to create a resume. Since all spaces are converted to
(non breaking space), the printable version created by LateX is broken (line doesn't break at the right space).Reverting to 2.0.2 works.
Steps for Reproduction
Expected behavior:
Input:
This is a test
Output:
This is a test
Actual behavior:
Input:
This is a test
Output:
This&nbps;is&nbps;a&nbps;test
Platforms:
Angular application running on latest browser (tried safari, chrome, chromium, firefox)
Version:
The issue is located in this commit : 07b68c9 where you can see line 374 :
return escapedText.replaceAll(' ', ' ');
The text was updated successfully, but these errors were encountered: