@linzjs/step-ag-grid 27.2.1 → 27.2.3

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

@@ -2990,7 +2990,8 @@ const Grid = ({ 'data-testid': dataTestId, defaultPostSort = true, rowSelection
      * Handle double click edit
      */
     const onCellDoubleClick = useCallback((event) => {
-        if (!invokeEditAction(event)) {
+        const editable = fnOrVar(event.colDef?.editable, event);
+        if (editable && !invokeEditAction(event)) {
             void startCellEditing({ rowId: event.data.id, colId: event.column.getColId() });
         }
     }, [startCellEditing]);
@@ -2998,7 +2999,8 @@ const Grid = ({ 'data-testid': dataTestId, defaultPostSort = true, rowSelection
      * Handle single click edits
      */
     const onCellClicked = useCallback((event) => {
-        if (event.colDef.singleClickEdit ?? singleClickEdit) {
+        const editable = fnOrVar(event.colDef?.editable, event);
+        if ((editable && event.colDef.singleClickEdit) ?? singleClickEdit) {
             void startCellEditing({ rowId: event.data.id, colId: event.column.getColId() });
         }
     }, [singleClickEdit, startCellEditing]);
@@ -3404,6 +3406,7 @@ const GridCellFiller = () => ({
     colId: GridCellFillerColId,
     headerName: '',
     flex: 1,
+    editable: false,
 });
 
 const Editor = (props) => ({
@@ -5376,6 +5379,7 @@ const GridContextProvider = (props) => {
         const shouldSelectNode = !rowNode.isSelected();
         if (shouldSelectNode) {
             externallySelectedItemsAreInSyncRef.current = false;
+            setExternallySelectedItemsAreInSync(false);
             rowNode.setSelected(true, true);
             await waitForExternallySelectedItemsToBeInSync();
         }
@@ -5516,6 +5520,9 @@ const GridContextProvider = (props) => {
         }
     }, 50), [gridApi]);
     const onFilterChanged = useMemo(() => debounce(() => {
+        if (!gridApi || gridApi?.isDestroyed()) {
+            return;
+        }
         // This is terrible, but there's no other way for me to check whether a filter has changed the grid
         const getDisplayedRowsHash = () => {
             const arr = [];