Closed Bug 24517 Opened 25 years ago Closed 25 years ago

Spell Check under mail gives javascript error - does not launch

Categories

(MailNews Core :: Composition, defect, P3)

defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: pmock, Assigned: bugzilla)

Details

Build Date & Platform Bug Found: Win32 commercial seamonkey build 2000-01-19-13-m13 installed on P166 Win98 Linux commercial seamonkey build 2000-01-19-16-m13 installed on P200 RedHat 6.1 MacOS mozilla seamonkey build 2000-01-19-09-m13 installed on G3/400 OS 8.6 Overview Description: Spell Check dialog does not launch under mail. It gives a javascript error in the terminal window. JavaScript Error: ReferenceError: Spelling is not defined". The spell check dialog does launch under win32 and linux Page Composer. Steps to Reproduce: 1) Launch Seamonkey 2) Open Messenger 3) Start a new mail message The default compose window appears: HTML 4) Type some misspelled text in the message body 5) Press the Spell Check button off the toolbar No dialog appears. Javascript error appears in the win32 or linux terminal window. Actual Results: Javascript error appears in windows and linux terminal window. Nothing happens when the spelling button is pressed off the mail compose toolbar. On Mac, there is no terminal window. Expected Results: The spell check dialog should appear. Additional Builds and Platforms Tested On: The Mac commercial seamonkey build 2000-01-19-09-m13 fails to launch and could not be tested. Additional Information: According to Charle Manske (cmanske@netscape.com), he wrote in his email response "I just tweaked the UI for the Spell Checker dialog yesterday, so look at the M13 build to see what I hope is the final version of the layout, except for fixing widget size and alignment issues. Feel free to comment on this. The underlying interface is defined in nsISpellChecker.h". Note: The spell check dialog comes up in Page Composer on Win32 and Linux builds. It does not come up on Mac Page Composer, but I could only test the mozilla build.
QA Contact: lchiang → pmock
Status: NEW → ASSIGNED
Target Milestone: M14
Accepting. We probably need to include another JS file in msg compose.
SpellChecked has ben removed from message compose for M14/B1. move it to M15
Target Milestone: M14 → M15
Mass moving to M16 to get these off the M15 radar. Please let me know if this is really an M15 stopper.
Target Milestone: M15 → M16
We probably need to enable the spell checker, which is currently set for M17, before we can do this bug.
Spell checker is enable in Commercial build.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Thank you Jeff :)
Verified as fixed on win32, macos, and linux using the following builds: win32 commercial seamonkey build 00050909-m16 installed on Dell P500, Win98 macos commercial seamonkey build 00050908-m16 installed on G3/400, MacOS 9.04 linux commercial seamonkey build 00050908-m16 installed on P200, RedHat Spell check dialog does appears when prompted.
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.