@linzjs/step-ag-grid 32.0.0 → 32.1.1

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

@@ -15,6 +15,8 @@ export interface GridOnRowDragEndProps<TData extends GridBaseRow> {
 }
 export interface ColDefT<TData extends GridBaseRow, ValueType = any> extends ColDef<TData, ValueType> {
     editable?: boolean | EditableCallback<TData, ValueType>;
+    /** When true, this column's editable state ignores the grid-level `readOnly` prop. */
+    ignoreGridReadOnly?: boolean;
     valueGetter?: string | ValueGetterFunc<TData, ValueType>;
     valueFormatter?: string | ValueFormatterFunc<TData, ValueType>;
     cellRenderer?: ((props: ICellRendererParams<TData, ValueType>) => ReactElement | string | false | null | undefined) | string;

package/dist/step-ag-grid.cjs

@@ -61425,9 +61425,11 @@ maxInitialWidth,
         }
         else {
             const colDefEditable = colDef.editable;
-            const editable = combineEditables(params.loading !== true && params.readOnly !== true, params.defaultColDef?.editable, colDefEditable);
+            const ignoreGridReadOnly = colDef.ignoreGridReadOnly === true;
+            const editable = combineEditables(params.loading !== true, ignoreGridReadOnly ? true : params.readOnly !== true, params.defaultColDef?.editable, colDefEditable);
             return {
-                ...colDef,
+                // ag-grid warns on unrecognised colDef properties, so it can't be left on the returned colDef
+                ...lodashEs.omit(colDef, 'ignoreGridReadOnly'),
                 editable,
                 cellClassRules: {
                     ...colDef.cellClassRules,
@@ -63801,7 +63803,11 @@ const GridFormPopoverMenu = (props) => {
         invalid: () => subComponentSelected && !subComponentIsValid.current,
         save,
     });
-    return popoverWrapper(jsxRuntime.jsx(ComponentLoadingWrapper, { loading: !options, className: 'GridFormPopupMenu', children: jsxRuntime.jsx(jsxRuntime.Fragment, { children: lodashEs.isEmpty(options) ? (jsxRuntime.jsx(MenuItem, { className: 'GridPopoverMenu-noOptions', disabled: true, children: "No actions" }, `GridPopoverMenu-empty`)) : (options?.map((item, index) => item.label === '__isMenuSeparator__' ? (jsxRuntime.jsx(MenuDivider, {}, `$$divider_${index}`)) : (!item.hidden && (jsxRuntime.jsxs(React13.Fragment, { children: [item.subMenu ? (jsxRuntime.jsx(SubMenu, { disabled: !!item.disabled, label: item.label, title: item.disabled && typeof item.disabled !== 'boolean' ? item.disabled : '', children: jsxRuntime.jsx(item.subMenu, {}) })) : (jsxRuntime.jsx(MenuItem, { onClick: (e) => void onMenuItemClick(e, item), disabled: !!item.disabled, title: item.disabled && typeof item.disabled !== 'boolean' ? item.disabled : '', children: item.label })), item.subComponent && subComponentSelected === item && (jsxRuntime.jsx(FocusableItem, { className: 'LuiDeprecatedForms', children: () => (jsxRuntime.jsx(GridSubComponentContext.Provider, { value: {
+    // isEmpty(options) alone isn't enough: options can be a non-empty array where every item is
+    // hidden (e.g. all actions unavailable in a given mode), which must also show "No actions"
+    // rather than rendering an empty menu body.
+    const hasVisibleOption = options?.some((item) => !item.hidden && item.label !== __isMenuSeparator__);
+    return popoverWrapper(jsxRuntime.jsx(ComponentLoadingWrapper, { loading: !options, className: 'GridFormPopupMenu', children: jsxRuntime.jsx(jsxRuntime.Fragment, { children: !hasVisibleOption ? (jsxRuntime.jsx(MenuItem, { className: 'GridPopoverMenu-noOptions', disabled: true, children: "No actions" }, `GridPopoverMenu-empty`)) : (options?.map((item, index) => item.label === '__isMenuSeparator__' ? (jsxRuntime.jsx(MenuDivider, {}, `$$divider_${index}`)) : (!item.hidden && (jsxRuntime.jsxs(React13.Fragment, { children: [item.subMenu ? (jsxRuntime.jsx(SubMenu, { disabled: !!item.disabled, label: item.label, title: item.disabled && typeof item.disabled !== 'boolean' ? item.disabled : '', children: jsxRuntime.jsx(item.subMenu, {}) })) : (jsxRuntime.jsx(MenuItem, { onClick: (e) => void onMenuItemClick(e, item), disabled: !!item.disabled, title: item.disabled && typeof item.disabled !== 'boolean' ? item.disabled : '', children: item.label })), item.subComponent && subComponentSelected === item && (jsxRuntime.jsx(FocusableItem, { className: 'LuiDeprecatedForms', children: () => (jsxRuntime.jsx(GridSubComponentContext.Provider, { value: {
                                 context: {},
                                 data,
                                 value: subSelectedValue,
@@ -64052,8 +64058,13 @@ const BooleanCellRenderer = (props) => {
             return;
         }
         const cell = eGridCell?.querySelector('.ag-cell-focus .grid-edit-boolean input.ag-checkbox-input:not(:disabled)');
-        if (cell && cell.ownerDocument.activeElement !== cell) {
-            cell.focus();
+        if (cell) {
+            // When you redraw a cell the activeElement moves to the cell div away from input
+            // Refocus here if the cell div is active back to the input
+            const activeElement = cell.ownerDocument.activeElement;
+            if (activeElement && activeElement.getAttribute('role') === 'gridcell' && activeElement !== cell) {
+                cell.focus();
+            }
         }
     });
     return (jsxRuntime.jsx("div", { className: clsx('grid-edit-boolean ag-wrapper ag-input-wrapper ag-checkbox-input-wrapper', {
@@ -64170,6 +64181,9 @@ const GridPopoverMenu = (colDef, custom) => GridCell({
     editable: colDef.editable != null ? colDef.editable : true,
     exportable: false,
     cellStyle: { flex: 1, justifyContent: 'center' },
+    suppressMovable: true,
+    lockPosition: 'right',
+    resizable: false,
     cellRenderer: GridRenderPopoutMenuCell,
     // Menus open on single click, this parameter is picked up in Grid.tsx
     singleClickEdit: true,