Bug 983632 - [Australis, Windows HiDPI] button icons can be completely hidden but still be on toolbar and not in overflow menu if window resized/width decreased. r=Gijs
authorJared Wein <jwein@mozilla.com>
Fri, 14 Mar 2014 11:13:41 -0400
changeset 191802 d7e5a58a314df0515526468d2ae93782c9d3c2ea
parent 191801 8f1f641ad257315e989679d39acd172f3e34fa4c
child 191803 26e1df0e1ee1852f951be1aa678be11f6219512f
push id474
push userasasaki@mozilla.com
push dateMon, 02 Jun 2014 21:01:02 +0000
treeherdermozilla-release@967f4cf1b31c [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersGijs
bugs983632
milestone30.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
Bug 983632 - [Australis, Windows HiDPI] button icons can be completely hidden but still be on toolbar and not in overflow menu if window resized/width decreased. r=Gijs
browser/components/customizableui/src/CustomizableUI.jsm
--- a/browser/components/customizableui/src/CustomizableUI.jsm
+++ b/browser/components/customizableui/src/CustomizableUI.jsm
@@ -3676,17 +3676,21 @@ OverflowableToolbar.prototype = {
       CustomizableUI.removeListener(this);
     }
   },
 
   _onLazyResize: function() {
     if (!this._enabled)
       return;
 
-    this._moveItemsBackToTheirOrigin();
+    if (this._target.scrollLeftMax > 0) {
+      this.onOverflow();
+    } else {
+      this._moveItemsBackToTheirOrigin();
+    }
   },
 
   _disable: function() {
     this._enabled = false;
     this._moveItemsBackToTheirOrigin(true);
     if (this._lazyResizeHandler) {
       this._lazyResizeHandler.disarm();
     }