two spaces between words throws off spellchecker

VERIFIED INVALID

Status

()

Core
Editor
VERIFIED INVALID
17 years ago
16 years ago

People

(Reporter: sujay, Assigned: Frank Tang)

Tracking

Trunk
mozilla1.0
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: EDITORBASE+)

(Reporter)

Description

17 years ago
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

17 years ago
Target Milestone: --- → mozilla1.2

Comment 1

17 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.
(Reporter)

Comment 2

17 years ago
nominating...we should fix this...its a usability problem with spellchecker.
Keywords: nsbeta1
Whiteboard: EDITORBASE

Comment 3

17 years ago
Over to akkana for investigation. Bug in us? Bug in the spell checker? EDITORBASE+
Assignee: kin → akkana
Keywords: nsbeta1 → nsbeta1+
Whiteboard: EDITORBASE → EDITORBASE+
Target Milestone: mozilla1.2 → mozilla1.0

Comment 4

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

Comment 5

17 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

17 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

17 years ago
Syd, can we move this bug to bugscape?

Comment 8

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

Comment 9

17 years ago
give to ftang for now.
Assignee: nhotta → ftang
Status: ASSIGNED → NEW
(Assignee)

Comment 10

17 years ago
assign
Status: NEW → ASSIGNED
(Assignee)

Comment 11

17 years ago
move the bug the bugzcape 12496. close this bug as remind. please move the
discussion to the bugscape bug.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → REMIND
REMIND is deprecated per bug 35839.
Status: RESOLVED → REOPENED
Resolution: REMIND → ---
INVALID; this is a Netscape bug.
Status: REOPENED → RESOLVED
Last Resolved: 17 years ago16 years ago
Resolution: --- → INVALID
(Reporter)

Comment 14

16 years ago
verified.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.