@vaadin/grid - npm Package Compare versions

Comparing version 23.0.0-alpha5 to 23.0.0-beta1

package.json

 {
   "name": "@vaadin/grid",
-  "version": "23.0.0-alpha5",
+  "version": "23.0.0-beta1",
   "publishConfig": {
@@ -44,12 +44,12 @@ "access": "public"
     "@polymer/polymer": "^3.0.0",
-    "@vaadin/checkbox": "23.0.0-alpha5",
-    "@vaadin/component-base": "23.0.0-alpha5",
-    "@vaadin/text-field": "23.0.0-alpha5",
-    "@vaadin/vaadin-lumo-styles": "23.0.0-alpha5",
-    "@vaadin/vaadin-material-styles": "23.0.0-alpha5",
-    "@vaadin/vaadin-themable-mixin": "23.0.0-alpha5"
+    "@vaadin/checkbox": "23.0.0-beta1",
+    "@vaadin/component-base": "23.0.0-beta1",
+    "@vaadin/text-field": "23.0.0-beta1",
+    "@vaadin/vaadin-lumo-styles": "23.0.0-beta1",
+    "@vaadin/vaadin-material-styles": "23.0.0-beta1",
+    "@vaadin/vaadin-themable-mixin": "23.0.0-beta1"
   },
   "devDependencies": {
     "@esm-bundle/chai": "^4.3.4",
-    "@vaadin/polymer-legacy-adapter": "23.0.0-alpha5",
+    "@vaadin/polymer-legacy-adapter": "23.0.0-beta1",
     "@vaadin/testing-helpers": "^0.3.2",
@@ -59,3 +59,3 @@ "lit": "^2.0.0",
   },
-  "gitHead": "74f9294964eb8552d96578c14af6ad214f5257bc"
+  "gitHead": "467244b76021176c109df675799b07029b293e58"
 }

src/vaadin-grid-column.js

@@ -500,3 +500,5 @@ /**
       this.__renderCellsContent(headerRenderer, [headerCell]);
-      this._grid.__updateHeaderFooterRowVisibility(headerCell.parentElement);
+      if (this._grid) {
+        this._grid.__updateHeaderFooterRowVisibility(headerCell.parentElement);
+      }
     }
@@ -540,3 +542,5 @@
       this.__renderCellsContent(footerRenderer, [footerCell]);
-      this._grid.__updateHeaderFooterRowVisibility(footerCell.parentElement);
+      if (this._grid) {
+        this._grid.__updateHeaderFooterRowVisibility(footerCell.parentElement);
+      }
     }
@@ -543,0 +547,0 @@

src/vaadin-grid-keyboard-navigation-mixin.js

@@ -134,2 +134,33 @@ /**
     /**
+     * Since the focused cell/row state is stored as an element reference, the reference may get
+     * out of sync when the virtual indexes for elements update due to effective size change.
+     * This function updates the reference to the correct element after a possible index change.
+     * @private
+     */
+    __updateItemsFocusable() {
+      if (!this._itemsFocusable) {
+        return;
+      }
+
+      const wasFocused = this.shadowRoot.activeElement === this._itemsFocusable;
+
+      this._getVisibleRows().forEach((row) => {
+        if (row.index === this._focusedItemIndex) {
+          if (this.__rowFocusMode) {
+            // Row focus mode
+            this._itemsFocusable = row;
+          } else if (this._itemsFocusable.parentElement) {
+            // Cell focus mode
+            const cellIndex = [...this._itemsFocusable.parentElement.children].indexOf(this._itemsFocusable);
+            this._itemsFocusable = row.children[cellIndex];
+          }
+        }
+      });
+
+      if (wasFocused) {
+        this._itemsFocusable.focus();
+      }
+    }
+
+    /**
      * @param {!KeyboardEvent} e
@@ -535,3 +566,7 @@ * @protected
         if (wantInteracting) {
-          const focusTarget = cell._content.querySelector('[focus-target]') || cell._content.firstElementChild;
+          const focusTarget =
+            cell._content.querySelector('[focus-target]') ||
+            // If a child element hasn't been explicitly marked as a focus target,
+            // fall back to any focusable element inside the cell.
+            [...cell._content.querySelectorAll('*')].find((node) => this._isFocusable(node));
           if (focusTarget) {
@@ -810,2 +845,4 @@ e.preventDefault();
         }
+      } else {
+        this.__updateItemsFocusable();
       }
@@ -812,0 +849,0 @@ }

src/vaadin-grid-row-details-mixin.js

@@ -86,3 +86,5 @@ /**
             this._updateRow(row, this._columnTree[this._columnTree.length - 1]);
-            this._a11yUpdateRowDetailsOpened(row, false);
+            const isDetailsOpened = this._isDetailsOpened(row._item);
+            this._a11yUpdateRowDetailsOpened(row, isDetailsOpened);
+            this._toggleDetailsCell(row, isDetailsOpened);
           }
@@ -89,0 +91,0 @@ });

src/vaadin-grid-selection-mixin.js

@@ -80,8 +80,4 @@ /**
     /** @private */
-    _selectedItemsChanged(e) {
-      if (this.$.items.children.length && (e.path === 'selectedItems' || e.path === 'selectedItems.splices')) {
-        Array.from(this.$.items.children).forEach((row) => {
-          this._updateItem(row, row._item);
-        });
-      }
+    _selectedItemsChanged() {
+      this.requestContentUpdate();
     }
@@ -88,0 +84,0 @@

No alert changes

Improved metrics

Total package byte prevSize

328967

increased by0.41%

Lines of code

8943

increased by0.39%

Dependency changes

• + Added@vaadin/checkbox@23.0.0-beta1(transitive)

• + Added@vaadin/component-base@23.0.0-beta1(transitive)

• + Added@vaadin/field-base@23.0.0-beta1(transitive)

• + Added@vaadin/icon@23.0.0-beta1(transitive)

• + Added@vaadin/input-container@23.0.0-beta1(transitive)

• + Added@vaadin/text-field@23.0.0-beta1(transitive)

• + Added@vaadin/vaadin-lumo-styles@23.0.0-beta1(transitive)

• + Added@vaadin/vaadin-material-styles@23.0.0-beta1(transitive)

• + Added@vaadin/vaadin-themable-mixin@23.0.0-beta1(transitive)

• - Removed@vaadin/checkbox@23.0.0-alpha5(transitive)

• - Removed@vaadin/component-base@23.0.0-alpha5(transitive)

• - Removed@vaadin/field-base@23.0.0-alpha5(transitive)

• - Removed@vaadin/icon@23.0.0-alpha5(transitive)

• - Removed@vaadin/input-container@23.0.0-alpha5(transitive)

• - Removed@vaadin/text-field@23.0.0-alpha5(transitive)

• - Removed@vaadin/vaadin-lumo-styles@23.0.0-alpha5(transitive)

• - Removed@vaadin/vaadin-material-styles@23.0.0-alpha5(transitive)

• - Removed@vaadin/vaadin-themable-mixin@23.0.0-alpha5(transitive)

• Updated@vaadin/checkbox@23.0.0-beta1

• Updated@vaadin/component-base@23.0.0-beta1

• Updated@vaadin/text-field@23.0.0-beta1

• Updated@vaadin/vaadin-lumo-styles@23.0.0-beta1

• Updated@vaadin/vaadin-material-styles@23.0.0-beta1

• Updated@vaadin/vaadin-themable-mixin@23.0.0-beta1