@vaadin/grid 23.1.0-beta2 → 23.1.0-rc1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vaadin/grid",
-  "version": "23.1.0-beta2",
+  "version": "23.1.0-rc1",
   "publishConfig": {
     "access": "public"
   },
@@ -43,20 +43,20 @@
   "dependencies": {
     "@open-wc/dedupe-mixin": "^1.3.0",
     "@polymer/polymer": "^3.0.0",
-    "@vaadin/checkbox": "23.1.0-beta2",
-    "@vaadin/component-base": "23.1.0-beta2",
-    "@vaadin/lit-renderer": "23.1.0-beta2",
-    "@vaadin/text-field": "23.1.0-beta2",
-    "@vaadin/vaadin-lumo-styles": "23.1.0-beta2",
-    "@vaadin/vaadin-material-styles": "23.1.0-beta2",
-    "@vaadin/vaadin-themable-mixin": "23.1.0-beta2"
+    "@vaadin/checkbox": "23.1.0-rc1",
+    "@vaadin/component-base": "23.1.0-rc1",
+    "@vaadin/lit-renderer": "23.1.0-rc1",
+    "@vaadin/text-field": "23.1.0-rc1",
+    "@vaadin/vaadin-lumo-styles": "23.1.0-rc1",
+    "@vaadin/vaadin-material-styles": "23.1.0-rc1",
+    "@vaadin/vaadin-themable-mixin": "23.1.0-rc1"
   },
   "devDependencies": {
     "@esm-bundle/chai": "^4.3.4",
-    "@vaadin/polymer-legacy-adapter": "23.1.0-beta2",
+    "@vaadin/polymer-legacy-adapter": "23.1.0-rc1",
     "@vaadin/testing-helpers": "^0.3.2",
     "lit": "^2.0.0",
     "sinon": "^13.0.2"
   },
-  "gitHead": "f11f9245a0b5e6bf912725a501c27c24b74e7c8d"
+  "gitHead": "5ecb85e16e938df827fefca4bd2a665a1e29913e"
 }

package/src/lit/column-renderer-directives.js

@@ -5,8 +5,8 @@
  */
 /* eslint-disable max-classes-per-file */
 import { directive } from 'lit/directive.js';
-import { microTask } from '@vaadin/component-base/src/async';
-import { Debouncer } from '@vaadin/component-base/src/debounce';
+import { microTask } from '@vaadin/component-base/src/async.js';
+import { Debouncer } from '@vaadin/component-base/src/debounce.js';
 import { LitRendererDirective } from '@vaadin/lit-renderer';
 import { CONTENT_UPDATE_DEBOUNCER } from './renderer-directives.js';
 

package/src/vaadin-grid-data-provider-mixin.d.ts

@@ -18,13 +18,13 @@ export interface GridSorterDefinition {
   direction: GridSorterDirection;
 }
 
-export type GridDataProviderCallback<TItem> = (items: Array<TItem>, size?: number) => void;
+export type GridDataProviderCallback<TItem> = (items: TItem[], size?: number) => void;
 
 export type GridDataProviderParams<TItem> = {
   page: number;
   pageSize: number;
-  filters: Array<GridFilterDefinition>;
-  sortOrders: Array<GridSorterDefinition>;
+  filters: GridFilterDefinition[];
+  sortOrders: GridSorterDefinition[];
   parentItem?: TItem;
 };
 

package/src/vaadin-grid-data-provider-mixin.js

@@ -219,13 +219,13 @@ export const DataProviderMixin = (superClass) =>
          */
         __expandedKeys: {
           type: Object,
-          value: () => new Set(),
+          computed: '__computeExpandedKeys(itemIdPath, expandedItems.*)',
         },
       };
     }
 
     static get observers() {
-      return ['_sizeChanged(size)', '_itemIdPathChanged(itemIdPath)', '_expandedItemsChanged(expandedItems.*)'];
+      return ['_sizeChanged(size)', '_expandedItemsChanged(expandedItems.*)'];
     }
 
     /** @private */
