Merge pull request #8415 from IgniteUI/mkirova/fix-8407

fix(igxGrid): Fix issue with actionStrip.show being called too many t…

Author: ChronosSF

projects/igniteui-angular/src/lib/action-strip/grid-actions/grid-editing-actions.component.spec.ts

@@ -139,10 +139,10 @@ describe('igxGridEditingActions #grid ', () => {
             actionStrip = fixture.componentInstance.actionStrip;
             grid = fixture.componentInstance.grid;
         }));
-        it('should auto-show on mouse over of row.', () => {
+        it('should auto-show on mouse enter of row.', () => {
             const row = grid.getRowByIndex(0);
             const rowElem = row.nativeElement;
-            UIInteractions.simulateMouseEvent('mouseover', rowElem, 0, 0);
+            UIInteractions.simulateMouseEvent('mouseenter', rowElem, 0, 0);
             fixture.detectChanges();
 
             expect(actionStrip.context).toBe(row);
@@ -171,18 +171,18 @@ describe('igxGridEditingActions #grid ', () => {
             hierarchicalGrid = fixture.componentInstance.hgrid;
         }));
 
-        it('should auto-show root actionStrip on mouse over of root row.', () => {
+        it('should auto-show root actionStrip on mouse enter of root row.', () => {
             const row = hierarchicalGrid.getRowByIndex(0);
             const rowElem = row.nativeElement;
-            UIInteractions.simulateMouseEvent('mouseover', rowElem, 0, 0);
+            UIInteractions.simulateMouseEvent('mouseenter', rowElem, 0, 0);
             fixture.detectChanges();
 
             expect(actionStripRoot.context).toBe(row);
             expect(actionStripRoot.hidden).toBeFalse();
             expect(actionStripChild.context).toBeUndefined();
         });
 
-        it('should auto-show row island actionStrip on mouse over of child row.', () => {
+        it('should auto-show row island actionStrip on mouse enter of child row.', () => {
             const row = hierarchicalGrid.getRowByIndex(0) as IgxHierarchicalRowComponent;
             row.toggle();
             fixture.detectChanges();
@@ -191,7 +191,7 @@ describe('igxGridEditingActions #grid ', () => {
 
             const childRow = childGrid.getRowByIndex(0);
             const rowElem = childRow.nativeElement;
-            UIInteractions.simulateMouseEvent('mouseover', rowElem, 0, 0);
+            UIInteractions.simulateMouseEvent('mouseenter', rowElem, 0, 0);
             fixture.detectChanges();
 
             expect(actionStripChild.context).toBe(childRow);

projects/igniteui-angular/src/lib/grids/row.directive.ts

@@ -396,7 +396,7 @@ export class IgxRowDirective<T extends IgxGridBaseDirective & GridType> implemen
      * @hidden
      * @internal
      */
-    @HostListener('mouseover', ['$event'])
+    @HostListener('mouseenter', ['$event'])
     public showActionStrip(event: MouseEvent) {
         if (this.grid.actionStrip) {
             this.grid.actionStrip.show(this);