Closed
Bug 240504
Opened 21 years ago
Closed 21 years ago
REGRESSION: Cannot tab backwards from mail text compose area
Categories
(MailNews Core :: Composition, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: aaronlev, Assigned: bryner)
References
(Blocks 1 open bug)
Details
(Keywords: access, regression)
Attachments
(1 file)
(deleted),
patch
|
neil
:
review+
jst
:
superreview+
|
Details | Diff | Splinter Review |
Steps: 1. Start new mail with Ctrl+M 2. Enter "to" address 3. Press tab 4. Enter subject 5. Press tab 6. Type some text for the body 7. Press Shift+Tab What happens: Focus remains in body composition area What happens: Focus should go back to the subject. Note that it's normal for an unmodified tab key press to not navigate in this situation, because it's used for entering a TAB character. However, shift+tab should work -- I'm pretty sure it used to work. Not sure when it regressed. Found with Mozilla 1.7b Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7b) Gecko/20040413
Reporter | ||
Updated•21 years ago
|
Flags: blocking1.7?
Comment 1•21 years ago
|
||
Further testing reveals that this affects web pages too - you can't shift+tab out of content. I also tried a two-frame set that I happen to have bookmarked and I couldn't shift+tab out of the second frame - and shift+tab out of the first frame went to the second frame too. It looks as if when shift+tab gets to the beginning of the current frame it restarts from the end of the last frame.
Keywords: regression
Comment 2•21 years ago
|
||
Backing out bryner's trunk changes to nsEventStateManager seems to fix this.
Flags: blocking1.7?
Reporter | ||
Comment 3•21 years ago
|
||
> Backing out bryner's trunk changes to nsEventStateManager seems to fix this.
Has it been backed out of the trunk? Mark this fixed?
Assignee: sspitzer → bryner
Reporter | ||
Comment 4•21 years ago
|
||
This might be affecting only 1.8
Assignee | ||
Comment 5•21 years ago
|
||
Assignee | ||
Updated•21 years ago
|
Attachment #146510 -
Flags: superreview?(jst)
Attachment #146510 -
Flags: review?(neil.parkwaycc.co.uk)
Comment 6•21 years ago
|
||
Comment on attachment 146510 [details] [diff] [review] fix thinko I remember this one, this was that convoluted code that I asked you to simplify for bug 214306.
Attachment #146510 -
Flags: review?(neil.parkwaycc.co.uk) → review+
Comment 7•21 years ago
|
||
Comment on attachment 146510 [details] [diff] [review] fix thinko sr=jst
Attachment #146510 -
Flags: superreview?(jst) → superreview+
Assignee | ||
Comment 8•21 years ago
|
||
checked in.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Updated•20 years ago
|
Product: MailNews → Core
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•