@linzjs/step-ag-grid 29.1.0 → 29.1.2

package/dist/src/components/GridPopoverHook.d.ts

@@ -6,7 +6,7 @@ export interface GridPopoverHookProps<TData> {
     save?: (selectedRows: TData[]) => Promise<boolean>;
     dontSaveOnExternalClick?: boolean;
 }
-export declare const useGridPopoverHook: <TData extends GridBaseRow>(props: GridPopoverHookProps<TData>) => {
+export declare const useGridPopoverHook: <TData extends GridBaseRow>({ className, save, invalid, dontSaveOnExternalClick, }: GridPopoverHookProps<TData>) => {
     popoverWrapper: (children: ReactElement) => import("react/jsx-runtime").JSX.Element;
     triggerSave: (reason?: string) => Promise<void>;
 };

package/dist/src/index.d.ts

@@ -17,3 +17,4 @@ export * from './utils/deferredPromise';
 export * from './utils/textMatcher';
 export * from './utils/textValidator';
 export * from './utils/util';
+export * from './utils/waitForCondition';

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

@@ -15,7 +15,7 @@ export const setUpUserEvent = (customisedUserEvent: any) => {
 };
 
 export const countRows = (within?: HTMLElement): number => {
-  return getAllQuick({ tagName: `div[row-id]:not(:empty)` }, within).length;
+  return getAllQuick({ tagName: `.ag-center-cols-container div[row-id]:not(:empty)` }, within).length;
 };
 
 export const findRowByIndex = async (rowIndex: number | string, within?: HTMLElement): Promise<HTMLDivElement> => {

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

@@ -16,7 +16,7 @@ export const setUpUserEvent = (customisedUserEvent: any) => {
 };
 
 export const countRows = (within?: HTMLElement): number => {
-  return getAllQuick({ tagName: `div[row-id]:not(:empty)` }, within).length;
+  return getAllQuick({ tagName: `.ag-center-cols-container div[row-id]:not(:empty)` }, within).length;
 };
 
 export const findRowByIndex = async (rowIndex: number | string, within?: HTMLElement): Promise<HTMLDivElement> => {

package/dist/src/utils/waitForCondition.d.ts

@@ -0,0 +1 @@
+export declare const waitForCondition: (condition: () => boolean, timeoutMs: number) => Promise<boolean>;

package/dist/step-ag-grid.cjs

@@ -3938,8 +3938,8 @@ const textMatch = (text, filter) => {
             negativeFilters.every((superFilter) => values.every((value) => isMatch(value, superFilter)))));
 };
 
-const useGridPopoverHook = (props) => {
-    const { onBulkEditingComplete } = useGridContext();
+const useGridPopoverHook = ({ className, save, invalid, dontSaveOnExternalClick, }) => {
+    const { onBulkEditingComplete, redrawRows } = useGridContext();
     const { anchorRef, saving, updateValue, stopEditing } = useGridPopoverContext();
     const saveButtonRef = React.useRef(null);
     const [isOpen, setOpen] = React.useState(false);
@@ -3950,22 +3950,25 @@ const useGridPopoverHook = (props) => {
         if (reason == CloseReason.CANCEL) {
             stopEditing();
             onBulkEditingComplete();
+            redrawRows();
             return;
         }
-        if (props?.invalid?.()) {
+        if (invalid?.()) {
             // Don't close, don't do anything it's invalid
             return;
         }
-        if (!props.save) {
+        if (!save) {
             // No save method so just close
             stopEditing();
             onBulkEditingComplete();
+            redrawRows();
             return;
         }
-        if (await updateValue(props.save, reason === CloseReason.TAB_FORWARD ? 1 : reason === CloseReason.TAB_BACKWARD ? -1 : 0)) {
+        if (await updateValue(save, reason === CloseReason.TAB_FORWARD ? 1 : reason === CloseReason.TAB_BACKWARD ? -1 : 0)) {
             stopEditing();
+            redrawRows();
         }
-    }, [onBulkEditingComplete, props, stopEditing, updateValue]);
+    }, [invalid, onBulkEditingComplete, redrawRows, save, stopEditing, updateValue]);
     const popoverWrapper = React.useCallback((children) => {
         return (jsxRuntime.jsx(jsxRuntime.Fragment, { children: anchorRef.current && (jsxRuntime.jsxs(ControlledMenu, { state: isOpen ? 'open' : 'closed', portal: true, unmountOnClose: true, anchorRef: anchorRef, saveButtonRef: saveButtonRef, menuClassName: 'step-ag-grid-react-menu', onClose: (event) => {
                     // Prevent menu from closing when modals are invoked
@@ -3973,15 +3976,15 @@ const useGridPopoverHook = (props) => {
                         return;
                     }
                     void triggerSave(event.reason);
-                }, viewScroll: 'auto', dontShrinkIfDirectionIsTop: true, className: props.className, children: [saving && ( // This is the overlay that prevents editing when the editor is saving
+                }, viewScroll: 'auto', dontShrinkIfDirectionIsTop: true, className: className, children: [saving && ( // This is the overlay that prevents editing when the editor is saving
                     jsxRuntime.jsx("div", { className: 'ComponentLoadingWrapper-saveOverlay' })), children, jsxRuntime.jsx("button", { ref: saveButtonRef, "data-reason": '', onClick: (e) => {
                             let reason = e.currentTarget.getAttribute('data-reason') ?? undefined;
-                            if (props.dontSaveOnExternalClick && reason === CloseReason.BLUR) {
+                            if (dontSaveOnExternalClick && reason === CloseReason.BLUR) {
                                 reason = CloseReason.CANCEL;
                             }
                             void triggerSave(reason);
                         }, style: { display: 'none' } })] })) }));
-    }, [anchorRef, isOpen, props.className, props.dontSaveOnExternalClick, saving, triggerSave]);
+    }, [anchorRef, isOpen, className, dontSaveOnExternalClick, saving, triggerSave]);
     return {
         popoverWrapper,
         triggerSave,
@@ -5061,6 +5064,18 @@ const GridWrapper = ({ children, maxHeight }) => (jsxRuntime.jsx("div", { classN
 const getColId = (colDef) => colDef.colId ?? '';
 const isFlexColumn = (colDef) => !!colDef.flex && !isGridCellFiller(colDef);
 
+const waitForCondition = async (condition, timeoutMs) => {
+    const endTime = Date.now() + timeoutMs;
+    while (Date.now() < endTime) {
+        if (condition()) {
+            return true;
+        }
+        await wait(100);
+    }
+    console.warn('waitForCondition failed');
+    return false;
+};
+
 /**
  * Context for AgGrid operations.
  * Make sure you wrap AgGrid in this.
@@ -5733,17 +5748,6 @@ const downloadCsvUseValueFormattersProcessCellCallback = (params) => {
     // We add an extra encodeToString here just in case valueFormatter is returning non string values
     return encodeToString(result);
 };
-const waitForCondition = async (condition, timeoutMs) => {
-    const endTime = Date.now() + timeoutMs;
-    while (Date.now() < endTime) {
-        if (condition()) {
-            return true;
-        }
-        await wait(100);
-    }
-    console.warn('waitForCondition failed');
-    return false;
-};
 
 const GridUpdatingContextProvider = (props) => {
     const updatingBlocks = React.useRef({});
@@ -5973,4 +5977,5 @@ exports.useGridPopoverHook = useGridPopoverHook;
 exports.useMenuState = useMenuState;
 exports.usePostSortRowsHook = usePostSortRowsHook;
 exports.wait = wait;
+exports.waitForCondition = waitForCondition;
 //# sourceMappingURL=step-ag-grid.cjs.map