Closed Bug 119192 Opened 23 years ago Closed 20 years ago

Add accesskeys to Recipient widget

Categories

(MailNews Core :: Composition, defect)

x86
All
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED WORKSFORME

People

(Reporter: robert, Assigned: sspitzer)

References

Details

(Keywords: access)

Attachments

(1 file)

The To/Cc/Bcc control only responds to up/down arrow keys.  It should also
respond to "t" to select "To:", "c" to select "Cc:", "b" to select "Bcc:", etc.
In other words, we need to add accesskeys (and as a side note, we're still 
awaiting the checkin of the big patch to implement true accesskey support).

->Composition
Assignee: sspitzer → ducarroz
Status: UNCONFIRMED → NEW
Component: Mail Window Front End → Composition
Ever confirmed: true
QA Contact: esther → sheelar
Summary: To/Cc/Bcc doesn't respond to keys → Add accesskeys to Recipient widget
UI stuff goes to Varada, correct?
Assignee: ducarroz → varada
OS: Windows 2000 → All
right, Varada takes care of the front-end of message compose...
change qa contact to olga
QA Contact: sheelar → olgam
taking all of varada's bugs.
Assignee: varada → sspitzer
Blocks: 154249
added access keyword.
Keywords: access
The attached patch adds the access keys. This is still not the same behavior as
in Netscape 4: there, tabbing to the To/Cc/Bcc control and then pressing the
associated key suffices; here, you first have to press the down arrow to expand
the list, then use the associated key.
Attachment #148727 - Flags: review?
Comment on attachment 148727 [details] [diff] [review]
adds accesskey support for To/Cc/Bcc/... 

Chrisitan Wenz, you have to specify someone to take the review when you request
it.
Attachment #148727 - Flags: review? → review?(neil.parkwaycc.co.uk)
Since this bug was filed, bug 92491 was fixed which allows use of the first
letter without an access key and bug 64157 was fixed which allows use of letters
without opening the menulist first. This makes the recipient widget accessible.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → WORKSFORME
Comment on attachment 148727 [details] [diff] [review]
adds accesskey support for To/Cc/Bcc/... 

Patch is no longer necessary as per previous comment.
Attachment #148727 - Flags: review?(neil.parkwaycc.co.uk)
*** Bug 144303 has been marked as a duplicate of this bug. ***
It's *almost* fixed.  The only quibble I have is that the accelerator keys work
on a "cumulative" basis rather than a single keypress.  For instance, if you
type "B" to change the field to "Bcc", and then change your mind a few times by
hitting "T" for "To", "B" for "Bcc", etc., the selection will eventually no
longer change.  An accelerator key should be a unique, non-cumulative single
keypress, and should probably show up with the accelerator key underlined (just
as the pull-down menus have their hot key letters underlined).
(In reply to comment #12)
> The only quibble I have is that the accelerator keys work
> on a "cumulative" basis rather than a single keypress.

See bug 64157 for the design discussion, such as it was, about that feature.
Note that there is a timer, so if you stop typing for a second, the "cumulative 
buffer" clears and the next character typed is the first.
Status: RESOLVED → VERIFIED
Product: MailNews → Core
doesn't work for me with the released 1.7.8 version on various Windows platforms
(I am on the road right now and cannot test current builds and other systems).
Can someone verify?
(In reply to comment #14)
> doesn't work for me with the released 1.7.8 version on various Windows
> platforms

That's because the fix is in the trunk -- you need to be running a 1.8xxx build.
The fix will *not* be ported to 1.7.x.
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: