@linzjs/step-ag-grid 27.3.3 → 28.0.0

package/dist/src/utils/__tests__/storybookTestUtil.ts

@@ -1,4 +1,4 @@
-import { expect, waitFor } from '@storybook/test';
+import { expect, waitFor } from 'storybook/test';
 
 export const waitForGridReady = ({ canvasElement }: { canvasElement: HTMLElement }) =>
   waitFor(() => expect(canvasElement.querySelector('.Grid-ready')).toBeInTheDocument(), { timeout: 5000 });

package/dist/step-ag-grid.cjs

@@ -3183,6 +3183,12 @@ const Grid = ({ 'data-testid': dataTestId, defaultPostSort = true, rowSelection
             void params.onRowDragEnd({ movedRow, targetRow, direction: yDiff > 0 ? 1 : -1 });
         }
     }, [params, clearHighlightRowClasses]);
+    React.useEffect(() => {
+        if (params.setExternalSelectedItems && selectable == null) {
+            console.warn('<Grid/> has setExternalSelectedItems parameter, but is missing selectable parameter,' +
+                'this will cause weird delays in editing.\nIf you need to hide selection column use hideSelectColumn=true');
+        }
+    }, [params.setExternalSelectedItems, selectable]);
     // This is setting a ref in the GridContext so won't be triggering an update loop
     setOnCellEditingComplete(params.onCellEditingComplete);
     const selectWidth = params.hideSelectColumn ? 0 : selectable && params.onRowDragEnd ? 76 : 48;
@@ -3287,7 +3293,7 @@ const GridCellMultiSelectClassRules = {
                 .getSelectedNodes()
                 ?.map((row) => row.id)
                 ?.includes(node.id) &&
-            api.getEditingCells().some((cell) => cell.column.getColDef() === colDef));
+            api.getEditingCells().some((cell) => cell.colId === colDef.colId));
     },
 };