@vaadin/component-base 24.8.0-alpha1 → 24.8.0-alpha10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vaadin/component-base",
-  "version": "24.8.0-alpha1",
+  "version": "24.8.0-alpha10",
   "publishConfig": {
     "access": "public"
   },
@@ -38,10 +38,10 @@
     "lit": "^3.0.0"
   },
   "devDependencies": {
-    "@vaadin/chai-plugins": "24.8.0-alpha1",
-    "@vaadin/test-runner-commands": "24.8.0-alpha1",
+    "@vaadin/chai-plugins": "24.8.0-alpha10",
+    "@vaadin/test-runner-commands": "24.8.0-alpha10",
     "@vaadin/testing-helpers": "^1.1.0",
     "sinon": "^18.0.0"
   },
-  "gitHead": "dfec8767c5d1c343ae689c93429c3364f4182f76"
+  "gitHead": "f8c79ffc67eccc3ade226dfe52fbf7d3d46428cf"
 }

package/src/define.js

@@ -13,7 +13,7 @@ function dashToCamelCase(dash) {
 
 const experimentalMap = {};
 
-export function defineCustomElement(CustomElement, version = '24.8.0-alpha1') {
+export function defineCustomElement(CustomElement, version = '24.8.0-alpha10') {
   Object.defineProperty(CustomElement, 'version', {
     get() {
       return version;

package/src/i18n-mixin.d.ts

@@ -5,17 +5,6 @@
  */
 import type { Constructor } from '@open-wc/dedupe-mixin';
 
-/**
- * Recursively makes all properties of an i18n object optional.
- *
- * For internal use only.
- */
-export type PartialI18n<T> = T extends object
-  ? {
-      [P in keyof T]?: PartialI18n<T[P]>;
-    }
-  : T;
-
 /**
  * A mixin that allows to set partial I18N properties.
  */

package/src/i18n-mixin.js

@@ -41,6 +41,16 @@ export const I18nMixin = (defaultI18n, superClass) =>
   class I18nMixinClass extends superClass {
     static get properties() {
       return {
+        /** @private */
+        // Technically declaring a Polymer property is not needed, as we have a
+        // getter/setter for it below. However, the React components currently
+        // rely on the Polymer property declaration to detect which properties
+        // are available on a custom element, so we add a dummy declaration for
+        // it.
+        i18n: {
+          type: Object,
+        },
+
         /** @private */
         __effectiveI18n: {
           type: Object,
@@ -80,6 +90,9 @@ export const I18nMixin = (defaultI18n, superClass) =>
      * @param {Object} value
      */
     set i18n(value) {
+      if (value === this.__customI18n) {
+        return;
+      }
       this.__customI18n = value;
       this.__effectiveI18n = deepMerge({}, defaultI18n, this.__customI18n);
     }

package/src/slot-styles-mixin.js

@@ -50,7 +50,7 @@ export const SlotStylesMixin = dedupingMixin(
        * @protected
        */
       get slotStyles() {
-        return {};
+        return [];
       }
 
       /** @protected */

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

@@ -178,6 +178,16 @@ export class IronListAdapter {
     }
   }
 
+  hostConnected() {
+    // Restore scroll position, which is reset when host is removed from DOM,
+    // since virtualizer doesn't re-render when adding it to the DOM again.
+    // If the scroll target isn't visible and its `offsetParent` is `null`, wait
+    // for the ResizeObserver to handle this case (hiding -> moving -> showing).
+    if (this.scrollTarget.offsetParent && this.scrollTarget.scrollTop !== this._scrollPosition) {
+      this.scrollTarget.scrollTop = this._scrollPosition;
+    }
+  }
+
   update(startIndex = 0, endIndex = this.size - 1) {
     const updatedElements = [];
     this.__getVisibleElements().forEach((el) => {

package/src/virtualizer.js

@@ -80,4 +80,13 @@ export class Virtualizer {
   flush() {
     this.__adapter.flush();
   }
+
+  /**
+   * Notifies the virtualizer about its host element connected to the DOM.
+   *
+   * @method hostConnected
+   */
+  hostConnected() {
+    this.__adapter.hostConnected();
+  }
 }