@linzjs/step-ag-grid 7.6.1 → 7.7.0

package/dist/GridTheme.scss

@@ -75,8 +75,6 @@
   }
 
   .ag-cell.ag-cell-inline-editing {
-    padding-left: 9px;
-    padding-right: 9px;
     bottom: 0;
   }
 
@@ -84,9 +82,11 @@
     word-break: break-word;
   }
 
-  .ag-cell-popup-editing,
-  .ag-selected-for-edit,
+  .ag-cell.ag-cell-popup-editing,
+  .ag-cell.ag-selected-for-edit,
   .ag-cell-inline-editing {
+    padding-left: 9px;
+    padding-right: 9px;
     background: rgb(72 160 244 / 20%);
     // These are important, it needs to override ag-grid to work
     border: 3px solid #48a0f4 !important;

package/dist/index.js

@@ -1327,7 +1327,7 @@ var wait$2 = function (timeoutMs) {
     });
 };
 var isFloat = function (value) {
-    var regexp = /^-?\d+(\.\d+)?$/;
+    var regexp = /^-?\d*(\.\d+)?$/;
     return regexp.test(value);
 };
 var findParentWithClass = function (className, child) {
@@ -2287,7 +2287,10 @@ var GridContextProvider = function (props) {
                                                         case 0:
                                                             // Need to refresh to get spinners to work on all rows
                                                             gridApi.refreshCells({ rowNodes: props.selectedRows, force: true });
-                                                            return [4 /*yield*/, fnUpdate(selectedRows)];
+                                                            return [4 /*yield*/, fnUpdate(selectedRows)["catch"](function (ex) {
+                                                                    console.error("Exception during modifyUpdating", ex);
+                                                                    return false;
+                                                                })];
                                                         case 1:
                                                             ok = _a.sent();
                                                             return [2 /*return*/];
@@ -2435,7 +2438,7 @@ var GridPopoverContextProvider = function (_a) {
     var selectedRows = React.useMemo(function () { return (multiEdit ? lodashEs.sortBy(getSelectedRows(), function (row) { return row.id !== props.data.id; }) : [props.data]); }, [getSelectedRows, multiEdit, props.data]);
     var field = (_d = (_c = props.colDef) === null || _c === void 0 ? void 0 : _c.field) !== null && _d !== void 0 ? _d : "";
     var updateValue = React.useCallback(function (saveFn, tabDirection) { return __awaiter(void 0, void 0, void 0, function () {
-        var r, _a;
+        var _a;
         return __generator(this, function (_b) {
             switch (_b.label) {
                 case 0:
@@ -2449,10 +2452,7 @@ var GridPopoverContextProvider = function (_a) {
                 case 2:
                     _a = _b.sent();
                     _b.label = 3;
-                case 3:
-                    r = _a;
-                    //if (r) stopEditing();
-                    return [2 /*return*/, r];
+                case 3: return [2 /*return*/, _a];
             }
         });
     }); }, [field, saving, selectedRows, updatingCells]);
@@ -2889,7 +2889,7 @@ var GridCellRenderer = function (props) {
 var GridCell = function (props, custom) {
     var _a;
     return __assign(__assign(__assign(__assign(__assign(__assign({ colId: props.field, sortable: !!((props === null || props === void 0 ? void 0 : props.field) || (props === null || props === void 0 ? void 0 : props.valueGetter)), resizable: true }, ((custom === null || custom === void 0 ? void 0 : custom.editor) && {
-        cellClassRules: GridCellMultiSelectClassRules,
+        cellClassRules: custom.multiEdit ? GridCellMultiSelectClassRules : undefined,
         editable: (_a = props.editable) !== null && _a !== void 0 ? _a : true,
         cellEditor: GenericCellEditorComponentWrapper(custom)
     })), { suppressKeyboardEvent: function (e) {
@@ -4197,17 +4197,40 @@ var TextInputValidator = function (props, value, data, context) {
         return null;
     // This can happen because subcomponent is invoked without type safety
     if (typeof value !== "string") {
-        console.error("Value is not a string", value);
-        return null;
+        return "Value is not a string";
     }
-    if (props.required && value.length === 0) {
-        return "Some text is required";
+    if (props.required && value == "") {
+        return "Must not be empty";
     }
     if (props.maxLength && value.length > props.maxLength) {
-        return "Text must be no longer than ".concat(props.maxLength, " characters");
+        return "Must be no longer than ".concat(props.maxLength, " characters");
     }
     if (props.maxBytes && stringByteLengthIsInvalid(value, props.maxBytes)) {
-        return "Text must be no longer than ".concat(props.maxLength, " bytes");
+        return "Must be no longer than ".concat(props.maxBytes, " bytes");
+    }
+    var nf = props.numberFormat;
+    if (nf) {
+        if (value != "" && !isFloat(value)) {
+            return "Must be a valid number";
+        }
+        var number = parseFloat(value);
+        if (nf.gtMin != null && number <= nf.gtMin) {
+            return "Must be greater than ".concat(nf.gtMin);
+        }
+        if (nf.geMin != null && number < nf.geMin) {
+            return "Must not be less than ".concat(nf.geMin);
+        }
+        if (nf.ltMax != null && number >= nf.ltMax) {
+            return "Must be less than ".concat(nf.ltMax);
+        }
+        if (nf.leMax != null && number > nf.leMax) {
+            return "Must not be greater than ".concat(nf.leMax);
+        }
+        if (nf.precision != null && value != "") {
+            if (parseFloat(number.toPrecision(nf.precision + 1)) != number) {
+                return nf.precision == 0 ? "Must be a whole number" : "Must have no more than ".concat(nf.precision, " decimal places");
+            }
+        }
     }
     if (props.invalid) {
         return props.invalid(value, data, context);
@@ -4443,6 +4466,12 @@ var ActionButton = function (_a) {
 var useDeferredPromise = function () {
     var promiseResolve = React.useRef();
     var promiseReject = React.useRef();
+    // End promise on unload
+    React.useEffect(function () {
+        return function () {
+            promiseReject.current && promiseReject.current();
+        };
+    }, []);
     return {
         invoke: function () {
             return new Promise(function (resolve, reject) {