@vaadin/grid 23.1.0-beta4 → 23.1.0-rc3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vaadin/grid",
-  "version": "23.1.0-beta4",
+  "version": "23.1.0-rc3",
   "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-beta4",
-    "@vaadin/component-base": "23.1.0-beta4",
-    "@vaadin/lit-renderer": "23.1.0-beta4",
-    "@vaadin/text-field": "23.1.0-beta4",
-    "@vaadin/vaadin-lumo-styles": "23.1.0-beta4",
-    "@vaadin/vaadin-material-styles": "23.1.0-beta4",
-    "@vaadin/vaadin-themable-mixin": "23.1.0-beta4"
+    "@vaadin/checkbox": "23.1.0-rc3",
+    "@vaadin/component-base": "23.1.0-rc3",
+    "@vaadin/lit-renderer": "23.1.0-rc3",
+    "@vaadin/text-field": "23.1.0-rc3",
+    "@vaadin/vaadin-lumo-styles": "23.1.0-rc3",
+    "@vaadin/vaadin-material-styles": "23.1.0-rc3",
+    "@vaadin/vaadin-themable-mixin": "23.1.0-rc3"
   },
   "devDependencies": {
     "@esm-bundle/chai": "^4.3.4",
-    "@vaadin/polymer-legacy-adapter": "23.1.0-beta4",
+    "@vaadin/polymer-legacy-adapter": "23.1.0-rc3",
     "@vaadin/testing-helpers": "^0.3.2",
     "lit": "^2.0.0",
     "sinon": "^13.0.2"
   },
-  "gitHead": "06e283473964ecb3085aacf3eddb5333d052a045"
+  "gitHead": "49c312fbe0228adb559296d45655bbfd4eac6235"
 }

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

@@ -16,7 +16,9 @@ class AbstractGridColumnRendererDirective extends LitRendererDirective {
    *
    * @abstract
    */
-  rendererProperty;
+  get rendererProperty() {
+    throw new Error('The `rendererProperty` getter must be implemented.');
+  }
 
   /**
    * Adds the renderer callback to the grid column.
@@ -47,7 +49,9 @@ class AbstractGridColumnRendererDirective extends LitRendererDirective {
 }
 
 export class GridColumnBodyRendererDirective extends AbstractGridColumnRendererDirective {
-  rendererProperty = 'renderer';
+  get rendererProperty() {
+    return 'renderer';
+  }
 
   addRenderer() {
     this.element[this.rendererProperty] = (root, column, model) => {
@@ -57,11 +61,15 @@ export class GridColumnBodyRendererDirective extends AbstractGridColumnRendererD
 }
 
 export class GridColumnHeaderRendererDirective extends AbstractGridColumnRendererDirective {
-  rendererProperty = 'headerRenderer';
+  get rendererProperty() {
+    return 'headerRenderer';
+  }
 }
 
 export class GridColumnFooterRendererDirective extends AbstractGridColumnRendererDirective {
-  rendererProperty = 'footerRenderer';
+  get rendererProperty() {
+    return 'footerRenderer';
+  }
 }
 
 /**

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

@@ -27,13 +27,13 @@ export const SelectionMixin = (superClass) =>
          */
         __selectedKeys: {
           type: Object,
-          value: () => new Set(),
+          computed: '__computeSelectedKeys(itemIdPath, selectedItems.*)',
         },
       };
     }
 
     static get observers() {
-      return ['_updateSelectedKeys(itemIdPath, selectedItems.*)'];
+      return ['__selectedItemsChanged(itemIdPath, selectedItems.*)'];
     }
 
     /**
@@ -85,14 +85,19 @@ export const SelectionMixin = (superClass) =>
     }
 
     /** @private */
-    _updateSelectedKeys() {
-      const selectedItems = this.selectedItems || [];
-      this.__selectedKeys = new Set();
-      selectedItems.forEach((item) => {
-        this.__selectedKeys.add(this.getItemId(item));
+    __selectedItemsChanged() {
+      this.requestContentUpdate();
+    }
+
+    /** @private */
+    __computeSelectedKeys(itemIdPath, selectedItems) {
+      const selected = selectedItems.base || [];
+      const selectedKeys = new Set();
+      selected.forEach((item) => {
+        selectedKeys.add(this.getItemId(item));
       });
 
-      this.requestContentUpdate();
+      return selectedKeys;
     }
 
     /**

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.