@vaadin/vaadin-board 2.2.1 → 2.2.2

package/package.json

@@ -13,7 +13,7 @@
   "repository": "vaadin/vaadin-board",
   "homepage": "https://vaadin.com/components",
   "name": "@vaadin/vaadin-board",
-  "version": "2.2.1",
+  "version": "2.2.2",
   "main": "vaadin-board.js",
   "author": "Vaadin Ltd",
   "license": "https://raw.githubusercontent.com/vaadin/vaadin-board/master/LICENSE.txt",

package/src/vaadin-board-row.js

@@ -232,9 +232,7 @@ class BoardRowElement extends ElementMixin(mixinBehaviors([IronResizableBehavior
   _recalculateFlexBasis(forceResize) {
     const width = this.getBoundingClientRect().width;
     const breakpoints = this._measureBreakpointsInPx();
-    if (forceResize || width != this._oldWidth
-        || breakpoints.smallSize != this._oldBreakpoints.smallSize
-        || breakpoints.mediumSize != this._oldBreakpoints.mediumSize) {
+    if (forceResize || this._shouldRecalculate(width, breakpoints)) {
       const nodes = this.$.insertionPoint.assignedNodes({ flatten: true });
       const isElementNode = function (node) {
         return !((node.nodeType === node.TEXT_NODE)
@@ -257,6 +255,37 @@ class BoardRowElement extends ElementMixin(mixinBehaviors([IronResizableBehavior
     }
   }
 
+  /** @private */
+  _shouldRecalculate(width, breakpoints) {
+    // Should not recalculate if row is invisible
+    if (this._isElementHidden()) {
+      return false;
+    }
+    return (
+      width !== this._oldWidth ||
+      breakpoints.smallSize !== this._oldBreakpoints.smallSize ||
+      breakpoints.mediumSize !== this._oldBreakpoints.mediumSize
+    );
+  }
+
+  /** @private */
+  _isElementHidden() {
+    // `offsetParent` is `null` when the element itself
+    // or one of its ancestors is hidden with `display: none`.
+    // https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/offsetParent
+    // However `offsetParent` is also null when the element is using fixed
+    // positioning, so additionally check if the element takes up layout space.
+    if (this.offsetParent === null && this.clientWidth === 0 && this.clientHeight === 0) {
+      return true;
+    }
+    // Check inline style first to save a re-flow.
+    if (this.style.visibility === 'hidden' || this.style.display === 'none') {
+      return true;
+    }
+    const computedStyle = window.getComputedStyle(this);
+    return computedStyle.visibility === 'hidden' || computedStyle.display === 'none';
+  }
+
   /**
    * Measure the breakpoints in pixels.
    *

package/src/vaadin-board.js

@@ -51,7 +51,7 @@ class BoardElement extends ElementMixin(mixinBehaviors([IronResizableBehavior],
   }
 
   static get version() {
-    return '2.2.1';
+    return '2.2.2';
   }
 
   static get cvdlName() {