Closed
Bug 1000002
Opened 10 years ago
Closed 10 years ago
Eyedropper and Style Editor use/share same commandkey: 'y', but should be different
Categories
(Firefox :: Menus, defect)
Tracking
()
RESOLVED
FIXED
Firefox 31
People
(Reporter: aryx, Assigned: aryx)
Details
(Whiteboard: [lang=text][mentor=aryx])
Attachments
(1 file, 1 obsolete file)
2.54 KB,
patch
|
Details | Diff | Splinter Review |
Firefox Nightly 20140422030204 on Windows 8.1 In Tools > Web Developer, 'Eyedropper' and 'Style Editor' use the same accesskey 'y' for accessing the menu items with the keyboard. They should be different.
Assignee | ||
Comment 1•10 years ago
|
||
This patch changes the one of 'Style Editor' to 'l' because all the letters in 'Eyedropper' are already used elsewhere and 'l' is the only available letter which doesn't duplicate an access key. Unfortunately, that letter isn't wide. With devtools.chrome.enabled and remote debugger enabled, there are 17 menu items with 16 accesskeys. Two available letters in other labels are 'g' (several occurences) and 'k' (Network) - we could also change the access key for Page Source from 'o' to 'g' and use the 'o' e.g. for Eyedropper. Furthermore, I changed the accesskey for App Manager from lowercase 'a' to uppercase 'A' so the first letter of the label gets underlined.
Comment 2•10 years ago
|
||
Comment on attachment 8412620 [details] [diff] [review] patch, v1 Review of attachment 8412620 [details] [diff] [review]: ----------------------------------------------------------------- ::: browser/locales/en-US/chrome/browser/browser.dtd @@ +258,5 @@ > <!ENTITY devToolbarCloseButton.tooltiptext "Close Developer Toolbar"> > <!ENTITY devToolbarMenu.label "Developer Toolbar"> > <!ENTITY devToolbarMenu.accesskey "v"> > <!ENTITY devAppMgrMenu.label "App Manager"> > +<!ENTITY devAppMgrMenu.accesskey "A"> Please update the entity name here since other locales that continue to use "App Manager" as the label may want to update their accesskey to match. ::: browser/locales/en-US/chrome/browser/devtools/styleeditor.properties @@ +57,5 @@ > open.commandkey=VK_F7 > > # LOCALIZATION NOTE (open.accesskey): The access key used to open the style > # editor. > +open.accesskey=l Ditto.
Attachment #8412620 -
Flags: review?(jaws) → review-
Assignee | ||
Comment 3•10 years ago
|
||
From IRC: <I> 23.04.2014 10:17:02 hi, can you please take a look at https://bugzilla.mozilla.org/show_bug.cgi?id=1000002 and tell me if it requires a string change? <flod> 23.04.2014 10:41:13 no, accesskeys don't require new ID, since each locale has its own But I can see where you are coming from. ni Pike
Flags: needinfo?(l10n)
Comment 4•10 years ago
|
||
Yeah, no ID change. Might be good to post to .l10n to get folks' attention, though.
Flags: needinfo?(l10n)
Comment 5•10 years ago
|
||
Comment on attachment 8412620 [details] [diff] [review] patch, v1 Review of attachment 8412620 [details] [diff] [review]: ----------------------------------------------------------------- Ok, sounds good. Thanks for double-checking. Can you send the post to the l10n mailing list?
Attachment #8412620 -
Flags: review- → review+
Assignee | ||
Comment 6•10 years ago
|
||
Thanks, I will post to mozilla.dev.l10n after the patch has landed.
Attachment #8412620 -
Attachment is obsolete: true
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 7•10 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/4af2b4c97db5
Keywords: checkin-needed
Whiteboard: [lang=text][mentor=aryx] → [lang=text][mentor=aryx][fixed-in-fx-team]
Comment 8•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/4af2b4c97db5
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [lang=text][mentor=aryx][fixed-in-fx-team] → [lang=text][mentor=aryx]
Target Milestone: --- → Firefox 31
Updated•10 years ago
|
QA Whiteboard: [good first verify]
You need to log in
before you can comment on or make changes to this bug.
Description
•