chore(*): Additional small fixes related to navigation.

Author: MKirova

projects/igniteui-angular/src/lib/directives/for-of/for_of.directive.ts

@@ -721,16 +721,33 @@ export class IgxForOfDirective<T> implements OnInit, OnChanges, DoCheck, OnDestr
         return scroll;
     }
 
-    
     /**
      * Returns the index of the element at the specified offset.
      * ```typescript
      * this.parentVirtDir.getIndexAtScroll(100);
      * ```
      */
-     public getIndexAtScroll(scrollOffset: number) {
+    public getIndexAtScroll(scrollOffset: number) {
         return this.getIndexAt(scrollOffset, this.sizesCache);
     }
+    /**
+     * Returns whether the target index is outside the view.
+     * ```typescript
+     * this.parentVirtDir.isIndexOutsideView(10);
+     * ```
+     */
+    public isIndexOutsideView(index: number) {
+        const targetNode = index >= this.state.startIndex && index <= this.state.startIndex + this.state.chunkSize ?
+        this._embeddedViews.map(view =>
+            view.rootNodes.find(node => node.nodeType === Node.ELEMENT_NODE) || view.rootNodes[0].nextElementSibling)[index - this.state.startIndex] : null;
+        const rowHeight = this.getSizeAt(index);
+        const containerSize = parseInt(this.igxForContainerSize, 10);
+        const containerOffset = -(this.scrollPosition - this.sizesCache[this.state.startIndex]);
+        const endTopOffset = targetNode ? targetNode.offsetTop + rowHeight + containerOffset : containerSize + rowHeight;
+        return !targetNode || targetNode.offsetTop < Math.abs(containerOffset)
+            || containerSize && endTopOffset - containerSize > 5;
+    }
+
 
     /**
      * @hidden

projects/igniteui-angular/src/lib/grids/pivot-grid/pivot-grid-navigation.service.ts

@@ -2,6 +2,9 @@ import { IgxGridNavigationService } from '../grid-navigation.service';
 import { Injectable } from '@angular/core';
 import { IgxPivotGridComponent } from './pivot-grid.component';
 import { HEADER_KEYS } from '../../core/utils';
+import { IgxGridForOfDirective } from '../../directives/for-of/for_of.directive';
+import { first } from 'rxjs/operators';
+import { ComboSampleComponent } from 'src/app/combo/combo.sample';
 
 @Injectable()
 export class IgxPivotGridNavigationService extends IgxGridNavigationService {
@@ -43,7 +46,7 @@ export class IgxPivotGridNavigationService extends IgxGridNavigationService {
                 newActiveNode.row = ctrl ? 0 : this.activeNode.row - 1;
             }
             if ((key.includes('down')) && this.activeNode.row < this.findLastDataRowIndex()) {
-                newActiveNode.row = ctrl ? this.findLastDataRowIndex() : this.activeNode.row + 1;
+                newActiveNode.row = ctrl ? verticalContainer.igxForOf.length - 1 : Math.min(this.activeNode.row + 1, verticalContainer.igxForOf.length - 1);
             }
 
             if (key.includes('left') || key.includes('right')) {
@@ -59,7 +62,9 @@ export class IgxPivotGridNavigationService extends IgxGridNavigationService {
                 };
             }
             this.setActiveNode(newActiveNode);
-            verticalContainer.scrollTo(newActiveNode.row);
+            if (verticalContainer.isIndexOutsideView(newActiveNode.row)) {
+                verticalContainer.scrollTo(newActiveNode.row);
+            }
         } else {
             super.handleNavigation(event);
         }

projects/igniteui-angular/src/lib/grids/pivot-grid/pivot-grid.component.ts

@@ -1039,7 +1039,10 @@ export class IgxPivotGridComponent extends IgxGridBaseDirective implements OnIni
 
     protected verticalScrollHandler(event) {
         super.verticalScrollHandler(event);
-        this.verticalRowDimScrollContainers.forEach(x => x.onScroll(event));
+        this.verticalRowDimScrollContainers.forEach(x => { 
+            x.onScroll(event);
+            x.cdr.detectChanges();
+        });
     }
 
     /**