Merge pull request #7391 from IgniteUI/nrobakova/fix-issue-7388

zdrawku · web-flow · commit 7f85ddb5d65c · 2020-05-26T15:07:24.000+03:00
Update selection when remote data is used
diff --git a/projects/igniteui-angular/src/lib/grids/grid-navigation.service.ts b/projects/igniteui-angular/src/lib/grids/grid-navigation.service.ts
@@ -171,7 +171,8 @@ export class IgxGridNavigationService {
     }
 
     focusTbody(event) {
-        if (!this.activeNode || this.activeNode.row < 0 || this.activeNode.row > this.grid.dataView.length - 1) {
+        const gridRows = this.grid.verticalScrollContainer.totalItemCount ?? this.grid.dataView.length;
+        if (!this.activeNode || this.activeNode.row < 0 || this.activeNode.row > gridRows - 1) {
             this.activeNode = { row: 0, column: 0 };
             this.grid.navigateTo(0, 0, (obj) => {
                 this.grid.clearCellSelection();

Original file line number	Diff line number	Diff line change
`@@ -171,7 +171,8 @@ export class IgxGridNavigationService {`
`171`	`171`	`}`
`172`	`172`
`173`	`173`	`focusTbody(event) {`
`174`		`- if (!this.activeNode \|\| this.activeNode.row < 0 \|\| this.activeNode.row > this.grid.dataView.length - 1) {`
	`174`	`+ const gridRows = this.grid.verticalScrollContainer.totalItemCount ?? this.grid.dataView.length;`
	`175`	`+ if (!this.activeNode \|\| this.activeNode.row < 0 \|\| this.activeNode.row > gridRows - 1) {`
`175`	`176`	`this.activeNode = { row: 0, column: 0 };`
`176`	`177`	`this.grid.navigateTo(0, 0, (obj) => {`
`177`	`178`	`this.grid.clearCellSelection();`