-
-
Notifications
You must be signed in to change notification settings - Fork 129
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
Upgraded and improved editor features #1744
Conversation
Great work, would love to give it a try! |
Thanks for your feedback. I've done. |
Yes, it would be nice if Squire-RTE can be used. But i also see you added DOMPurify. |
Thank you so much. |
Ok, Squire-RTE can't be used by default due to the important changes. A good example why SnappyMail uses a modified Squire is this issue #962
Now see the difference in speed with Squire-RTE and ours. Feel free to create a new PR with only the RTL fixes. |
Hello,
It seems that due to issues and the absence of certain features in the editor, which have been frequently mentioned in various issues, it would be better to make changes and developments in this section and upgrade the Squire editor version along with it. Some of the issues include the lack of text alignment, problems with text direction, the inability to add text styles as inline CSS, issues with directly copying tables, and visual and display problems.
Therefore, implementing these changes will address many needs, especially for RTL languages. I would appreciate it if you could accept this PR to improve performance and provide more convenience for users.