@linzjs/step-ag-grid 27.0.0 → 27.1.0

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

@@ -15,7 +15,6 @@ export interface ColDefT<TData extends GridBaseRow, ValueType = any> extends Col
     valueFormatter?: string | ValueFormatterFunc<TData, ValueType>;
     cellRenderer?: ((props: ICellRendererParams<TData, ValueType>) => ReactElement | string | false | null | undefined) | string;
     cellRendererParams?: {
-        singleClickEdit?: boolean;
         rightHoverElement?: ReactElement;
         originalCellRenderer?: any;
         editAction?: (selectedRows: TData[]) => void;

package/dist/src/components/gridRender/GridRenderGenericCell.d.ts

@@ -7,7 +7,6 @@ export interface GenericCellColDef<TData extends GridBaseRow, TValue = any> exte
     exportable?: boolean;
 }
 export interface GenericCellRendererParams<TData extends GridBaseRow> {
-    singleClickEdit?: boolean;
     rightHoverElement?: ReactElement | undefined;
     editAction?: (selectedRows: TData[]) => void;
     shortcutKeys?: Record<string, ((params: SuppressKeyboardEventParams) => boolean | void) | undefined>;

package/dist/src/contexts/GridContext.d.ts

@@ -17,7 +17,7 @@ export interface GridContextType<TData extends GridBaseRow> {
     getColDef: (colId?: string) => ColDef | undefined;
     getColumns: (filter?: keyof ColDef | ((r: ColDef) => boolean | undefined | null | number | string)) => ColDefT<TData, any>[];
     getColumnIds: (filter?: keyof ColDef | ((r: ColDef) => boolean | undefined | null | number | string)) => string[];
-    setApis: (gridApi: GridApi | undefined, dataTestId?: string) => void;
+    setApis: (gridApi: GridApi | undefined, hasExternallySelectedItems: boolean, dataTestId?: string) => void;
     prePopupOps: () => void;
     postPopupOps: () => void;
     setQuickFilter: (quickFilter: string) => void;

package/dist/step-ag-grid.cjs

@@ -2939,14 +2939,15 @@ const Grid = ({ 'data-testid': dataTestId, defaultPostSort = true, rowSelection
             };
         });
     }, [params.columnDefs, params.loading, params.readOnly, params.defaultColDef?.editable]);
+    const hasExternallySelectedItems = !!params.setExternalSelectedItems;
     /**
      * When grid is ready set the apis to the grid context and sync selected items to grid.
      */
     const onGridReady = React.useCallback((event) => {
-        setApis(event.api, dataTestId);
+        setApis(event.api, hasExternallySelectedItems, dataTestId);
         event.api.showNoRowsOverlay();
         synchroniseExternallySelectedItemsToGrid();
-    }, [dataTestId, setApis, synchroniseExternallySelectedItemsToGrid]);
+    }, [dataTestId, hasExternallySelectedItems, setApis, synchroniseExternallySelectedItemsToGrid]);
     /**
      * When the grid is being initialized the data may be empty.
      * This will resize columns when we have at least one row.
@@ -2999,7 +3000,7 @@ const Grid = ({ 'data-testid': dataTestId, defaultPostSort = true, rowSelection
      * Handle single click edits
      */
     const onCellClicked = React.useCallback((event) => {
-        if (event.colDef?.cellRendererParams?.singleClickEdit ?? singleClickEdit) {
+        if (event.colDef.singleClickEdit ?? singleClickEdit) {
             void startCellEditing({ rowId: event.data.id, colId: event.column.getColId() });
         }
     }, [singleClickEdit, startCellEditing]);
@@ -4988,11 +4989,9 @@ const GridPopoverMenu = (colDef, custom) => GridCell({
     exportable: false,
     cellStyle: { flex: 1, justifyContent: 'center' },
     cellRenderer: GridRenderPopoutMenuCell,
+    // Menus open on single click, this parameter is picked up in Grid.tsx
+    singleClickEdit: true,
     ...colDef,
-    cellRendererParams: {
-        // Menus open on single click, this parameter is picked up in Grid.tsx
-        singleClickEdit: true,
-    },
 }, {
     editor: GridFormPopoverMenu,
     preventAutoEdit: true,
@@ -5002,10 +5001,7 @@ const GridPopoverMenu = (colDef, custom) => GridCell({
 const GridPopoverMessage = (colDef, props) => GridCell({
     resizable: true,
     ...colDef,
-    cellRendererParams: {
-        singleClickEdit: true,
-        ...colDef.cellRendererParams,
-    },
+    singleClickEdit: true,
 }, {
     editor: GridFormMessage,
     ...props,
@@ -5035,6 +5031,7 @@ const GridContextProvider = (props) => {
     const [quickFilter, _setQuickFilter] = React.useState('');
     const [invisibleColumnIds, _setInvisibleColumnIds] = React.useState();
     const testId = React.useRef();
+    const hasExternallySelectedItemsRef = React.useRef(false);
     const idsBeforeUpdate = React.useRef([]);
     const prePopupFocusedCell = React.useRef();
     const [externallySelectedItemsAreInSync, setExternallySelectedItemsAreInSync] = React.useState(false);
@@ -5098,7 +5095,8 @@ const GridContextProvider = (props) => {
     /**
      * Set the grid api when the grid is ready.
      */
-    const setApis = React.useCallback((gridApi, dataTestId) => {
+    const setApis = React.useCallback((gridApi, hasExternallySelectedItems, dataTestId) => {
+        hasExternallySelectedItemsRef.current = hasExternallySelectedItems;
         testId.current = dataTestId;
         setGridApi(gridApi);
         gridApi?.setGridOption('quickFilterText', quickFilter);
@@ -5352,10 +5350,17 @@ const GridContextProvider = (props) => {
         externallySelectedItemsAreInSyncRef.current = externallySelectedItemsAreInSync;
     }, [externallySelectedItemsAreInSync]);
     const waitForExternallySelectedItemsToBeInSync = React.useCallback(async () => {
+        if (!hasExternallySelectedItemsRef.current) {
+            externallySelectedItemsAreInSyncRef.current = true;
+            return;
+        }
         // Wait for up to 5 seconds
         for (let i = 0; i < 5000 / 200 && !externallySelectedItemsAreInSyncRef.current; i++) {
             await wait(200);
         }
+        if (!externallySelectedItemsAreInSyncRef.current) {
+            console.error('externallySelectedItems did not sync with ag-grid selection');
+        }
     }, []);
     const startCellEditing = React.useCallback(async ({ rowId, colId }) => {
         if (!gridApi)