@linzjs/step-ag-grid 14.3.1 → 14.3.2

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

@@ -4968,32 +4968,29 @@ var GridContextProvider = function (props) {
                 !((_a = nextColDef.cellEditorParams) === null || _a === void 0 ? void 0 : _a.preventAutoEdit) &&
                 !((_b = nextColDef.cellRendererParams) === null || _b === void 0 ? void 0 : _b.editAction));
         };
-        var foundEditableCell = false;
         // Just in case I've missed something, we don't want the loop to hang everything
-        var maxIterations = 50;
-        var preRow = null;
-        var postRow = null;
-        do {
-            preRow = gridApi.getFocusedCell();
+        for (var maxIterations = 0; maxIterations < 50; maxIterations++) {
+            var preRow = gridApi.getFocusedCell();
             tabDirection === 1 ? gridApi.tabToNextCell() : gridApi.tabToPreviousCell();
-            postRow = gridApi.getFocusedCell();
-            foundEditableCell = focusedCellIsEditable();
-        } while ((preRow === null || preRow === void 0 ? void 0 : preRow.rowIndex) === (postRow === null || postRow === void 0 ? void 0 : postRow.rowIndex) &&
-            (preRow === null || preRow === void 0 ? void 0 : preRow.column) !== (postRow === null || postRow === void 0 ? void 0 : postRow.column) &&
-            !foundEditableCell &&
-            maxIterations-- > 0);
-        if (foundEditableCell) {
-            prePopupOps();
-            var focusedCell = gridApi === null || gridApi === void 0 ? void 0 : gridApi.getFocusedCell();
-            if (focusedCell) {
-                gridApi.startEditingCell({
-                    rowIndex: focusedCell.rowIndex,
-                    colKey: focusedCell.column.getColId()
-                });
-                return false;
+            var postRow = gridApi.getFocusedCell();
+            if ((preRow === null || preRow === void 0 ? void 0 : preRow.rowIndex) !== (postRow === null || postRow === void 0 ? void 0 : postRow.rowIndex) || (preRow === null || preRow === void 0 ? void 0 : preRow.column) === (postRow === null || postRow === void 0 ? void 0 : postRow.column)) {
+                // We didn't find an editable cell in the same row, or the cell column didn't change
+                // implying it was start/end of grid
+                break;
+            }
+            if (focusedCellIsEditable()) {
+                var focusedCell = gridApi === null || gridApi === void 0 ? void 0 : gridApi.getFocusedCell();
+                if (focusedCell) {
+                    prePopupOps();
+                    gridApi.startEditingCell({
+                        rowIndex: focusedCell.rowIndex,
+                        colKey: focusedCell.column.getColId()
+                    });
+                    return true;
+                }
             }
         }
-        return true;
+        return false;
     }, [gridApi, prePopupOps]);
     var updatingCells = useCallback(function (props, fnUpdate, setSaving, tabDirection) { return __awaiter(void 0, void 0, void 0, function () {
         return __generator(this, function (_a) {
@@ -5045,7 +5042,7 @@ var GridContextProvider = function (props) {
                                             postPopupFocusedCell &&
                                             prePopupFocusedCell.current.rowIndex == postPopupFocusedCell.rowIndex &&
                                             prePopupFocusedCell.current.column.getColId() == postPopupFocusedCell.column.getColId()) {
-                                            if (!tabDirection || selectNextEditableCell(tabDirection)) {
+                                            if (!tabDirection || !selectNextEditableCell(tabDirection)) {
                                                 cellEditingCompleteCallbackRef.current && cellEditingCompleteCallbackRef.current();
                                             }
                                         }