Closed
Bug 307169
Opened 19 years ago
Closed 19 years ago
change the kbd shortcut for 'Check for Updates' to 'U'
Categories
(Firefox :: Keyboard Navigation, defect)
Firefox
Keyboard Navigation
Tracking
()
RESOLVED
WONTFIX
Firefox1.5
People
(Reporter: harig.in, Assigned: Gavin)
Details
Attachments
(1 file)
|
1.02 KB,
patch
|
asaf
:
review-
|
Details | Diff | Splinter Review |
the menu item 'Check for updates' has a keyboard shortcut 'o'. Could be better / consistent if 'u' is used. (Alt + h + u - to launch the update dialog in Windows)
| Assignee | ||
Updated•19 years ago
|
OS: Windows 2000 → All
Hardware: PC → All
| Assignee | ||
Updated•19 years ago
|
Attachment #195599 -
Flags: review?(mconnor)
| Assignee | ||
Updated•19 years ago
|
Whiteboard: [patch-r?]
Comment 2•19 years ago
|
||
Comment on attachment 195599 [details] [diff] [review] patch ok, but why did you move it below the label, but not flip the one right above that's also backwards? Please fix this (trivial) on checkin.
Attachment #195599 -
Flags: review?(mconnor) → review+
| Assignee | ||
Updated•19 years ago
|
Whiteboard: [patch-r?] → [patch-r+][checkin needed]
Target Milestone: --- → Firefox1.5
| Assignee | ||
Comment 3•19 years ago
|
||
mozilla/browser/locales/en-US/chrome/browser/baseMenuOverlay.dtd; new revision: 1.5; previous revision: 1.4
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Whiteboard: [patch-r+][checkin needed]
| Assignee | ||
Comment 4•19 years ago
|
||
Comment on attachment 195599 [details] [diff] [review] patch Trivial change, makes the update menu easier to access.
Attachment #195599 -
Flags: approval1.8b5?
Updated•19 years ago
|
Attachment #195599 -
Flags: approval1.8b5? → approval1.8b5+
| Assignee | ||
Comment 5•19 years ago
|
||
This doesn't have any localization impact, since it only changes the accesskey for the en-US string.
Keywords: late-l10n
| Assignee | ||
Comment 6•19 years ago
|
||
mozilla/browser/locales/en-US/chrome/browser/baseMenuOverlay.dtd; new revision: 1.4.4.1;
Keywords: fixed1.8
Comment 7•19 years ago
|
||
Comment on attachment 195599 [details] [diff] [review] patch This doesn't play well with the dynamic label of this menuitem.
Attachment #195599 -
Flags: review+ → review-
Updated•19 years ago
|
| Assignee | ||
Comment 8•19 years ago
|
||
Backed out branch and trunk.
Status: REOPENED → RESOLVED
Closed: 19 years ago → 19 years ago
Resolution: --- → WONTFIX
Updated•19 years ago
|
Attachment #195599 -
Flags: approval1.8b5+
You need to log in
before you can comment on or make changes to this bug.
Description
•