124195 - two spaces between words throws off spellchecker

Status: VERIFIED INVALID

(Core :: DOM: Editor, defect)

Description

[sujay]

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"

Comment 1

[michael wendell]

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.

Comment 2

[sujay]

nominating...we should fix this...its a usability problem with spellchecker.

Comment 3

[Syd Logan]

Over to akkana for investigation. Bug in us? Bug in the spell checker? EDITORBASE+

Comment 4

[Akkana Peck]

Kin's the expert on the spellchecker.  Kin?  Any thoughts about what the problem
might be here?

Comment 5

[Akkana Peck]

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

Comment 6

[nhottanscp]

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.

Comment 7

[nhottanscp]

Syd, can we move this bug to bugscape?

Comment 8

[kinmoz]

nhotta, go ahead and move it, it's a netscape spellchecker (backend)
implementation issue.

Comment 9

[Frank Tang]

give to ftang for now.

Comment 10

[Frank Tang]

assign

Comment 11

[Frank Tang]

move the bug the bugzcape 12496. close this bug as remind. please move the
discussion to the bugscape bug.

Comment 12

[Christopher Hoess (gone)]

REMIND is deprecated per bug 35839.

Comment 13

[Christopher Hoess (gone)]

INVALID; this is a Netscape bug.

Comment 14

[sujay]

verified.