linagora / tmail-flutter

A multi-platform (Flutter) application for reading your emails, with your favorite devices, using the JMAP protocol!
GNU Affero General Public License v3.0
319 stars 73 forks source link

Speller checker for Safari not work #2962

Open hoangdat opened 4 months ago

hoangdat commented 4 months ago

Desc:

not see the spell checker in composer (Language Tools)

DoD:

dab246 commented 3 months ago
tddang-linagora commented 1 month ago
  • Speller check not work HtmlElementView of Flutter Web but it work on html native web of Safari

https://github.com/user-attachments/assets/fcf8dac6-9e82-4186-8e08-13eeddae5042

It works fine?

dab246 commented 1 month ago
  • Speller check not work HtmlElementView of Flutter Web but it work on html native web of Safari

Screen.Recording.2024-09-10.at.11.47.15.mov It works fine?

What you see underlined is not Speller check. It is autocorrect, please install speller check extension to check.

tddang-linagora commented 1 month ago

What you see underlined is not Speller check

https://github.com/user-attachments/assets/79cac88e-b86a-47cc-976a-92145fe0a89a

The browser says it is spelling though.

dab246 commented 1 month ago

What you see underlined is not Speller check

Screen.Recording.2024-09-10.at.13.22.31.mov The browser says it is spelling though.

I think you should take a closer look. Use the LanguageTool extension to check.

tddang-linagora commented 1 month ago

Upon further inspection, LanguageTools spell checker isn't work at all if the text editor is inside the iframe. You can try this code snippet below in Safari.

<!DOCTYPE html>
<html>
  <head>
    <title>Page Title</title>
  </head>
  <body>
    <p>This will check spelling</p>
    <div contenteditable="true" spellcheck="true" style="border:1px solid grey;"></div>
    <br>
    <iframe srcdoc="<!DOCTYPE html>
      <html>
      <head>
      <title>Page Title</title>
      </head>
      <body>
      <p>This will NOT check spelling</p>
      <div contenteditable='true' spellcheck='true' style='border:1px solid grey;'></div>
      </body>
      </html>
      "/>
  </body>
</html>

Related information

Conclusion I need to update my Macbook to the latest version, in order to install latest beta Safari. It's likely to be fixed there. Will update with more info after further testing

tddang-linagora commented 1 month ago

This is an issue of LanguageTools https://forum.languagetool.org/t/lt-doesnt-work-on-latest-safari/10421

dab246 commented 1 month ago

This is an issue of LanguageTools https://forum.languagetool.org/t/lt-doesnt-work-on-latest-safari/10421

Please research other Speller check extensions for comparison.