@linzjs/step-ag-grid 8.3.0 → 8.3.2

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

@@ -4,6 +4,7 @@ export interface GridPopoverHookProps<RowType> {
     className: string | undefined;
     invalid?: () => Promise<JSX.Element | boolean | string | null | undefined> | JSX.Element | boolean | string | null | undefined;
     save?: (selectedRows: RowType[]) => Promise<boolean>;
+    dontSaveOnExternalClick?: boolean;
 }
 export declare const useGridPopoverHook: <RowType extends GridBaseRow>(props: GridPopoverHookProps<RowType>) => {
     popoverWrapper: (children: JSX.Element) => JSX.Element;

package/dist/step-ag-grid.esm.js

@@ -2864,7 +2864,7 @@ var Grid = function (params) {
         var adjustColDefs = params.columnDefs.map(function (colDef) {
             var _a;
             var colDefEditable = colDef.editable;
-            var editable = combineEditables(params.readOnly !== false, (_a = params.defaultColDef) === null || _a === void 0 ? void 0 : _a.editable, colDefEditable);
+            var editable = combineEditables(params.readOnly !== true, (_a = params.defaultColDef) === null || _a === void 0 ? void 0 : _a.editable, colDefEditable);
             return __assign(__assign({}, colDef), { editable: editable, cellClassRules: __assign(__assign({}, colDef.cellClassRules), { "GridCell-readonly": function (ccp) { return !editable(ccp); } }) });
         });
         return params.selectable
@@ -3143,9 +3143,13 @@ var useGridPopoverHook = function (props) {
                             zIndex: 1000
                         } })), children, jsx("button", { ref: saveButtonRef, "data-reason": "", onClick: function (e) {
                             var _a;
-                            triggerSave((_a = e.currentTarget.getAttribute("data-reason")) !== null && _a !== void 0 ? _a : undefined).then();
+                            var reason = (_a = e.currentTarget.getAttribute("data-reason")) !== null && _a !== void 0 ? _a : undefined;
+                            if (props.dontSaveOnExternalClick && reason === CloseReason.BLUR) {
+                                reason = CloseReason.CANCEL;
+                            }
+                            triggerSave(reason).then();
                         }, style: { display: "none" } })] }))) }));
-    }, [anchorRef, isOpen, props.className, saving, triggerSave]);
+    }, [anchorRef, isOpen, props.className, props.dontSaveOnExternalClick, saving, triggerSave]);
     return {
         popoverWrapper: popoverWrapper,
         triggerSave: triggerSave
@@ -3436,26 +3440,11 @@ var GridFormDropDown = function (props) {
             }
         });
     }); }, [filter, filteredValues, options, props, selectItemHandler, selectedItem, selectedRows, subSelectedValue]);
-    var onMouseEnterFocus = useCallback(function (item) {
-        setSelectedItem(item);
-        if (item.subComponent) {
-            subComponentIsValid.current = true;
-            subComponentInitialValue.current = null;
-        }
-        else {
-            setSubSelectedValue(null);
-            subComponentIsValid.current = true;
-        }
-    }, []);
-    var onMouseLeaveFocus = useCallback(function () {
-        setSelectedItem(null);
-        setSubSelectedValue(null);
-        subComponentIsValid.current = true;
-    }, []);
     var popoverWrapper = useGridPopoverHook({
         className: props.className,
         invalid: function () { return !!(selectedItem && !subComponentIsValid.current); },
-        save: save
+        save: save,
+        dontSaveOnExternalClick: true
     }).popoverWrapper;
     var lastHeader = null;
     var showHeader = null;
@@ -3484,7 +3473,17 @@ var GridFormDropDown = function (props) {
                                     lastHeader = null;
                                 }
                             }
-                            return ((!filteredValues || filteredValues.includes(item)) && (jsxs(Fragment$1, { children: [showHeader, jsxs("div", { children: [jsxs(MenuItem, __assign({ disabled: !!item.disabled, title: item.disabled && typeof item.disabled !== "boolean" ? item.disabled : "", value: item.value, onFocus: function () { return onMouseEnterFocus(item); }, onMouseEnter: function () { return onMouseEnterFocus(item); }, onMouseLeave: onMouseLeaveFocus, onClick: function (e) {
+                            return ((!filteredValues || filteredValues.includes(item)) && (jsxs(Fragment, { children: [showHeader, jsxs("div", { children: [jsxs(MenuItem, __assign({ disabled: !!item.disabled, title: item.disabled && typeof item.disabled !== "boolean" ? item.disabled : "", value: item.value, onFocus: function () {
+                                                    setSelectedItem(item);
+                                                    if (item.subComponent) {
+                                                        subComponentIsValid.current = true;
+                                                        subComponentInitialValue.current = null;
+                                                    }
+                                                    else {
+                                                        setSubSelectedValue(null);
+                                                        subComponentIsValid.current = true;
+                                                    }
+                                                }, onClick: function (e) {
                                                     if (item.subComponent) {
                                                         e.keepOpen = true;
                                                     }
@@ -3508,7 +3507,7 @@ var GridFormDropDown = function (props) {
                                                                 return [2 /*return*/];
                                                             });
                                                         }); }
-                                                    } }, { children: item.subComponent && (jsx("div", __assign({ className: "subComponent" }, { children: jsx(item.subComponent, {}, "".concat(fieldToString(field), "-").concat(index, "_subcomponent_inner")) }))) }))); } }), "".concat(item.label, "_subcomponent")))] }, "menu-wrapper-".concat(index))] })));
+                                                    } }, { children: item.subComponent && (jsx("div", __assign({ className: "subComponent" }, { children: jsx(item.subComponent, {}, "".concat(fieldToString(field), "-").concat(index, "_subcomponent_inner")) }))) }))); } }), "".concat(item.label, "_subcomponent")))] }, "menu-wrapper-".concat(index))] }, "".concat(index))));
                         })] }) }))] }));
 };