@linzjs/step-ag-grid 21.0.3 → 21.1.0

package/package.json

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

package/src/components/Grid.tsx

@@ -187,12 +187,12 @@ export const Grid = ({
         return;
       }
       if (gridRendered === "empty") {
-        if (!hasSetContentSizeEmpty.current && result && !hasSetContentSize.current) {
+        if (!hasSetContentSizeEmpty.current && !hasSetContentSize.current) {
           hasSetContentSizeEmpty.current = true;
           params.onContentSize?.(result);
         }
       } else if (gridRendered === "rows-visible") {
-        if (result && !hasSetContentSize.current) {
+        if (!hasSetContentSize.current) {
           hasSetContentSize.current = true;
           params.onContentSize?.(result);
         }
@@ -425,7 +425,9 @@ export const Grid = ({
     if (!isEmpty(updatingCols())) return;
 
     const skipHeader = sizeColumns === "auto-skip-headers";
-    autoSizeColumns({ skipHeader, userSizedColIds: userSizedColIds.current, colIds: colIdsEdited.current });
+    if (hasSetContentSize.current) {
+      autoSizeColumns({ skipHeader, userSizedColIds: userSizedColIds.current, colIds: colIdsEdited.current });
+    }
     colIdsEdited.current.clear();
   }, [autoSizeColumns, rowData?.length, setInitialContentSize, sizeColumns, updatedDep, updatingCols]);
 
@@ -563,13 +565,15 @@ export const Grid = ({
       if (!isEmpty(colIdsEdited.current)) {
         const skipHeader = sizeColumns === "auto-skip-headers";
         if (sizeColumns === "auto" || skipHeader) {
-          defer(() =>
-            autoSizeColumns({
-              skipHeader,
-              userSizedColIds: userSizedColIds.current,
-              colIds: colIdsEdited.current,
-            }),
-          );
+          defer(() => {
+            if (hasSetContentSize.current) {
+              autoSizeColumns({
+                skipHeader,
+                userSizedColIds: userSizedColIds.current,
+                colIds: colIdsEdited.current,
+              });
+            }
+          });
         }
         colIdsEdited.current.clear();
       }

package/src/contexts/GridPopoverContext.tsx

@@ -6,6 +6,7 @@ export interface GridPopoverContextType<TData extends GridBaseRow> {
   anchorRef: RefObject<Element>;
   saving: boolean;
   setSaving: (saving: boolean) => void;
+  colId: string;
   field: keyof TData;
   value: any;
   data: TData;
@@ -18,6 +19,7 @@ export const GridPopoverContext = createContext<GridPopoverContextType<any>>({
   anchorRef: { current: null },
   saving: false,
   setSaving: () => {},
+  colId: "",
   field: "",
   value: null,
   data: {} as GridBaseRow,

package/src/contexts/GridPopoverContextProvider.tsx

@@ -26,7 +26,9 @@ export const GridPopoverContextProvider = ({ props, children }: PropsWithChildre
       multiEdit ? sortBy(getFilteredSelectedRows(), (row) => row.id !== props.data.id) : [props.data as GridBaseRow],
     [getFilteredSelectedRows, multiEdit, props.data],
   );
+
   const field = props.colDef?.field ?? "";
+  const colId = props.colDef?.colId ?? field ?? "";
 
   const updateValue = useCallback(
     async (saveFn: (selectedRows: any[]) => Promise<boolean>, tabDirection: 1 | 0 | -1): Promise<boolean> => {
@@ -44,6 +46,7 @@ export const GridPopoverContextProvider = ({ props, children }: PropsWithChildre
         saving,
         setSaving,
         selectedRows,
+        colId,
         field,
         data: props.data,
         value: props.value,