Note: There are a few cases of duplicates in user autocompletion which are being worked on.

mailnews spell check - [Enter] no longer chooses 'replace' although 'replace' is 'active' button.

RESOLVED FIXED

Status

()

Core
Spelling checker
RESOLVED FIXED
13 years ago
10 years ago

People

(Reporter: Bishop Clark (LC957), Assigned: neil@parkwaycc.co.uk)

Tracking

({fixed-seamonkey1.1.7, regression})

Trunk
x86
Windows 98
fixed-seamonkey1.1.7, regression
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

13 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8b2) Gecko/20050317
Build Identifier: Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8b2) Gecko/20050317

(Sorry for the bad Component:  none of the mailnews onew were appropriate!)

When editing a new message, I've grown into the habit, with mozilla spellcheck,
of being able to type in my word correction and hitting [Enter] to have it
accepted (similar to finding the mouse, moving over to the [replace] button and
hitting it).  

1.8b2 doesn't seem to accept my repeated banging on the [enter] key as
equivalent to clicking [replace].  1.7.5 does exactly as I expect, however.

Reproducible: Always

Steps to Reproduce:
1. compose new email, text mode, with some words spelled badly
2. open Spell check
3. ignore the suggestions:  retype the first correction and hit [enter]

Actual Results:  
correction entry had correct spelling, but the word wasn't changed;  had to
click [replace] to perform the replacement

Expected Results:  
it should have performed the replacement and moved to the next word: since
[replace] was the only un-dimmed box, hitting [enter] should have done the same
as if I were to have clicked it.
(Reporter)

Updated

13 years ago
Keywords: regression

Updated

13 years ago
Assignee: general → mscott
Component: General → Spelling checker
Product: Mozilla Application Suite → Core
QA Contact: general → spelling-checker
Version: unspecified → Trunk
(Assignee)

Comment 1

13 years ago
Created attachment 177868 [details] [diff] [review]
Fix last caller of Replace()
Assignee: mscott → neil.parkwaycc.co.uk
Status: UNCONFIRMED → ASSIGNED
Attachment #177868 - Flags: superreview?(mscott)
Attachment #177868 - Flags: review?(daniel)
(Assignee)

Comment 2

13 years ago
FYI this was a regression from bug 233275.

Comment 3

13 years ago
Comment on attachment 177868 [details] [diff] [review]
Fix last caller of Replace()

FYI, I checked this change into Thunderbird as well.
Attachment #177868 - Flags: superreview?(mscott) → superreview+
fails for me 
Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9a9pre) Gecko/2007102203 SeaMonkey/2.0a1pre
fails also SeaMonkey 1.1.4
works thunderbird v2
(Assignee)

Comment 6

10 years ago
Comment on attachment 177868 [details] [diff] [review]
Fix last caller of Replace()

Trying different reviewer. (Also need a= for branch!)
Attachment #177868 - Flags: review?(daniel) → review?(iann_bugzilla)

Comment 7

10 years ago
Comment on attachment 177868 [details] [diff] [review]
Fix last caller of Replace()

r=me
a-SM1.1.7=me though I suspect that is not sufficient.
Attachment #177868 - Flags: review?(iann_bugzilla) → review+
(Assignee)

Comment 8

10 years ago
Fix checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Keywords: fixed-seamonkey1.1.7
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.