the-djmaze / snappymail

Simple, modern & fast web-based email client
https://snappymail.eu
GNU Affero General Public License v3.0
981 stars 114 forks source link

Replace default editor with Quill #1684

Closed rezaei92 closed 3 weeks ago

rezaei92 commented 1 month ago

Since the default text editor of the webmail has minimal features and has major weaknesses in supporting RTL languages as mentioned in issue #1389, I decided to replace it with the quill text editor. I have done the initial work of this replacement and the necessary items We added it to Gulp and performed the initial tests, which were successful. This editor has rich features and fully meets the requirements of RTL languages such as Persian language or copying the Excel table directly from clipboard. The editor is different from the previous editor and it has been successfully tested. Quill editor screenshots has been shown below. Also features such as text alignment are not available in the Squire editor. But at the time of build with Gulp, the following warning was displayed, which needs your review because it is imported into the HtmlEditor.js file.

image

rezaei92 commented 1 month ago

Gulp warning: image

the-djmaze commented 1 month ago

Add it to globals in https://github.com/the-djmaze/snappymail/blob/master/.eslintrc.js

the-djmaze commented 1 month ago

Also there are some important things you should implement:

I had trouble with it due to how Quill operates (it is not HTML but "delta")

rezaei92 commented 3 weeks ago

Yes, you are right. The Squire editor is more suitable for webmail applications than the Quill editor. And it is better to improve it instead of changing it.

I'll do it and send you the PR. Thank you.