@linzjs/step-ag-grid 7.6.1 → 7.6.2

package/package.json

@@ -2,7 +2,7 @@
   "name": "@linzjs/step-ag-grid",
   "repository": "github:linz/step-ag-grid.git",
   "license": "MIT",
-  "version": "7.6.1",
+  "version": "7.6.2",
   "keywords": [
     "aggrid",
     "ag-grid",

package/src/components/GridCell.tsx

@@ -68,7 +68,7 @@ export const GridCell = <RowType extends GridBaseRow, Props extends CellEditorCo
     sortable: !!(props?.field || props?.valueGetter),
     resizable: true,
     ...(custom?.editor && {
-      cellClassRules: GridCellMultiSelectClassRules,
+      cellClassRules: custom.multiEdit ? GridCellMultiSelectClassRules : undefined,
       editable: props.editable ?? true,
       cellEditor: GenericCellEditorComponentWrapper(custom),
     }),

package/src/contexts/GridContextProvider.tsx

@@ -251,7 +251,10 @@ export const GridContextProvider = (props: GridContextProps): ReactElement => {
         async () => {
           // Need to refresh to get spinners to work on all rows
           gridApi.refreshCells({ rowNodes: props.selectedRows as RowNode[], force: true });
-          ok = await fnUpdate(selectedRows);
+          ok = await fnUpdate(selectedRows).catch((ex) => {
+            console.error("Exception during modifyUpdating", ex);
+            return false;
+          });
         },
       );
 

package/src/contexts/GridPopoverContextProvider.tsx

@@ -31,9 +31,7 @@ export const GridPopoverContextProvider = ({ props, children }: GridPopoverConte
     async (saveFn: (selectedRows: any[]) => Promise<boolean>, tabDirection: 1 | 0 | -1): Promise<boolean> => {
       if (hasSaved.current) return true;
       hasSaved.current = true;
-      const r = saving ? false : await updatingCells({ selectedRows, field }, saveFn, setSaving, tabDirection);
-      //if (r) stopEditing();
-      return r;
+      return saving ? false : await updatingCells({ selectedRows, field }, saveFn, setSaving, tabDirection);
     },
     [field, saving, selectedRows, updatingCells],
   );

package/src/styles/GridTheme.scss

@@ -75,8 +75,6 @@
   }
 
   .ag-cell.ag-cell-inline-editing {
-    padding-left: 9px;
-    padding-right: 9px;
     bottom: 0;
   }
 
@@ -84,9 +82,11 @@
     word-break: break-word;
   }
 
-  .ag-cell-popup-editing,
-  .ag-selected-for-edit,
+  .ag-cell.ag-cell-popup-editing,
+  .ag-cell.ag-selected-for-edit,
   .ag-cell-inline-editing {
+    padding-left: 9px;
+    padding-right: 9px;
     background: rgb(72 160 244 / 20%);
     // These are important, it needs to override ag-grid to work
     border: 3px solid #48a0f4 !important;

package/src/utils/deferredPromise.ts

@@ -1,9 +1,16 @@
-import { useRef } from "react";
+import { useEffect, useRef } from "react";
 
 export const useDeferredPromise = <T>() => {
   const promiseResolve = useRef<((value: T | PromiseLike<T>) => void) | undefined>();
   const promiseReject = useRef<() => void>();
 
+  // End promise on unload
+  useEffect(() => {
+    return () => {
+      promiseReject.current && promiseReject.current();
+    };
+  }, []);
+
   return {
     invoke: () =>
       new Promise<T>((resolve, reject) => {