Closed Bug 233275 Opened 16 years ago Closed 16 years ago

spell checker incorrectly replaces words on double-click

Categories

(MailNews Core :: Composition, defect)

x86
Windows XP
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: fishbert, Assigned: neil)

References

Details

Attachments

(1 file)

User-Agent:       
Build Identifier: Mozilla Thunderbird 0.5a (20040128)

Double-clicking blank space, or even the scroll bar up/down arrows, in the
suggested words box of the spell checker always replaces the word being checked
by the highlighted suggestion (should instead do nothing).

Reproducible: Always
Steps to Reproduce:
1. Compose a new message: "spellng is gud" is an example
2. Spell check the message
3. 'spellng' is highlighted; double click the empty space below suggested word
and/or
4. 'gud' is highlighted; double click the scroll bar's up/down arrows in the
suggested word box

Actual Results:  
'spellng' is replaced with 'spelling' and/or 'gud is replaced with 'sud'

Expected Results:  
If anything, the highlighted selection should be un-highlighted in both instances.
No words should be replaced, and the spell checker should not move on to another
word.

Not sure if this also works when double-clicking the scroll bar between the
up/down arrows -- it doesn't in the 'gud' example.
bug 222541
QA Contact: esther → core.spelling-checker
Also if there are no suggestions then double-clicking is the same as ignore.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Attached patch Proposed patchSplinter Review
Assignee: sspitzer → neil.parkwaycc.co.uk
Status: NEW → ASSIGNED
Attachment #146180 - Flags: review?(daniel)
Comment on attachment 146180 [details] [diff] [review]
Proposed patch

r=daniel@glazman.org
Attachment #146180 - Flags: review?(daniel) → review+
Attachment #146180 - Flags: superreview?(mscott)
Attachment #146180 - Flags: superreview?(mscott) → superreview+
Fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Blocks: 240476
fixed on the tbird 0.6 branch
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.