@@ -282,25 +282,20 @@ export const DataProviderMixin = (superClass) =>
 
     /** @private */
     _expandedItemsChanged() {
-      this.__cacheExpandedKeys();
       this._cache.updateSize();
       this._effectiveSize = this._cache.effectiveSize;
       this.__updateVisibleRows();
     }
 
     /** @private */
-    _itemIdPathChanged() {
-      this.__cacheExpandedKeys();
-    }
+    __computeExpandedKeys(itemIdPath, expandedItems) {
+      const expanded = expandedItems.base || [];
+      const expandedKeys = new Set();
+      expanded.forEach((item) => {
+        expandedKeys.add(this.getItemId(item));
+      });
 
-    /** @private */
-    __cacheExpandedKeys() {
-      if (this.expandedItems) {
-        this.__expandedKeys = new Set();
-        this.expandedItems.forEach((item) => {
-          this.__expandedKeys.add(this.getItemId(item));
-        });
-      }
+      return expandedKeys;
     }
 
     /**

package/src/vaadin-grid-row-details-mixin.d.ts

@@ -20,7 +20,7 @@ export declare class RowDetailsMixinClass<TItem> {
   /**
    * An array containing references to items with open row details.
    */
-  detailsOpenedItems: Array<TItem>;
+  detailsOpenedItems: TItem[];
 
   /**
    * Custom function for rendering the content of the row details.

package/src/vaadin-grid-selection-mixin.d.ts

@@ -13,7 +13,7 @@ export declare class SelectionMixinClass<TItem> {
   /**
    * An array that contains the selected items.
    */
-  selectedItems: Array<TItem>;
+  selectedItems: TItem[];
 
   /**
    * Selects the given item.

package/src/vaadin-grid-selection-mixin.js

@@ -20,11 +20,20 @@ export const SelectionMixin = (superClass) =>
           notify: true,
           value: () => [],
         },
+
+        /**
+         * Set of selected item ids
+         * @private
+         */
+        __selectedKeys: {
+          type: Object,
+          computed: '__computeSelectedKeys(itemIdPath, selectedItems.*)',
+        },
       };
     }
 
     static get observers() {
-      return ['_selectedItemsChanged(selectedItems.*)'];
+      return ['__selectedItemsChanged(itemIdPath, selectedItems.*)'];
     }
 
     /**
@@ -33,7 +42,7 @@ export const SelectionMixin = (superClass) =>
      * @protected
      */
     _isSelected(item) {
-      return this.selectedItems && this._getItemIndexInArray(item, this.selectedItems) > -1;
+      return this.__selectedKeys.has(this.getItemId(item));
     }
 
     /**
@@ -68,8 +77,7 @@ export const SelectionMixin = (superClass) =>
      * @protected
      */
     _toggleItem(item) {
-      const index = this._getItemIndexInArray(item, this.selectedItems);
-      if (index === -1) {
+      if (!this._isSelected(item)) {
         this.selectItem(item);
       } else {
         this.deselectItem(item);
@@ -77,10 +85,21 @@ export const SelectionMixin = (superClass) =>
     }
 
     /** @private */
-    _selectedItemsChanged() {
+    __selectedItemsChanged() {
       this.requestContentUpdate();
     }
 
+    /** @private */
+    __computeSelectedKeys(itemIdPath, selectedItems) {
+      const selected = selectedItems.base || [];
+      const selectedKeys = new Set();
+      selected.forEach((item) => {
+        selectedKeys.add(this.getItemId(item));
+      });
+
+      return selectedKeys;
+    }
+
     /**
      * Fired when the `selectedItems` property changes.
      *

package/src/vaadin-grid.d.ts

@@ -78,7 +78,7 @@ export type GridCellFocusEvent<TItem> = CustomEvent<{ context: GridEventContext<
 /**
  * Fired when the columns in the grid are reordered.
  */
-export type GridColumnReorderEvent<TItem> = CustomEvent<{ columns: GridColumn<TItem>[] }>;
+export type GridColumnReorderEvent<TItem> = CustomEvent<{ columns: Array<GridColumn<TItem>> }>;
 
 /**
  * Fired when the grid column resize is finished.