@vaadin/component-base 24.0.0-alpha6 → 24.0.0-alpha7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vaadin/component-base",
-  "version": "24.0.0-alpha6",
+  "version": "24.0.0-alpha7",
   "publishConfig": {
     "access": "public"
   },
@@ -42,5 +42,5 @@
     "@vaadin/testing-helpers": "^0.3.2",
     "sinon": "^13.0.2"
   },
-  "gitHead": "0004ac92b6e5f415b5fa949e0582d1d11e527b1f"
+  "gitHead": "aeb4535336813636736759e0a5de148b26bfc3b6"
 }

package/src/element-mixin.js

@@ -15,14 +15,14 @@ import { DirMixin } from './dir-mixin.js';
 // for buttons that are based on `[role=button]` e.g vaadin-button.
 setCancelSyntheticClickEvents(false);
 
-window.Vaadin = window.Vaadin || {};
+window.Vaadin ||= {};
 
 /**
  * Array of Vaadin custom element classes that have been finalized.
  */
-window.Vaadin.registrations = window.Vaadin.registrations || [];
+window.Vaadin.registrations ||= [];
 
-window.Vaadin.developmentModeCallback = window.Vaadin.developmentModeCallback || {};
+window.Vaadin.developmentModeCallback ||= {};
 
 window.Vaadin.developmentModeCallback['vaadin-usage-statistics'] = function () {
   usageStatistics();
@@ -39,7 +39,7 @@ const registered = new Set();
 export const ElementMixin = (superClass) =>
   class VaadinElementMixin extends DirMixin(superClass) {
     static get version() {
-      return '24.0.0-alpha6';
+      return '24.0.0-alpha7';
     }
 
     /** @protected */

package/src/iron-list-core.js

@@ -491,9 +491,9 @@ export const ironList = {
       this._physicalIndexForKey = {};
       this._firstVisibleIndexVal = null;
       this._lastVisibleIndexVal = null;
-      this._physicalCount = this._physicalCount || 0;
-      this._physicalItems = this._physicalItems || [];
-      this._physicalSizes = this._physicalSizes || [];
+      this._physicalCount ||= 0;
+      this._physicalItems ||= [];
+      this._physicalSizes ||= [];
       this._physicalStart = 0;
       if (this._scrollTop > this._scrollOffset) {
         this._resetScrollPosition(0);
@@ -639,8 +639,8 @@ export const ironList = {
       this._physicalBottom +
       Math.max(this._virtualCount - this._physicalCount - this._virtualStart, 0) * this._physicalAverage;
 
-    forceUpdate = forceUpdate || this._scrollHeight === 0;
-    forceUpdate = forceUpdate || this._scrollPosition >= this._estScrollHeight - this._physicalSize;
+    forceUpdate ||= this._scrollHeight === 0;
+    forceUpdate ||= this._scrollPosition >= this._estScrollHeight - this._physicalSize;
     // Amortize height adjustment, so it won't trigger large repaints too often.
     if (forceUpdate || Math.abs(this._estScrollHeight - this._scrollHeight) >= this._viewportHeight) {
       this.$.items.style.height = `${this._estScrollHeight}px`;
@@ -740,7 +740,7 @@ export const ironList = {
   },
 
   _debounce(name, cb, asyncModule) {
-    this._debouncers = this._debouncers || {};
+    this._debouncers ||= {};
     this._debouncers[name] = Debouncer.debounce(this._debouncers[name], asyncModule, cb.bind(this));
     enqueueDebouncer(this._debouncers[name]);
   },

package/src/overflow-controller.js

@@ -92,7 +92,7 @@ export class OverflowController {
       overflow += ' top';
     }
 
-    if (target.scrollTop < target.scrollHeight - target.clientHeight) {
+    if (Math.ceil(target.scrollTop) < Math.ceil(target.scrollHeight - target.clientHeight)) {
       overflow += ' bottom';
     }
 
@@ -101,7 +101,7 @@ export class OverflowController {
       overflow += ' start';
     }
 
-    if (scrollLeft < target.scrollWidth - target.clientWidth) {
+    if (Math.ceil(scrollLeft) < Math.ceil(target.scrollWidth - target.clientWidth)) {
       overflow += ' end';
     }
 

package/src/polylit-mixin.js

@@ -10,7 +10,7 @@ const caseMap = {};
 const CAMEL_TO_DASH = /([A-Z])/g;
 
 function camelToDash(camel) {
-  caseMap[camel] = caseMap[camel] || camel.replace(CAMEL_TO_DASH, '-$1').toLowerCase();
+  caseMap[camel] ||= camel.replace(CAMEL_TO_DASH, '-$1').toLowerCase();
   return caseMap[camel];
 }
 
@@ -169,7 +169,7 @@ const PolylitMixinImplementation = (superclass) => {
     firstUpdated() {
       super.firstUpdated();
 
-      this.$ = this.$ || {};
+      this.$ ||= {};
       this.shadowRoot.querySelectorAll('[id]').forEach((node) => {
         this.$[node.id] = node;
       });

package/src/slot-controller.js

@@ -14,12 +14,12 @@ export class SlotController extends EventTarget {
   /**
    * Ensure that every instance has unique ID.
    *
-   * @param {string} slotName
    * @param {HTMLElement} host
+   * @param {string} slotName
    * @return {string}
    * @protected
    */
-  static generateId(slotName, host) {
+  static generateId(host, slotName) {
     const prefix = slotName || 'default';
     return `${prefix}-${host.localName}-${generateUniqueId()}`;
   }
@@ -42,7 +42,7 @@ export class SlotController extends EventTarget {
 
     // Only generate the default ID if requested by the controller.
     if (useUniqueId) {
-      this.defaultId = SlotController.generateId(slotName, host);
+      this.defaultId = this.constructor.generateId(host, slotName);
     }
   }
 

package/src/slot-observe-controller.js

@@ -107,7 +107,7 @@ export class SlotObserveController extends SlotController {
         if (mutation.type === 'attributes') {
           // We use attributeFilter to only observe ID mutation,
           // no need to check for attribute name separately.
-          if (isCurrentNodeMutation && target.id !== this.defaultId) {
+          if (isCurrentNodeMutation) {
             this.__updateNodeId(target);
           }
         } else if (isCurrentNodeMutation || target.parentElement === this.node) {
@@ -141,8 +141,7 @@ export class SlotObserveController extends SlotController {
     }
 
     return (
-      node.children.length > 0 ||
-      (node.nodeType === Node.ELEMENT_NODE && customElements.get(node.localName)) ||
+      (node.nodeType === Node.ELEMENT_NODE && (customElements.get(node.localName) || node.children.length > 0)) ||
       (node.textContent && node.textContent.trim() !== '')
     );
   }
@@ -168,7 +167,9 @@ export class SlotObserveController extends SlotController {
    * @private
    */
   __updateNodeId(node) {
-    if (node.nodeType === Node.ELEMENT_NODE && !node.id) {
+    // When in multiple mode, only set ID attribute on the element in default slot.
+    const isFirstNode = !this.nodes || node === this.nodes[0];
+    if (node.nodeType === Node.ELEMENT_NODE && isFirstNode && !node.id) {
       node.id = this.defaultId;
     }
   }

package/src/virtualizer-iron-list-adapter.js

@@ -376,7 +376,7 @@ export class IronListAdapter {
       deltaY *= this._scrollPageHeight;
     }
 
-    this._deltaYAcc = this._deltaYAcc || 0;
+    this._deltaYAcc ||= 0;
 
     if (this._wheelAnimationFrame) {
       // Accumulate wheel delta while a frame is being processed