Closed
Bug 191169
Opened 22 years ago
Closed 22 years ago
Duplicate accesskey "h" in proxies preferences
Categories
(Core :: DOM: UI Events & Focus Handling, defect)
Core
DOM: UI Events & Focus Handling
Tracking
()
VERIFIED
FIXED
mozilla1.4alpha
People
(Reporter: piers, Assigned: piers)
References
Details
(Keywords: access)
Attachments
(1 file)
947 bytes,
patch
|
deanis74
:
review+
dmosedale
:
superreview+
|
Details | Diff | Splinter Review |
"HTTP Proxy" currently uses "h" as the accesskey which is reserved for the Help
button.
HTTP Proxy
Can't use H, reserved for Help
Can't use T, P, R or O, reserved for one of the Ports
Attachment #113004 -
Flags: superreview?(dmose)
Attachment #113004 -
Flags: review?(dean_tessman)
Comment on attachment 113004 [details] [diff] [review]
Change accesskey from h to x
I guess. We can't remove H from Help because that's pretty standard
throughtout the app. We can't steal the Port accesskeys because there's not
much choice for those. The only other thing I can think of is toasting the
accesskeys on the Port fields altogether, and then the user can tab to them.
But that's out of the scope of this bug.
r=me
Attachment #113004 -
Flags: review?(dean_tessman) → review+
Comment 3•22 years ago
|
||
Comment on attachment 113004 [details] [diff] [review]
Change accesskey from h to x
sr=dmose, assuming aaronl is ok with this change
Attachment #113004 -
Flags: superreview?(dmose) → superreview+
Comment 4•22 years ago
|
||
Looks good to me - thanks Piers
![]() |
||
Comment 5•22 years ago
|
||
Checking in
mozilla/xpfe/components/prefwindow/resources/locale/en-US/pref-proxies.dtd;
/cvsroot/mozilla/xpfe/components/prefwindow/resources/locale/en-US/pref-proxies.dtd,v
<-- pref-proxies.dtd
new revision: 1.22; previous revision: 1.21
done
I checked in for piersc@cogs.susx.ac.uk, fixed.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Comment 6•22 years ago
|
||
vrfy'd fixed on linux rh8.0 and win2k with 2003.02.24 comm trunk bits.
Status: RESOLVED → VERIFIED
Updated•6 years ago
|
Component: Keyboard: Navigation → User events and focus handling
You need to log in
before you can comment on or make changes to this bug.
Description
•