author | Mike de Boer <mdeboer@mozilla.com> |
Fri, 08 Sep 2017 11:53:11 +0200 | |
changeset 379705 | 5a0fd0db0fb0188b7fe8a0db79138b72f977ebc0 |
parent 379704 | 9b99855c32cac452f6e6ded85e6b96769f9d6318 |
child 379706 | 5c148323ff77d13366337adf2ae343c573624354 |
push id | 32461 |
push user | kwierso@gmail.com |
push date | Fri, 08 Sep 2017 20:15:32 +0000 |
treeherder | mozilla-central@dd3736e98e4e [default view] [failures only] |
perfherder | [talos] [build metrics] [platform microbench] (compared to previous push) |
reviewers | Gijs |
bugs | 1390336 |
milestone | 57.0a1 |
first release with | nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
|
last release without | nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
|
--- a/browser/components/customizableui/PanelMultiView.jsm +++ b/browser/components/customizableui/PanelMultiView.jsm @@ -874,17 +874,17 @@ this.PanelMultiView = class { case "popupshowing": this.node.setAttribute("panelopen", "true"); // Bug 941196 - The panel can get taller when opening a subview. Disabling // autoPositioning means that the panel won't jump around if an opened // subview causes the panel to exceed the dimensions of the screen in the // direction that the panel originally opened in. This property resets // every time the popup closes, which is why we have to set it each time. this._panel.autoPosition = false; - if (this.panelViews) { + if (this.panelViews && !this.node.hasAttribute("disablekeynav")) { this.window.addEventListener("keydown", this); this._panel.addEventListener("mousemove", this); } // Before opening the panel, we have to limit the maximum height of any // view based on the space that will be available. We cannot just use // window.screen.availTop and availHeight because these may return an // incorrect value when the window spans multiple screens.
--- a/browser/components/customizableui/content/panelUI.inc.xul +++ b/browser/components/customizableui/content/panelUI.inc.xul @@ -327,17 +327,17 @@ <panel id="widget-overflow" role="group" type="arrow" noautofocus="true" position="bottomcenter topright" photon="true" hidden="true"> - <photonpanelmultiview mainViewId="widget-overflow-mainView"> + <photonpanelmultiview mainViewId="widget-overflow-mainView" disablekeynav="true"> <panelview id="widget-overflow-mainView" context="toolbar-context-menu"> <vbox class="panel-subview-body"> <vbox id="widget-overflow-list" class="widget-overflow-list" overflowfortoolbar="nav-bar"/> <toolbarseparator id="widget-overflow-fixed-separator" hidden="true"/> <vbox id="widget-overflow-fixed-list" class="widget-overflow-list" hidden="true" emptylabel="&customizeMode.emptyOverflowList.description;"/>