flush and explain why

PolymerElements · Apr 13, 2017 · b6f5441 · b6f5441
1 parent e599dde
commit b6f5441
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/iron-fit-behavior.html b/iron-fit-behavior.html
@@ -255,14 +255,16 @@
       }
       this.positionTarget = this.positionTarget || this._defaultPositionTarget;
       if (this.autoFitOnAttach) {
-        // If invisible or in a shadow root that's still not connected, wait a tick.
-        if (window.getComputedStyle(this).display === 'none' ||
-          document.body.contains(this) === false) {
+        if (window.getComputedStyle(this).display === 'none') {
           this.__deferredFit = setTimeout(function() {
             this.__deferredFit = null;
             this.fit();
           }.bind(this));
         } else {
+          // NOTE: shadydom won't apply the upgrades synchronously for performance reasons
+          // but this might lead to wrong layout information webcomponents/shadydom#120
+          // We flush because we need correct layout bounding rect info.
+          window.ShadyDOM && ShadyDOM.flush();
           this.fit();
         }
       }