Merge pull request #8319 from IgniteUI/dkamburov/fix-8307

fix(action-strip): Do not show grid actions if the row is in edit mode

Author: ChronosSF

projects/igniteui-angular/src/lib/action-strip/grid-actions/grid-actions-base.directive.ts

@@ -56,7 +56,7 @@ export class IgxGridActionsBaseDirective implements AfterViewInit {
      * @internal
      */
     get isRowContext(): boolean {
-        return this.isRow(this.strip.context);
+        return this.isRow(this.strip.context) && !this.strip.context.inEditMode;
     }
 
     /**

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

@@ -60,6 +60,12 @@ describe('igxGridEditingActions #grid ', () => {
             actionStrip.show(grid.rowList.first);
             fixture.detectChanges();
             deleteIcon = fixture.debugElement.queryAll(By.css(`igx-grid-editing-actions igx-icon`))[1];
+            // grid actions should not showing when the row is in edit mode #
+            expect(deleteIcon).toBeUndefined();
+            grid.endEdit();
+            actionStrip.show(grid.rowList.first);
+            fixture.detectChanges();
+            deleteIcon = fixture.debugElement.queryAll(By.css(`igx-grid-editing-actions igx-icon`))[1];
             expect(deleteIcon.nativeElement.innerText).toBe('delete');
             deleteIcon.parent.triggerEventHandler('click', new Event('click'));
             actionStrip.hide();
@@ -85,13 +91,19 @@ describe('igxGridEditingActions #grid ', () => {
             fixture.detectChanges();
             expect(actionStrip.menu.items.length).toBe(2);
             const editMenuItem = actionStrip.menu.items[0];
-            const deleteMenuItem = actionStrip.menu.items[1];
+
             // select edit
             actionStrip.menu.selectItem(editMenuItem);
             fixture.detectChanges();
 
             expect(row.inEditMode).toBeTrue();
 
+            grid.endEdit();
+            fixture.detectChanges();
+            actionStrip.menu.open();
+            fixture.detectChanges();
+            const deleteMenuItem = actionStrip.menu.items[1];
+
             // select delete
             actionStrip.menu.selectItem(deleteMenuItem);
             fixture.detectChanges();