Closed Bug 1354128 Opened 4 years ago Closed 4 years ago
Add 'Web Developer' button to static hamburger menu
59 bytes, text/x-review-board-request
This button opens a subview when clicked, which contains the same items as the current 'Developer' subiew widget. The title of the subview reads 'Web Developer', the same as the button label. The button should be placed right below the 'More' button.
Priority: -- → P2
QA Contact: gwimberly
Whiteboard: [photon] → [photon-structure]
Assignee: nobody → mdeboer
Status: NEW → ASSIGNED
Comment on attachment 8866882 [details] Bug 1354128 - Add Web Developer button to the photon app menu. https://reviewboard.mozilla.org/r/138480/#review142084 ::: browser/components/customizableui/PanelMultiView.jsm:398 (Diff revision 1) > + if (custWidget.onBeforeCreated) > + custWidget.onBeforeCreated(document); Hrm. onBeforeCreated normally is used when we create the widget (ie toolbarbutton) in the document. Now it might be called twice for a given document. Can we refactor this differently, maybe by giving the widget an onViewShowing method and/or ensuring that calls some other method on its definition that guards against being called multiple times? ::: browser/components/customizableui/content/panelUI.inc.xul:584 (Diff revision 1) > <toolbarbutton id="appMenu-more-button" > class="subviewbutton subviewbutton-nav" > label="&moreMenu.label;" > closemenu="none" > oncommand="PanelUI.showSubView('appMenu-moreView', this)"/> > + <toolbarbutton id="appMenu-more-button" The ID is wrong here. :-)
Attachment #8866882 - Flags: review?(gijskruitbosch+bugs)
Comment on attachment 8866882 [details] Bug 1354128 - Add Web Developer button to the photon app menu. https://reviewboard.mozilla.org/r/138480/#review142748
Attachment #8866882 - Flags: review?(gijskruitbosch+bugs) → review+
Thanks, and sorry for the delay. I'll try to get to the other reviews tomorrow.
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/5f44d08bfcb9 Add Web Developer button to the photon app menu. r=Gijs
Verified on Windows, Mac, and Ubuntu.
You need to log in before you can comment on or make changes to this bug.