@linzjs/step-ag-grid 27.3.3 → 28.0.0

package/dist/step-ag-grid.esm.js

@@ -3181,6 +3181,12 @@ const Grid = ({ 'data-testid': dataTestId, defaultPostSort = true, rowSelection
             void params.onRowDragEnd({ movedRow, targetRow, direction: yDiff > 0 ? 1 : -1 });
         }
     }, [params, clearHighlightRowClasses]);
+    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;
@@ -3285,7 +3291,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));
     },
 };