Tickets :: [#6909] safari + fckedit != working spellcheck

Summary	safari + fckedit != working spellcheck
Queue	IMP
Queue Version	4.2
Type	Bug
State	Resolved
Priority	1. Low
Owners	Michael Slusarz <slusarz (at) horde (dot) org>
Requester	liamr (at) umich (dot) edu
Created	06/13/2008 (162 days ago)
Due
Updated	06/16/2008 (159 days ago)
Assigned	06/16/2008 (159 days ago)
Resolved	06/16/2008 (159 days ago)
Attachments
Milestone
Patch	No

06/16/2008	Chuck Hagenbuch	State ⇒ Resolved

06/16/2008	liamr (at) umich (dot) edu	Comment #9	Reply to this comment
It looks like that fixed it.

06/16/2008	Michael Slusarz	Comment #8 State ⇒ Feedback	Reply to this comment
Does these commits finally fix things?

06/16/2008	CVS Commit	Comment #7	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/imp/docs/CHANGES?r1=1.699.2.311&r2=1.699.2.312&ty=u http://cvs.horde.org/diff.php/imp/js/SpellChecker.js?r1=1.38.2.10&r2=1.38.2.11&ty=u http://cvs.horde.org/diff.php/imp/js/src/SpellChecker.js?r1=1.36.2.9&r2=1.36.2.10&ty=u

06/16/2008	CVS Commit	Comment #6	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/imp/docs/CHANGES?r1=1.1139&r2=1.1140&ty=u http://cvs.horde.org/diff.php/imp/js/SpellChecker.js?r1=1.48&r2=1.49&ty=u http://cvs.horde.org/diff.php/imp/js/src/SpellChecker.js?r1=1.45&r2=1.46&ty=u

06/13/2008	Chuck Hagenbuch	State ⇒ Assigned

06/13/2008	liamr (at) umich (dot) edu	Comment #5	Reply to this comment
I've given it a go... Safari 3.1.1 on OSX.. w/fckedit, it does appear to check the spelling, but doesn't offer any suggestions. misspelled words are underlined, though, and we don't get shoved off into an endless "Checking..." w/xinha, it works as expected.

06/13/2008	Michael Slusarz	Comment #4 State ⇒ Resolved Assigned to Michael Slusarz	Reply to this comment
Fixed in IMP 4.2.1.

06/13/2008	CVS Commit	Comment #3	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/imp/js/SpellChecker.js?r1=1.38.2.9&r2=1.38.2.10&ty=u http://cvs.horde.org/diff.php/imp/js/src/SpellChecker.js?r1=1.36.2.8&r2=1.36.2.9&ty=u

06/13/2008	CVS Commit	Comment #2	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/imp/js/SpellChecker.js?r1=1.47&r2=1.48&ty=u http://cvs.horde.org/diff.php/imp/js/src/SpellChecker.js?r1=1.44&r2=1.45&ty=u

06/13/2008	liamr (at) umich (dot) edu	Comment #1 Milestone ⇒ Queue ⇒ IMP Summary ⇒ safari + fckedit != working spellcheck Type ⇒ Bug Priority ⇒ 1. Low State ⇒ Unconfirmed	Reply to this comment
If you compose a message using fckedit in Safari 3 and actually apply any formatting (font change, formatting), when you go to spell check, "Check Spelling" turns into "Checking..." and stays that way, w/o making any suggestions. Steps to reproduce: a. Click New Message b. Type subject 'html compose' c. Click 'switch to html composition link' d. Type text 'fannncy message' e. Select 'all' by highlighting or typing 'apple+A' f. Set formatting to 'Heading 1' or similar and font to 'Comic Sans' or similar g. Click 'Check spelling' -> Check spelling turns to 'Checking...' and red color and remains indefinitely. Send message is not responsive. Note that checking spelling works if no formatting is set. Xinha works correctly in Safari 3. FCKEditor works correctly (or does not exhibit the same problem as Safari 3) in Firefox 2 and 3 on OSX.