Closed Bug 1400428 Opened 7 years ago Closed 7 years ago

Foala editor spell checker does not work correctly

Categories

(Core :: Spelling checker, defect, P3)

55 Branch
defect

Tracking

()

RESOLVED INVALID
Tracking Status
firefox57 --- wontfix
firefox58 --- fix-optional

People

(Reporter: andy2020, Unassigned)

Details

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:55.0) Gecko/20100101 Firefox/55.0
Build ID: 20170824053838

Steps to reproduce:

Type several incorrect spelled words into Froala editor located here:

https://www.froala.com/wysiwyg-editor


Actual results:

Red underline is displayed only for a short time and is removed after more words are typed.


Expected results:

Red underline should remain on misspelled words even after you type other words.
Component: Untriaged → Spelling checker
Product: Firefox → Core
I am a developer at @Froala and I could provide more details about this. It appears the problem is coming from that fact that when some changes are made to the HTML via JS, spellchecker is no longer working. This should help to replicate the problem outside of the editor in a contenteditable field.
Debugging entire apps is often painful and time consuming. If you're willing, what would help the most is if you could cut things down until you have the smallest possible standalone testcase that reproduces the issue, and attach that here.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(stefan.neculai)
Priority: -- → P3
(In reply to Jonathan Watt [:jwatt] from comment #2)
> Debugging entire apps is often painful and time consuming. If you're
> willing, what would help the most is if you could cut things down until you
> have the smallest possible standalone testcase that reproduces the issue,
> and attach that here.

Where is there an attach button?
I can provide a jsFiddle wih a contenteditable field alone. Let me just find some time in the following days.
Flags: needinfo?(stefan.neculai)
(In reply to Andy Bajka from comment #3)
> Where is there an attach button?

See the "Attach File" link at the bottom of the all the header fields (just above your first comment).
(In reply to Stefan from comment #4)
> I can provide a jsFiddle wih a contenteditable field alone. Let me just find
> some time in the following days.

Thanks. That would certainly be an improvement, although an attached testcase would be better (whoever debugs this will likely want that).
Here is a jsFiddle: https://jsfiddle.net/udexf2js/. 

Steps to reproduce:
  1. Put cursor at the end of "Type here" text.
  2. Hit space.
  3. Type a misspelled word.
  4. Hit space.
  5. Type another misspelled word.


It appears that the main culprit for the problem is the normalize method. Commenting the line with this.normalize() would make the browser built-in spellchecker work correctly.
Froala has worked out a solution to this issue and should be available in their latest version shortly.

Please close this bug report as it is no longer needed.

Thank you.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.