Closed
Bug 124195
Opened 23 years ago
Closed 22 years ago
two spaces between words throws off spellchecker
Categories
(Core :: DOM: Editor, defect)
Core
DOM: Editor
Tracking
()
VERIFIED
INVALID
mozilla1.0
People
(Reporter: sujay, Assigned: ftang)
Details
(Whiteboard: EDITORBASE+)
using 2/6 build of netscape 1) launch netscape 2) launch composer 3) enter the following string: "This is a test. Hello World" notice only one space between "test." and "Hello" 4) click Spellcheck button no misspelled words. close the spellchecker dialog. 5) add another space between "test." and "Hello", so now you have two spaces between them. 6) run spellchecker again. Now it finds a misspelled word. Either "test." or "Hello"
Updated•23 years ago
|
Target Milestone: --- → mozilla1.2
Comment 1•23 years ago
|
||
Seems to me that the problem is that Spellchecker is getting screwed up with the . This can certianly be a problem since Composer uses everytime more then one space is used.
nominating...we should fix this...its a usability problem with spellchecker.
Keywords: nsbeta1
Whiteboard: EDITORBASE
Over to akkana for investigation. Bug in us? Bug in the spell checker? EDITORBASE+
Comment 4•23 years ago
|
||
Kin's the expert on the spellchecker. Kin? Any thoughts about what the problem might be here?
Comment 5•23 years ago
|
||
I checked with Kin -- he fixed this already, in rev 1.18 of ns/modules/spellchk/glue/nsSpellCheckGlue.cpp, but the fix got removed by Noaki's checkin: http://warp.mcom.com/webtools/bonsai/cvsview2.cgi?diff_mode=context&whitespace_mode=show&file=nsSpellCheckGlue.cpp&root=/m/src&subdir=ns/modules/spellchk/glue&command=DIFF_FRAMESET&rev1=1.27&rev2=1.28
Assignee: akkana → nhotta
Comment 6•23 years ago
|
||
nsSpellCheckGlue.cpp rev=1.28 for (i = 0; i < len; i++) { if (0xFF00 & ustr[i] || ustr[i] == NBSP_CHAR_CODE) aBuf->mData[i] = ' '; else aBuf->mData[i] = (char)ustr[i]; } So, the NSBP check is still needed. I thought I kept that part but obviously didn't, sorry about that.
Status: NEW → ASSIGNED
Comment 7•23 years ago
|
||
Syd, can we move this bug to bugscape?
nhotta, go ahead and move it, it's a netscape spellchecker (backend) implementation issue.
Assignee | ||
Comment 9•23 years ago
|
||
give to ftang for now.
Assignee: nhotta → ftang
Status: ASSIGNED → NEW
Assignee | ||
Comment 11•22 years ago
|
||
move the bug the bugzcape 12496. close this bug as remind. please move the discussion to the bugscape bug.
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → REMIND
Comment 12•22 years ago
|
||
REMIND is deprecated per bug 35839.
Status: RESOLVED → REOPENED
Resolution: REMIND → ---
Comment 13•22 years ago
|
||
INVALID; this is a Netscape bug.
Status: REOPENED → RESOLVED
Closed: 22 years ago → 22 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•