Skip to content

Commit

Permalink
PR comment #4
Browse files Browse the repository at this point in the history
  • Loading branch information
roastedcpu committed Feb 7, 2022
1 parent d0b4467 commit 3f9f345
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions packages/grid-pro/src/vaadin-grid-pro-inline-editing-mixin.js
Original file line number Diff line number Diff line change
@@ -459,7 +459,7 @@ export const InlineEditingMixin = (superClass) =>
* Triggers the editor for a given (row, col)
* row should be a number (element index)
* column can be either a number (element index) or a string (columnId)
* @param {number} row
* @param {number | string} row
* @param {number | string} col
* @public
*/
@@ -497,7 +497,7 @@ export const InlineEditingMixin = (superClass) =>
}

// Get rows (excluding header)
const tRows = this.$.table.getElementsByTagName('tbody').items.rows;
const tRows = this._getVisibleRows();

// Make sure row[rowIdx] exists
if (rowIdx > tRows.length || rowIdx < 0) {

0 comments on commit 3f9f345

Please # to comment.