Closed Bug 952742 Opened 11 years ago Closed 10 years ago

[Australis] "Move to menu" label is using the wrong entity for accesskey

Categories

(Firefox :: Toolbars and Customization, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 29

People

(Reporter: flod, Assigned: flod)

References

Details

Attachments

(1 file)

http://mxr.mozilla.org/mozilla-central/source/browser/base/content/browser.xul#249

247       <menuitem oncommand="gCustomizeMode.addToPanel(document.popupNode)"
248                 accesskey="&customizeMenu.addToPanel.accesskey;"
249                 label="&customizeMenu.moveToPanel.label;"
250                 class="customize-context-moveToPanel"/>

Accesskey should set to "customizeMenu.moveToPanel.accesskey"

Reference changeset: https://hg.mozilla.org/mozilla-central/rev/8e25403f7096
Whiteboard: [Australis:P4]
Unless I'm missing something obvious, this should be enough to fix it.
Assignee: nobody → francesco.lodolo
Attachment #8350897 - Flags: review?(dolske)
Summary: [Australis] "Move to menu" label is using the wrong accesskey → [Australis] "Move to menu" label is using the wrong entity for accesskey
Comment on attachment 8350897 [details] [diff] [review]
Replace entity name in accesskey attribute

Review of attachment 8350897 [details] [diff] [review]:
-----------------------------------------------------------------

derp=me. Nice catch.
Attachment #8350897 - Flags: review?(dolske) → review+
Keywords: checkin-needed
https://hg.mozilla.org/integration/fx-team/rev/79e3fe8c4ba9
Flags: in-testsuite?
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/79e3fe8c4ba9
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 29
Verified long time ago.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: