@vaadin/grid 23.0.4 → 23.0.7

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (2) hide show
  1. package/package.json +9 -9
  2. package/src/vaadin-grid.js +1 -25
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@vaadin/grid",
3
- "version": "23.0.4",
3
+ "version": "23.0.7",
4
4
  "publishConfig": {
5
5
  "access": "public"
6
6
  },
@@ -41,19 +41,19 @@
41
41
  "dependencies": {
42
42
  "@open-wc/dedupe-mixin": "^1.3.0",
43
43
  "@polymer/polymer": "^3.0.0",
44
- "@vaadin/checkbox": "^23.0.4",
45
- "@vaadin/component-base": "^23.0.4",
46
- "@vaadin/text-field": "^23.0.4",
47
- "@vaadin/vaadin-lumo-styles": "^23.0.4",
48
- "@vaadin/vaadin-material-styles": "^23.0.4",
49
- "@vaadin/vaadin-themable-mixin": "^23.0.4"
44
+ "@vaadin/checkbox": "^23.0.7",
45
+ "@vaadin/component-base": "^23.0.7",
46
+ "@vaadin/text-field": "^23.0.7",
47
+ "@vaadin/vaadin-lumo-styles": "^23.0.7",
48
+ "@vaadin/vaadin-material-styles": "^23.0.7",
49
+ "@vaadin/vaadin-themable-mixin": "^23.0.7"
50
50
  },
51
51
  "devDependencies": {
52
52
  "@esm-bundle/chai": "^4.3.4",
53
- "@vaadin/polymer-legacy-adapter": "^23.0.4",
53
+ "@vaadin/polymer-legacy-adapter": "^23.0.7",
54
54
  "@vaadin/testing-helpers": "^0.3.2",
55
55
  "lit": "^2.0.0",
56
56
  "sinon": "^9.2.0"
57
57
  },
58
- "gitHead": "d8db2046661c42fb5aac09ed683b500bf4613b26"
58
+ "gitHead": "005c2d85db8f50cd3b239dd89ac467add6a71049"
59
59
  }
@@ -340,11 +340,6 @@ class Grid extends ElementMixin(
340
340
  value: isTouch
341
341
  },
342
342
 
343
- /** @protected */
344
- tabindex: {
345
- value: undefined
346
- },
347
-
348
343
  /**
349
344
  * If true, the grid's height is defined by its rows.
350
345
  *
@@ -473,23 +468,6 @@ class Grid extends ElementMixin(
473
468
  }
474
469
  }
475
470
 
476
- /**
477
- * Override an observer from `DisabledMixin` to not
478
- * set `tabindex` on the grid when it is re-enabled.
479
- *
480
- * @param {boolean} disabled
481
- * @param {boolean} oldDisabled
482
- * @protected
483
- * @override
484
- */
485
- _disabledChanged(disabled, oldDisabled) {
486
- super._disabledChanged(disabled, oldDisabled);
487
-
488
- if (oldDisabled) {
489
- this.removeAttribute('tabindex');
490
- }
491
- }
492
-
493
471
  /** @private */
494
472
  __getBodyCellCoordinates(cell) {
495
473
  if (this.$.items.contains(cell) && cell.localName === 'td') {
@@ -691,7 +669,7 @@ class Grid extends ElementMixin(
691
669
  // If focus is on element within the cell content — respect it, do not change
692
670
  const contentContainsFocusedElement = cellContent.contains(this.getRootNode().activeElement);
693
671
  // Only focus if mouse is released on cell content itself
694
- const mouseUpWithinCell = cellContent.contains(event.target);
672
+ const mouseUpWithinCell = event.composedPath().includes(cellContent);
695
673
  if (!contentContainsFocusedElement && mouseUpWithinCell) {
696
674
  cell.focus();
697
675
  }
@@ -973,8 +951,6 @@ class Grid extends ElementMixin(
973
951
 
974
952
  requestAnimationFrame(() => {
975
953
  this.__scrollToPendingIndex();
976
- // This needs to be set programmatically in order to avoid an iOS 10 bug (disappearing grid)
977
- this.$.table.style.webkitOverflowScrolling = 'touch';
978
954
  });
979
955
  }
980
956
  }