Bug 1354109 - fix min-width issues with panels, r?mikedeboer draft
authorGijs Kruitbosch <gijskruitbosch@gmail.com>
Tue, 06 Jun 2017 16:12:40 +0100
changeset 589626 c93d2ce82776bd268a36499652d87e0c7a58d7d3
parent 589625 497b7203592d0051d343e27982c2ba6465e8307b
child 631962 36d55db9fac5fe6e4b4aa9b08e94cccd93f059b1
push id62459
push usergijskruitbosch@gmail.com
push dateTue, 06 Jun 2017 15:36:39 +0000
reviewersmikedeboer
bugs1354109
milestone55.0a1
Bug 1354109 - fix min-width issues with panels, r?mikedeboer MozReview-Commit-ID: 3h5hCvcJbTc
browser/components/customizableui/PanelMultiView.jsm
browser/themes/shared/customizableui/panelUI.inc.css
--- a/browser/components/customizableui/PanelMultiView.jsm
+++ b/browser/components/customizableui/PanelMultiView.jsm
@@ -823,16 +823,22 @@ this.PanelMultiView = class {
         // Now that the main view is visible, we can check the height of the
         // description elements it contains.
         this.descriptionHeightWorkaround();
         break;
       case "popuphidden":
         this.node.removeAttribute("panelopen");
         this.showMainView();
         if (this.panelViews) {
+          for (let panelView of this._viewStack.children) {
+            if (panelView.nodeName != "children") {
+              panelView.style.removeProperty("min-width");
+              panelView.style.removeProperty("max-width");
+            }
+          }
           this.window.removeEventListener("keydown", this);
           this._panel.removeEventListener("mousemove", this);
           this._resetKeyNavigation();
           this._mainViewHeight = 0;
         }
         break;
     }
   }
--- a/browser/themes/shared/customizableui/panelUI.inc.css
+++ b/browser/themes/shared/customizableui/panelUI.inc.css
@@ -341,16 +341,19 @@ panelview:not([mainview]) .toolbarbutton
 #appMenu-popup > .panel-arrowcontainer > .panel-arrowcontent {
   padding: 0;
   border-radius: 0;
 }
 
 photonpanelmultiview panelview {
   background: var(--arrowpanel-background);
   padding: 6px 0;
+}
+
+#appMenu-popup panelview {
   min-width: @menuPanelWidth@;
 }
 
 photonpanelmultiview panelview[title] {
   padding-top: 0;
 }
 
 photonpanelmultiview .panel-subview-body {