|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
59 bytes, text/x-review-board-request
|Details | Review|
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.
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. :-)
Comment on attachment 8866882 [details] Bug 1354128 - Add Web Developer button to the photon app menu. https://reviewboard.mozilla.org/r/138480/#review142748
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.