Skip to content

Commit 8064f06

Browse files
authored
Merge pull request #8242 from IgniteUI/iganchev/celledit-contextmenu-8187-9.1.x
Do not exit editing on RMBClick & allow copy/paste using context menu - 9.1.x
2 parents e25ed79 + 4ac9429 commit 8064f06

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed

projects/igniteui-angular/src/lib/grids/cell.component.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -765,7 +765,10 @@ export class IgxGridCellComponent implements OnInit, OnChanges, OnDestroy {
765765
this.selectionService.addKeyboardRange();
766766
this.selectionService.initKeyboardState();
767767
this.selectionService.primaryButton = false;
768-
this.gridAPI.submit_value();
768+
// Ensure RMB Click on edited cell does not end cell editing
769+
if (!this.selected) {
770+
this.gridAPI.submit_value();
771+
}
769772
return;
770773
}
771774
this.selectionService.pointerDown(this.selectionNode, event.shiftKey, event.ctrlKey);

projects/igniteui-angular/src/lib/grids/grid/grid-cell-selection.spec.ts

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2584,6 +2584,12 @@ describe('IgxGrid - Cell selection #grid', () => {
25842584
cell.editValue = 'No name';
25852585
fix.detectChanges();
25862586

2587+
UIInteractions.simulateNonPrimaryClick(cell);
2588+
fix.detectChanges();
2589+
expect(cell.editMode).toEqual(true);
2590+
expect(cell.editValue).toEqual('No name');
2591+
expect(cell.value).not.toEqual('No name');
2592+
25872593
const target = grid.getCellByColumn(0, 'Age');
25882594
UIInteractions.simulateNonPrimaryClick(target);
25892595
fix.detectChanges();

0 commit comments

Comments
 (0)