fix(Grouping): Draggable Grouping cols reorder missing `impactedColum…

…ns` (#1557) * fix(Grouping): Draggable Grouping cols reorder missing impactedColumns
ghiscoding · Jun 5, 2024 · 69125c2 · 69125c2
1 parent 9e2dd65
commit 69125c2
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/packages/common/src/extensions/__tests__/slickDraggableGrouping.spec.ts b/packages/common/src/extensions/__tests__/slickDraggableGrouping.spec.ts
@@ -377,7 +377,7 @@ describe('Draggable Grouping Plugin', () => {
 
       expect(setColumnsSpy).toHaveBeenCalledWith([mockColumns[2], mockColumns[2]]);
       expect(setColumnResizeSpy).toHaveBeenCalled();
-      expect(triggerSpy).toHaveBeenCalledWith(gridStub.onColumnsReordered, { grid: gridStub });
+      expect(triggerSpy).toHaveBeenCalledWith(gridStub.onColumnsReordered, { grid: gridStub, impactedColumns: expect.arrayContaining([mockColumns[2]]) });
     });
 
     it('should drag over dropzone and expect hover css class be added and removed when dragging outside of dropzone', () => {

diff --git a/packages/common/src/extensions/slickDraggableGrouping.ts b/packages/common/src/extensions/slickDraggableGrouping.ts
@@ -349,7 +349,7 @@ export class SlickDraggableGrouping {
           finalReorderedColumns.push(reorderedColumns[getColumnIndex.call(grid, reorderedId)]);
         }
         setColumns.call(grid, finalReorderedColumns);
-        trigger.call(grid, grid.onColumnsReordered, { grid });
+        trigger.call(grid, grid.onColumnsReordered, { grid, impactedColumns: finalReorderedColumns });
         e.stopPropagation();
         setupColumnResize.call(grid);
       }