@oracle/oraclejet 18.0.1 → 18.0.3

package/dist/js/libs/oj/debug_esm/ojtable.js

@@ -2068,26 +2068,16 @@ Table.prototype._checkViewportRejected = function () {
  * @private
  */
 Table.prototype._clearDataWaitingState = function (ignoreStatusMessage) {
-  if (this._pendingFetchStale) {
-    this._pendingFetchStale = false;
-    this._queueTask(
-      function () {
-        this._getLayoutManager().notifyTableUpdate(Table._UPDATE._DATA_REFRESH);
-        this._beforeDataRefresh();
-        return this._invokeDataFetchRows(this._pendingFetchOptions);
-      }.bind(this)
-    );
-  } else {
-    if (!ignoreStatusMessage) {
-      this._hideStatusMessage();
-    }
-    this._dataFetching = false;
-    this._pendingFetchStale = false;
+  if (!ignoreStatusMessage) {
+    this._hideStatusMessage();
+  }
+  this._dataFetching = false;
+  if (!this._hasRefreshInQueue) {
     this._pendingFetchOptions = null;
-    if (this._dataResolveFunc) {
-      this._dataResolveFunc();
-      this._dataResolveFunc = null;
-    }
+  }
+  if (this._dataResolveFunc) {
+    this._dataResolveFunc();
+    this._dataResolveFunc = null;
   }
 };
 
@@ -2242,8 +2232,6 @@ Table.prototype._clearScrollPosBusyState = function () {
  */
 Table.prototype._clearAllComponentBusyStates = function () {
   this._clearBusyStateStack();
-  // clear any pending fetch state to ensure clearing data waiting state below does not queue addition fetches
-  this._pendingFetchStale = false;
   this._clearDataWaitingState();
   this._clearFocusoutBusyState();
   this._clearIdleRenderBusyState();
@@ -2260,6 +2248,7 @@ Table.prototype._cleanComponent = function (isDestroy) {
   // cleanup needed for both, 'destroy()' and 'ReleaseResources()' calls
   this._animateOnFetch = null;
   this._isEditPending = null;
+  this._deferredEditInfo = null;
   this._focusoutEditKey = null;
   this._skipScrollOnFocus = null;
   this._active = null;
@@ -2587,17 +2576,15 @@ Table.prototype._refresh = function () {
   this._clearIdleCallback();
 
   // update edit state based on row removals before rows are removed to ensure row context is available
-  if (this._hasEditableRow()) {
-    // exit edit mode if editable row is deleted - treat as cancelling the edit
-    this._setTableEditable(false, true);
-  }
+  var beforeFetchPromise = this._hasEditableRow()
+    ? this._setTableEditable(false, true)
+    : Promise.resolve();
 
   // ensure we register 'refresh' update here
   this._getLayoutManager().notifyTableUpdate(Table._UPDATE._REFRESH);
-  if (initFetch) {
-    return this._initFetch();
-  }
-  return this._invokeDataFetchRows();
+  return beforeFetchPromise.then(() => {
+    return initFetch ? this._initFetch() : this._invokeDataFetchRows();
+  });
 };
 
 /**
@@ -4507,21 +4494,34 @@ Table.prototype._handleSortTableHeaderColumn = function (columnIdx, ascending, e
  * @private
  */
 Table.prototype._processFetchSort = function (result) {
-  try {
-    var fetchParameters = result.fetchParameters;
-    var sortCriteria = fetchParameters[Table._CONST_SORTCRITERIA];
-    if (sortCriteria != null && sortCriteria.length > 0) {
-      var sortAttribute = sortCriteria[0].attribute;
-      var sortDirection = sortCriteria[0].direction === Table._COLUMN_SORT_ORDER._ASCENDING;
-      this._refreshSortTableHeaderColumn(sortAttribute, sortDirection);
-      // set the current row
-      this._setCurrentRow(this.options.currentRow);
-    } else {
-      this._clearSortedHeaderColumn();
-    }
-  } catch (e) {
-    error(e);
-  }
+  return new Promise(
+    function (resolve) {
+      try {
+        var fetchParameters = result.fetchParameters;
+        var sortCriteria = fetchParameters[Table._CONST_SORTCRITERIA];
+        if (sortCriteria != null && sortCriteria.length > 0) {
+          var sortAttribute = sortCriteria[0].attribute;
+          var sortDirection = sortCriteria[0].direction === Table._COLUMN_SORT_ORDER._ASCENDING;
+          this._refreshSortTableHeaderColumn(sortAttribute, sortDirection);
+          // set the current row
+          var currentStatus = this._setCurrentRow(this.options.currentRow);
+          if (currentStatus instanceof Promise) {
+            currentStatus.then(() => {
+              resolve();
+            });
+          } else {
+            resolve();
+          }
+        } else {
+          this._clearSortedHeaderColumn();
+          resolve();
+        }
+      } catch (e) {
+        error(e);
+        resolve();
+      }
+    }.bind(this)
+  );
 };
 
 /**
@@ -4983,7 +4983,7 @@ Table.prototype._registerDomScroller = function () {
   // eslint-disable-next-line no-unused-vars
   this._domScrollerErrorFunc = function (reason) {
     // only clear data waiting state on abort if fetch is stale (mutation / refresh events)
-    if (controller && controller.signal.aborted && this._pendingFetchStale) {
+    if (controller && controller.signal.aborted && this._hasRefreshInQueue) {
       this._clearDataWaitingState();
     }
     this._clearScrollBuffer();
@@ -5976,11 +5976,12 @@ Table.prototype._invokeDataFetchRows = function (options) {
               ).then(
                 function () {
                   this._clearDataWaitingState();
-                  this._processFetchSort(value);
-                  if (this._isLoadMoreOnScroll()) {
-                    this._registerDomScroller();
-                  }
-                  resolve(result);
+                  this._processFetchSort(value).then(() => {
+                    if (this._isLoadMoreOnScroll()) {
+                      this._registerDomScroller();
+                    }
+                    resolve(result);
+                  });
                 }.bind(this),
                 // eslint-disable-next-line no-unused-vars
                 function (reason) {
@@ -6060,15 +6061,14 @@ Table.prototype._invokeDataSort = function (sortField, ascending, event) {
     header: sortCriteria[0][Table._CONST_ATTRIBUTE],
     direction: sortCriteria[0].direction
   });
-  this._beforeDataRefresh(true);
   // show the Fetching Data... message
   this._showStatusMessage();
-  this._queueTask(
-    function () {
-      this._getLayoutManager().notifyTableUpdate(Table._UPDATE._DATA_SORT);
+  this._queueTask(() => {
+    this._getLayoutManager().notifyTableUpdate(Table._UPDATE._DATA_SORT);
+    return this._beforeDataRefresh(true).then(() => {
       return this._initFetch({ sortCriteria: sortCriteria }, true);
-    }.bind(this)
-  );
+    });
+  });
 };
 
 /**
@@ -12775,6 +12775,22 @@ Table.prototype._unregisterDataSourceEventListeners = function () {
   }
 };
 
+/**
+ * Helper method to handle mutation events while a pending fetch is already in progress
+ */
+Table.prototype._handleStaleDataMutation = function (options) {
+  if (this._controller) {
+    this._controller.abort(getAbortReason(this.OuterWrapper));
+  }
+  this._hasRefreshInQueue = true;
+  this._queueTask(() => {
+    this._getLayoutManager().notifyTableUpdate(Table._UPDATE._DATA_REFRESH);
+    return this._beforeDataRefresh().then(() => {
+      return this._invokeDataFetchRows(options);
+    });
+  });
+};
+
 /**
  * Callback handler for refresh in the datasource. Refresh entire
  * table body DOM and refresh the table dimensions.
@@ -12789,11 +12805,7 @@ Table.prototype._handleDataRefresh = function (event) {
     }
     // if already handling a fetch, mark pending fetch result as stale (to ensure new fetch is triggered), abort, and return
     if (this._dataFetching) {
-      if (this._controller) {
-        this._controller.abort(getAbortReason(this.OuterWrapper));
-      }
-      this._pendingFetchStale = true;
-      this._pendingFetchOptions = null;
+      this._handleStaleDataMutation();
       return;
     }
     if (event.detail && event.detail.disregardAfterKey !== undefined) {
@@ -12802,24 +12814,24 @@ Table.prototype._handleDataRefresh = function (event) {
           this._getLayoutManager().notifyTableUpdate(Table._UPDATE._ROWS_REMOVED);
           // reset active row to ensure active row index is not stale after refresh
           this._clearActiveRow(null, true);
-          this._removeRowsAfterLastValidRow(event.detail.disregardAfterKey);
-          if (!this._hasMoreToFetch()) {
-            this._registerDomScroller();
-          }
-          this._animateOnFetch = true;
+          this._removeRowsAfterLastValidRow(event.detail.disregardAfterKey).then(() => {
+            if (!this._hasMoreToFetch()) {
+              this._registerDomScroller();
+            }
+            this._animateOnFetch = true;
+          });
         }.bind(this)
       );
     } else {
       this._hasRefreshInQueue = true;
-      this._queueTask(
-        function () {
-          this._getLayoutManager().notifyTableUpdate(Table._UPDATE._DATA_REFRESH);
-          // reset active row to ensure active row index is not stale after refresh
-          this._clearActiveRow(null, true);
-          this._beforeDataRefresh();
+      this._queueTask(() => {
+        this._getLayoutManager().notifyTableUpdate(Table._UPDATE._DATA_REFRESH);
+        // reset active row to ensure active row index is not stale after refresh
+        this._clearActiveRow(null, true);
+        return this._beforeDataRefresh().then(() => {
           return this._invokeDataFetchRows();
-        }.bind(this)
-      );
+        });
+      });
     }
   } catch (e) {
     error(e);
@@ -12838,13 +12850,12 @@ Table.prototype._beforeDataRefresh = function (isSort) {
   if (this._lastSelectedRowIdxArray != null) {
     this._lastSelectedRowIdxArray = [];
   }
-  // update edit state based on row removals before rows are removed to ensure row context is available
-  if (this._hasEditableRow()) {
-    // exit edit mode if editable row is deleted - treat as cancelling the edit
-    this._setTableEditable(false, true);
-  }
   // reset selection validation state if data refreshed or changed
   this._initialSelectionStateValidated = false;
+
+  // update edit state based on row removals before rows are removed to ensure row context is available
+  // exit edit mode if editable row is deleted - treat as cancelling the edit
+  return this._hasEditableRow() ? this._setTableEditable(false, true) : Promise.resolve();
 };
 
 /**
@@ -12853,24 +12864,33 @@ Table.prototype._beforeDataRefresh = function (isSort) {
  * @private
  */
 Table.prototype._removeRowsAfterLastValidRow = function (lastValidKey) {
-  if (this._getCurrentScrollPosition().y > 0) {
-    this._bufferScrollerForLastRow(lastValidKey);
-  }
-  var lastValidItemIdx = this._getRowIdxForRowKey(lastValidKey);
-  if (lastValidItemIdx >= 0) {
-    // update edit state based on row removals before rows are removed to ensure row context is available
-    if (this._hasEditableRow()) {
-      if (this._getEditableRowIdx() > lastValidItemIdx) {
+  return new Promise(
+    function (resolve) {
+      if (this._getCurrentScrollPosition().y > 0) {
+        this._bufferScrollerForLastRow(lastValidKey);
+      }
+      var lastValidItemIdx = this._getRowIdxForRowKey(lastValidKey);
+      if (lastValidItemIdx >= 0) {
+        // update edit state based on row removals before rows are removed to ensure row context is available
         // exit edit mode if editable row is deleted - treat as cancelling the edit
-        this._setTableEditable(false, true);
+        var updateEditable =
+          this._hasEditableRow() && this._getEditableRowIdx() > lastValidItemIdx
+            ? this._setTableEditable(false, true)
+            : Promise.resolve();
+
+        updateEditable.then(() => {
+          var tableBodyRows = this._getTableBodyRows();
+          var tableBodyRowsCount = tableBodyRows.length;
+          for (var i = tableBodyRowsCount - 1; i > lastValidItemIdx; i--) {
+            this._removeTableBodyRow(i);
+          }
+          resolve();
+        });
+      } else {
+        resolve();
       }
-    }
-    var tableBodyRows = this._getTableBodyRows();
-    var tableBodyRowsCount = tableBodyRows.length;
-    for (var i = tableBodyRowsCount - 1; i > lastValidItemIdx; i--) {
-      this._removeTableBodyRow(i);
-    }
-  }
+    }.bind(this)
+  );
 };
 
 /**
@@ -12907,14 +12927,15 @@ Table.prototype._bufferScrollerForLastRow = function (lastValidKey) {
  * @private
  */
 Table.prototype._handleDataRowMutate = function (event) {
-  if (this._dataFetching) {
-    if (this._controller) {
-      this._controller.abort(getAbortReason(this.OuterWrapper));
+  try {
+    // if a refresh is already in the queue, but a fetch hasn't started, just return
+    if (this._hasRefreshInQueue) {
+      return;
+    }
+    if (this._dataFetching) {
+      this._handleStaleDataMutation(this._pendingFetchOptions);
+      return;
     }
-    this._pendingFetchStale = true;
-    return;
-  }
-  try{
     var self = this;
     this._queueTask(() => {
       return self._handleDataRowRemove(event.detail.remove, event.detail.add).then((resetFocus) => {
@@ -13012,15 +13033,6 @@ Table.prototype._handleDataRowRemove = function (eventDetail, addEventDetail) {
       }
     }
 
-    // update edit state based on row removals before rows are removed to ensure row context is available
-    if (self._hasEditableRow()) {
-      var editRowKey = self._getEditableRowKey();
-      if (containsKey([...eventDetail[Table._CONST_KEYS]], editRowKey)) {
-        // exit edit mode if editable row is deleted - treat as cancelling the edit
-        self._setTableEditable(false, true);
-      }
-    }
-
     // eslint-disable-next-line no-inner-declarations
     function _removeRows() {
       var ii;
@@ -13102,26 +13114,33 @@ Table.prototype._handleDataRowRemove = function (eventDetail, addEventDetail) {
       return Promise.resolve();
     }
 
-    if (removeAll) {
-      self._removeAllTableBodyRows();
-      return _afterRemoveRows().then(function () {
-        return Promise.resolve(resetFocus);
-      });
-    }
     return new Promise(function (resolve) {
-      if (self._IsCustomElement()) {
-        return self
-          ._animateVisibleRows(removedTableBodyRows, rowIdxArray, 'remove')
-          .then(function () {
+      // update edit state based on row removals before rows are removed to ensure row context is available
+      // exit edit mode if editable row is deleted - treat as cancelling the edit
+      var updateEditable =
+        self._hasEditableRow() &&
+        containsKey(eventDetailKeys, self._getEditableRowKey())
+          ? self._setTableEditable(false, true)
+          : Promise.resolve();
+
+      updateEditable.then(() => {
+        if (removeAll) {
+          self._removeAllTableBodyRows();
+          _afterRemoveRows().then(() => {
+            resolve(resetFocus);
+          });
+        } else {
+          var handleAnimation = self._IsCustomElement()
+            ? self._animateVisibleRows(removedTableBodyRows, rowIdxArray, 'remove')
+            : Promise.resolve();
+
+          handleAnimation.then(() => {
             _removeRows();
-            return _afterRemoveRows().then(function () {
+            _afterRemoveRows().then(() => {
               resolve(resetFocus);
             });
           });
-      }
-      _removeRows();
-      return _afterRemoveRows().then(function () {
-        resolve(resetFocus);
+        }
       });
     });
   }
@@ -17522,7 +17541,9 @@ Table.prototype._handleFocusout = function (event) {
       if (isClearEditMode) {
         if (this._focusoutEditKey != null) {
           // only clear edit mode if it was enabled prior to the focusout trigger AND the editRow has not changed
-          this._setTableEditable(false, false, 0, true, event);
+          this._queueTask(() => {
+            return this._setTableEditable(false, false, 0, true, event);
+          });
         } else {
           // otherwise ensure focus remains in the Table if it has not yet been regained and an editrow update is pending
           table.focus();
@@ -17723,7 +17744,7 @@ Table.prototype._events = {
     var keyboardCode1 = this._getKeyboardKeys()[0];
 
     if (
-      this._isEditPending ||
+      this._isPendingEdit() ||
       (!isEscapeKeyEvent(keyboardCode1) &&
         !isEnterKeyEvent(keyboardCode1) &&
         !isF2KeyEvent(keyboardCode1) &&
@@ -17832,7 +17853,7 @@ Table.prototype._events = {
     if (
       isEventClickthroughDisabled(event, this._getTable()) ||
       (isFromDefaultSelector(event) && !isShift) ||
-      this._isEditPending
+      this._isPendingEdit()
     ) {
       return;
     }
@@ -17918,7 +17939,7 @@ Table.prototype._events = {
   'mousedown .oj-table-column-header-cell': function (event) {
     if (
       isEventClickthroughDisabled(event, this._getTable()) ||
-      this._isEditPending
+      this._isPendingEdit()
     ) {
       return;
     }
@@ -17935,7 +17956,7 @@ Table.prototype._events = {
   'mousedown .oj-table-footer-cell': function (event) {
     if (
       isEventClickthroughDisabled(event, this._getTable()) ||
-      this._isEditPending
+      this._isPendingEdit()
     ) {
       return;
     }
@@ -18003,7 +18024,7 @@ Table.prototype._events = {
   'mousedown .oj-table-data-cell': function (event) {
     if (
       isEventClickthroughDisabled(event, this._getTable()) ||
-      this._isEditPending
+      this._isPendingEdit()
     ) {
       return;
     }
@@ -18032,7 +18053,7 @@ Table.prototype._events = {
    * invoke a sort on the column data when the mouse clicks the sort icon
    */
   'click .oj-table-sort-icon-container': function (event) {
-    if (this._isEditPending) {
+    if (this._isPendingEdit()) {
       return;
     }
     var columnIdx = this._getElementColumnIdx(event.target);
@@ -18061,7 +18082,7 @@ Table.prototype._events = {
   'click .oj-table-body-row': function (event) {
     if (
       isEventClickthroughDisabled(event, this._getTable()) ||
-      this._isEditPending ||
+      this._isPendingEdit() ||
       !this._shouldAllowSelection()
     ) {
       return;
@@ -18166,7 +18187,13 @@ Table.prototype._events = {
           columnIdx = this._getElementColumnIdx(event.target);
         }
         const rowIdx = this._getElementRowIdx(event.target);
-        this._setEditRow({ rowIndex: rowIdx }, columnIdx);
+        if (rowIdx != null) {
+          const rowKey = this._getRowKeyForRowIdx(rowIdx);
+          this._queueTask(() => {
+            return this._setEditRow();
+          });
+          this._deferredEditInfo = { editRow: { rowKey }, columnIdx };
+        }
       }
       this._currentPointerTarget = null;
       this._previousPointerTarget = null;
@@ -18178,7 +18205,7 @@ Table.prototype._events = {
   'contextmenu .oj-table-data-cell': function (event) {
     if (
       isEventClickthroughDisabled(event, this._getTable()) ||
-      this._isEditPending
+      this._isPendingEdit()
     ) {
       return;
     }
@@ -18194,7 +18221,7 @@ Table.prototype._events = {
   'click .oj-table-column-header-cell': function (event) {
     if (
       isEventClickthroughDisabled(event, this._getTable()) ||
-      this._isEditPending ||
+      this._isPendingEdit() ||
       !this._shouldAllowSelection()
     ) {
       return;
@@ -18243,7 +18270,7 @@ Table.prototype._events = {
   'click .oj-table-footer-cell': function (event) {
     if (
       isEventClickthroughDisabled(event, this._getTable()) ||
-      this._isEditPending ||
+      this._isPendingEdit() ||
       !this._shouldAllowSelection()
     ) {
       return;
@@ -18288,7 +18315,7 @@ Table.prototype._events = {
   'click .oj-table-add-row-placeholder': function (event) {
     if (
       isEventClickthroughDisabled(event, this._getTable()) ||
-      this._isEditPending
+      this._isPendingEdit()
     ) {
       return;
     }
@@ -18299,7 +18326,7 @@ Table.prototype._events = {
    * Set dragstart handler for column DnD.
    */
   'dragstart .oj-table-column-header-cell': function (event) {
-    if (this._isEditPending) {
+    if (this._isPendingEdit()) {
       return undefined;
     }
     var eventTarget = this._getEventTargetElement(event);
@@ -18348,7 +18375,7 @@ Table.prototype._events = {
    * handle the dragstart event on rows and invoke event callback.
    */
   'dragstart .oj-table-body-row': function (event) {
-    if (this._isEditPending) {
+    if (this._isPendingEdit()) {
       return undefined;
     }
     return this._getTableDndContext().handleRowDragStart(event);
@@ -18554,7 +18581,9 @@ Table.prototype._registerTouchEvents = function () {
           return function () {
             var eventTarget = this._getEventTargetElement(event);
             var columnIdx = this._getElementColumnIdx(eventTarget);
-            this._setTableEditable(true, false, columnIdx, true, event);
+            this._queueTask(() => {
+              return this._setTableEditable(true, false, columnIdx, true, event);
+            });
             event.preventDefault();
           }.bind(this);
         }.bind(this)(evt)
@@ -19432,36 +19461,30 @@ Table.prototype._setCurrentRow = function (currentRow, event, optionChange) {
         (updatedCurrentRow == null ||
           !oj$1.KeyUtils.equals(existingCurrentRowKey, updatedCurrentRow.rowKey))
       ) {
-        var updateEditable = this._setTableEditable(false, false, 0, true, event);
-        if (updateEditable instanceof Promise) {
-          return updateEditable.then(
-            function (value) {
-              if (value === false) {
-                this._resetCurrentRow(existingCurrentRow, existingCurrentRowIdx, event);
-                return Table._CURRENT_ROW_STATUS._IGNORED;
-              }
-              var existingBodyRow = this._getTableBodyRow(existingCurrentRowIdx);
-              if (existingBodyRow != null) {
-                existingBodyRow.classList.remove(Table.CSS_CLASSES._TABLE_DATA_CURRENT_ROW_CLASS);
-              }
-              this._currentRow = updatedCurrentRow;
-
-              this._updateCurrentRow(isClearCurrentRow, localRowIndex, event);
-              if (event == null) {
-                // only scroll to the new current row and highlight it if the Table has focus
-                this._setActiveRow(localRowIndex, event, false, !this._hasFocus());
-              }
-              return Table._CURRENT_ROW_STATUS._UPDATED;
-            }.bind(this),
-            function () {
+        return this._setTableEditable(false, false, 0, true, event).then(
+          (value) => {
+            if (value === false) {
               this._resetCurrentRow(existingCurrentRow, existingCurrentRowIdx, event);
-              return Table._CURRENT_ROW_STATUS._VETOED;
-            }.bind(this)
-          );
-        } else if (updateEditable === false) {
-          this._resetCurrentRow(existingCurrentRow, existingCurrentRowIdx, event);
-          return Table._CURRENT_ROW_STATUS._VETOED;
-        }
+              return Table._CURRENT_ROW_STATUS._IGNORED;
+            }
+            var existingBodyRow = this._getTableBodyRow(existingCurrentRowIdx);
+            if (existingBodyRow != null) {
+              existingBodyRow.classList.remove(Table.CSS_CLASSES._TABLE_DATA_CURRENT_ROW_CLASS);
+            }
+            this._currentRow = updatedCurrentRow;
+
+            this._updateCurrentRow(isClearCurrentRow, localRowIndex, event);
+            if (event == null) {
+              // only scroll to the new current row and highlight it if the Table has focus
+              this._setActiveRow(localRowIndex, event, false, !this._hasFocus());
+            }
+            return Table._CURRENT_ROW_STATUS._UPDATED;
+          },
+          () => {
+            this._resetCurrentRow(existingCurrentRow, existingCurrentRowIdx, event);
+            return Table._CURRENT_ROW_STATUS._VETOED;
+          }
+        );
       }
       var existingBodyRow = this._getTableBodyRow(existingCurrentRowIdx);
       if (existingBodyRow != null) {
@@ -19775,16 +19798,19 @@ Table.prototype._setActive = function (
       // update the current row if we are making a row active
       if (active.type === Table.ACTIVE_ELEMENT_TYPES._DATA_ROW) {
         if (updateCurrent) {
-          const updated = this._setCurrentRow({ rowKey: active.key }, event);
-          if (updated instanceof Promise) {
-            return updated.then((value) => {
-              if (value !== Table._CURRENT_ROW_STATUS._UPDATED) {
-                return false;
-              }
-              this._updateActive(true, active, skipScroll, skipFocus, scrollStuckRow);
-              return true;
+          const currentStatus = this._setCurrentRow({ rowKey: active.key }, event);
+          if (currentStatus instanceof Promise) {
+            this._queueTask(() => {
+              return currentStatus.then((value) => {
+                if (value !== Table._CURRENT_ROW_STATUS._UPDATED) {
+                  return false;
+                }
+                this._updateActive(true, active, skipScroll, skipFocus, scrollStuckRow);
+                return true;
+              });
             });
-          } else if (updated !== Table._CURRENT_ROW_STATUS._UPDATED) {
+            return true;
+          } else if (currentStatus !== Table._CURRENT_ROW_STATUS._UPDATED) {
             return false;
           }
         }
@@ -19799,17 +19825,20 @@ Table.prototype._setActive = function (
     // update the current row if we are clearing an active row
     if (this._hasActiveRow()) {
       if (updateCurrent) {
-        const updated = this._setCurrentRow(null, event);
-        if (updated instanceof Promise) {
-          return updated.then((value) => {
-            if (value !== Table._CURRENT_ROW_STATUS._UPDATED) {
-              return false;
-            }
-            this._unhighlightActive();
-            this._active = null;
-            return true;
+        const currentStatus = this._setCurrentRow(null, event);
+        if (currentStatus instanceof Promise) {
+          this._queueTask(() => {
+            return currentStatus.then((value) => {
+              if (value !== Table._CURRENT_ROW_STATUS._UPDATED) {
+                return false;
+              }
+              this._unhighlightActive();
+              this._active = null;
+              return true;
+            });
           });
-        } else if (updated !== Table._CURRENT_ROW_STATUS._UPDATED) {
+          return true;
+        } else if (currentStatus !== Table._CURRENT_ROW_STATUS._UPDATED) {
           return false;
         }
       }
@@ -20001,9 +20030,10 @@ Table.prototype._updateScrollPositionFromEventDetailRemove = function (eventDeta
     }
   }
   if (this._scrollPosition != null) {
-    var scrollPosRowKey = this._scrollPosition.rowKey != null
-      ? this._scrollPosition.rowKey
-      : this._getRowKeyForRowIdx(this._scrollPosition.rowIndex);
+    var scrollPosRowKey =
+      this._scrollPosition.rowKey != null
+        ? this._scrollPosition.rowKey
+        : this._getRowKeyForRowIdx(this._scrollPosition.rowIndex);
     if (scrollPosRowKey != null) {
       eventDetail[Table._CONST_KEYS].forEach((key) => {
         if (oj$1.KeyUtils.equals(key, scrollPosRowKey)) {
@@ -20831,69 +20861,134 @@ Table.prototype._hasEditableRow = function () {
  * Sets a row to be editable
  * @private
  */
-Table.prototype._setEditRow = function (editRow, columnIdx = 0) {
-  if (!this._isTableEditMode()) {
-    return;
-  }
-  if (this._isEditPending !== true) {
-    this._delayed_edit = null;
-    var rowKey = editRow.rowKey;
-    var rowIndex = editRow.rowIndex;
-    if (rowKey != null || rowIndex > -1) {
-      // an editable row has to be current also
-      var changed = this._setCurrentRow({ rowKey: rowKey, rowIndex: rowIndex });
-      if (changed instanceof Promise) {
-        changed.then(
-          function (value) {
-            if (value === Table._CURRENT_ROW_STATUS._UPDATED) {
-              this._setEditableHelper(columnIdx);
-            } else if (value === Table._CURRENT_ROW_STATUS._IGNORED) {
-              this._scrollToRowHelper(editRow, columnIdx);
+Table.prototype._setEditRow = function () {
+  return new Promise(
+    function (resolve) {
+      if (!this._isTableEditMode()) {
+        resolve();
+      }
+      this._isEditPending = true;
+
+      if (this._deferredEditInfo != null) {
+        var editRow = this._deferredEditInfo.editRow;
+        var columnIdx = this._deferredEditInfo.columnIdx || 0;
+        this._deferredEditInfo = null;
+
+        var rowKey = editRow.rowKey;
+        var rowIndex = editRow.rowIndex;
+        if (rowKey != null || rowIndex > -1) {
+          // an editable row has to be current also
+          var currentStatus = this._setCurrentRow({ rowKey: rowKey, rowIndex: rowIndex });
+          if (currentStatus instanceof Promise) {
+            currentStatus.then(
+              function (value) {
+                if (value === Table._CURRENT_ROW_STATUS._UPDATED) {
+                  this._setTableEditable(true, false, columnIdx, true, null).then(
+                    () => {
+                      this._isEditPending = false;
+                      resolve();
+                    },
+                    () => {
+                      this._isEditPending = false;
+                      resolve();
+                    }
+                  );
+                } else if (value === Table._CURRENT_ROW_STATUS._IGNORED) {
+                  this._scrollToRowHelper(editRow, columnIdx, resolve);
+                }
+              }.bind(this)
+            );
+          } else if (currentStatus === Table._CURRENT_ROW_STATUS._UPDATED) {
+            this._setTableEditable(true, false, columnIdx, true, null).then(
+              () => {
+                this._isEditPending = false;
+                resolve();
+              },
+              () => {
+                this._isEditPending = false;
+                resolve();
+              }
+            );
+          } else if (currentStatus === Table._CURRENT_ROW_STATUS._IGNORED) {
+            this._scrollToRowHelper(editRow, columnIdx, resolve);
+          }
+        } else {
+          this._setTableEditable(false, false, 0, true, null).then(
+            () => {
+              this._isEditPending = false;
+              resolve();
+            },
+            () => {
+              this._isEditPending = false;
+              resolve();
             }
-          }.bind(this)
-        );
-      } else if (changed === Table._CURRENT_ROW_STATUS._UPDATED) {
-        this._setEditableHelper(columnIdx);
-      } else if (changed === Table._CURRENT_ROW_STATUS._IGNORED) {
-        this._scrollToRowHelper(editRow, columnIdx);
+          );
+        }
+      } else {
+        this._isEditPending = false;
+        resolve();
       }
-    } else {
-      this._setTableEditable(false, false, 0, true, null);
-    }
-  } else {
-    this._delayed_edit = { editRow: editRow, columnIdx: columnIdx };
-  }
+    }.bind(this)
+  );
 };
 
-/**
- * Helper method to set table editable
- * @private
- */
-Table.prototype._setEditableHelper = function (columnIdx) {
-  if (this._delayed_edit != null) {
-    return;
-  }
-  this._setTableEditable(true, false, columnIdx, true, null);
+Table.prototype._isPendingEdit = function () {
+  return this._isEditPending || this._deferredEditInfo != null;
 };
 
 /**
  * Helper method to scroll to row and invoke _setEditRow
  * @private
  */
-Table.prototype._scrollToRowHelper = function (editRow, columnIdx) {
+Table.prototype._scrollToRowHelper = function (editRow, columnIdx, resolve) {
   // try to scroll to editRow.rowKey
-  const _scrollTop = this._scrollTop;
-  if (!this._syncScrollPosition({ rowKey: editRow.rowKey, rowIndex: editRow.rowIndex })) {
-    // this will be invoked after scroll and fetch is done
-    this._editRowCallback = function () {
-      // try again after scroll and fetch
-      if (this._scrollTop !== _scrollTop) {
-        this._setEditRow(editRow, columnIdx);
-      }
-    }.bind(this);
+  const scrollTop = this._scrollTop;
+  if (editRow.rowKey != null) {
+    var validationPromise = this._validateKeyForScroll(editRow.rowKey);
+    if (validationPromise != null) {
+      validationPromise.then(
+        (valid) => {
+          if (valid) {
+            this._setupEditRowCallback(editRow, columnIdx, scrollTop);
+            this._syncScrollPosition({ rowKey: editRow.rowKey }, true);
+          }
+          this._isEditPending = false;
+          resolve();
+        },
+        () => {
+          this._isEditPending = false;
+          resolve();
+        }
+      );
+    } else {
+      this._isEditPending = false;
+      resolve();
+    }
+  } else {
+    if (!this._syncScrollPosition({ rowIndex: editRow.rowIndex })) {
+      this._setupEditRowCallback(editRow, columnIdx, scrollTop);
+    }
+    this._isEditPending = false;
+    resolve();
   }
 };
 
+/**
+ * Helper method to setup a scroll callback for honoring an edit row setting.
+ */
+Table.prototype._setupEditRowCallback = function (editRow, columnIdx, scrollTop) {
+  // this will be invoked after scroll and fetch is done
+  this._editRowCallback = function () {
+    // try again after scroll and fetch
+    if (this._scrollTop !== scrollTop && this._deferredEditInfo == null) {
+      this._queueTask(() => {
+        return this._setEditRow();
+      });
+      this._deferredEditInfo = { editRow, columnIdx };
+    }
+  }.bind(this);
+};
+
 /**
  * Returns the row from editRowObject
  * @private
@@ -20926,29 +21021,17 @@ Table.prototype._setEditableRowIdx = function (rowIdx) {
  */
 Table.prototype._setAdjacentRowEditable = function (columnIdx, origColumnIdx, isNext, event) {
   var editableRowIdx = this._getEditableRowIdx();
-  var updateEditable = this._setTableEditable(false, false, columnIdx, isNext, event);
-  if (updateEditable instanceof Promise) {
-    updateEditable.then(
-      function () {
-        this._queueTask(
-          function () {
-            this._handleAdjacentEditEndSuccessful(columnIdx, editableRowIdx, isNext, event);
-          }.bind(this)
-        );
-      }.bind(this),
-      function () {
-        this._queueTask(
-          function () {
-            this._handleEditEndRejected(origColumnIdx, editableRowIdx, !isNext);
-          }.bind(this)
-        );
-      }.bind(this)
-    );
-  } else if (updateEditable === false) {
-    this._handleEditEndRejected(origColumnIdx, editableRowIdx, !isNext);
-  } else {
-    this._handleAdjacentEditEndSuccessful(columnIdx, editableRowIdx, isNext, event);
-  }
+  return this._setTableEditable(false, false, columnIdx, isNext, event).then(
+    (updateEditable) => {
+      if (updateEditable === false) {
+        return this._handleEditEndRejected(origColumnIdx, editableRowIdx, !isNext);
+      }
+      return this._handleAdjacentEditEndSuccessful(columnIdx, editableRowIdx, isNext, event);
+    },
+    () => {
+      return this._handleEditEndRejected(origColumnIdx, editableRowIdx, !isNext);
+    }
+  );
 };
 
 /**
@@ -20960,10 +21043,10 @@ Table.prototype._handleAdjacentEditEndSuccessful = function (columnIdx, rowIdx,
   var newRowIdx = this._findAdjacentEditableRow(rowIdx, tableBodyRows, isNext);
   if (newRowIdx !== null) {
     this._setActiveRow(newRowIdx, event, true, false, true);
-    this._setTableEditable(true, false, columnIdx, isNext, event);
-  } else {
-    this._getTable().focus();
+    return this._setTableEditable(true, false, columnIdx, isNext, event);
   }
+  this._getTable().focus();
+  return Promise.resolve();
 };
 
 /**
@@ -21026,240 +21109,284 @@ Table.prototype._setTableEditable = function (
   forwardSearch,
   event
 ) {
-  if (!this._isTableEditMode() || this._isEditPending) {
-    return undefined;
-  }
-  var isLegacyDataSource = this._getData() instanceof oj$1.TableDataSourceAdapter;
-  var currentRow = this._getCurrentRow();
-  if (currentRow != null) {
-    const rowElementToBeEdited = this._getRowToBeEdited(currentRow);
-    if (
-      rowElementToBeEdited &&
-      rowElementToBeEdited[Table._DATA_OJ_EDITABLE] !== Table._CONST_OFF
-    ) {
-      var rowKey = currentRow.rowKey;
-      var rowIdx = this._getRowIdxForRowKey(rowKey);
-      var tableBodyRow;
-      var rowContext;
-      var updateEditMode;
-      var newEditRowValue;
-      var editAcceptPromiseArray = [];
-      let updatedItemPromise;
-
-      // save the old editable row info
-      var prevEditableRowKey = this._getEditableRowKey();
+  return new Promise(
+    function (resolve) {
+      if (!this._isTableEditMode()) {
+        resolve();
+        return;
+      }
 
-      try {
-        if (editable && !this._hasEditableRow()) {
-          // fire the beforeEdit event if there are no existing editable rows
-          // and we are starting edit on a row
-          tableBodyRow = this._getTableBodyRow(rowIdx);
-          rowContext = this._getRendererContextObject(tableBodyRow, {
-            row: {
-              key: rowKey,
-              index: currentRow.rowIndex
-            },
-            isCurrentRow: true
-          });
-          rowContext.item = tableBodyRow[Table._ROW_ITEM_EXPANDO];
-          updateEditMode = this._trigger('beforeRowEdit', event, {
-            accept: function (acceptPromise) {
-              editAcceptPromiseArray.push(acceptPromise);
-            },
-            rowContext: rowContext
-          });
-          if (updateEditMode) {
-            newEditRowValue = { rowKey: rowKey, rowIndex: rowIdx };
+      var isLegacyDataSource = this._getData() instanceof oj$1.TableDataSourceAdapter;
+      var currentRow = this._getCurrentRow();
+      if (currentRow != null) {
+        this._isEditPending = true;
+        const rowElementToBeEdited = this._getRowToBeEdited(currentRow);
+        if (
+          rowElementToBeEdited &&
+          rowElementToBeEdited[Table._DATA_OJ_EDITABLE] !== Table._CONST_OFF
+        ) {
+          var rowKey = currentRow.rowKey;
+          var rowIdx = this._getRowIdxForRowKey(rowKey);
+          var tableBodyRow;
+          var rowContext;
+          var updateEditMode;
+          var newEditRowValue;
+          var editAcceptPromiseArray = [];
+          let updatedItemPromise;
+
+          // save the old editable row info
+          var prevEditableRowKey = this._getEditableRowKey();
+
+          try {
+            if (editable && !this._hasEditableRow()) {
+              // fire the beforeEdit event if there are no existing editable rows
+              // and we are starting edit on a row
+              tableBodyRow = this._getTableBodyRow(rowIdx);
+              rowContext = this._getRendererContextObject(tableBodyRow, {
+                row: {
+                  key: rowKey,
+                  index: currentRow.rowIndex
+                },
+                isCurrentRow: true
+              });
+              rowContext.item = tableBodyRow[Table._ROW_ITEM_EXPANDO];
+              updateEditMode = this._trigger('beforeRowEdit', event, {
+                accept: function (acceptPromise) {
+                  editAcceptPromiseArray.push(acceptPromise);
+                },
+                rowContext: rowContext
+              });
+              if (updateEditMode) {
+                newEditRowValue = { rowKey: rowKey, rowIndex: rowIdx };
+              }
+            } else if (!editable && this._hasEditableRow()) {
+              // only trigger the beforeRowEditEnd if we are actually ending an edit
+              // fire on the edited row
+              tableBodyRow = this._getTableBodyRow(this._getEditableRowIdx());
+              rowKey = this._getRowKeyForRowIdx(this._getEditableRowIdx());
+              rowContext = this._getRendererContextObject(tableBodyRow, {
+                row: {
+                  key: rowKey,
+                  index: this._getDataSourceRowIndexForRowKey(rowKey)
+                },
+                isCurrentRow: true
+              });
+              rowContext.item = tableBodyRow[Table._ROW_ITEM_EXPANDO];
+              updateEditMode = this._trigger('beforeRowEditEnd', event, {
+                accept: function (acceptPromise) {
+                  editAcceptPromiseArray.push(acceptPromise);
+                },
+                setUpdatedItem: function (updatedPromise) {
+                  updatedItemPromise = updatedPromise;
+                },
+                cancelEdit: cancelled,
+                rowContext: rowContext
+              });
+
+              if (updateEditMode) {
+                newEditRowValue = { rowKey: null, rowIndex: -1 };
+              }
+            } else {
+              // No updates so just exit
+              this._isEditPending = false;
+              resolve();
+              return;
+            }
+          } catch (err) {
+            this._isEditPending = false;
+            resolve(false);
+            return;
           }
-        } else if (!editable && this._hasEditableRow()) {
-          // only trigger the beforeRowEditEnd if we are actually ending an edit
-          // fire on the edited row
-          tableBodyRow = this._getTableBodyRow(this._getEditableRowIdx());
-          rowKey = this._getRowKeyForRowIdx(this._getEditableRowIdx());
-          rowContext = this._getRendererContextObject(tableBodyRow, {
-            row: {
-              key: rowKey,
-              index: this._getDataSourceRowIndexForRowKey(rowKey)
-            },
-            isCurrentRow: true
-          });
-          rowContext.item = tableBodyRow[Table._ROW_ITEM_EXPANDO];
-          updateEditMode = this._trigger('beforeRowEditEnd', event, {
-            accept: function (acceptPromise) {
-              editAcceptPromiseArray.push(acceptPromise);
-            },
-            setUpdatedItem: function (updatedPromise) {
-              updatedItemPromise = updatedPromise;
-            },
-            cancelEdit: cancelled,
-            rowContext: rowContext
-          });
+          if (!updateEditMode) {
+            this._syncActiveElement();
+            this._isEditPending = false;
+            resolve(false);
+            return;
+          }
+          var table = this._getTable();
 
-          if (updateEditMode) {
-            newEditRowValue = { rowKey: null, rowIndex: -1 };
+          // shift focus to the overall table if the target row contains focus - otherwise, the overall
+          // focusout handler may kick in, or tooltips or other ihteractive elements may cause issues
+          if (tableBodyRow.contains(document.activeElement)) {
+            // flag used to skip scrolling 'focused' things while the Table
+            // is actively updating focus due to editing requirements
+            this._isSkipScrollOnFocus = !cancelled;
+            table.focus();
           }
-        } else {
-          // No updates so just exit
-          return undefined;
-        }
-      } catch (err) {
-        return false;
-      }
-      if (!updateEditMode) {
-        this._syncActiveElement();
-        return false;
-      }
-      var table = this._getTable();
 
-      // shift focus to the overall table if the target row contains focus - otherwise, the overall
-      // focusout handler may kick in, or tooltips or other ihteractive elements may cause issues
-      if (tableBodyRow.contains(document.activeElement)) {
-        // flag used to skip scrolling 'focused' things while the Table
-        // is actively updating focus due to editing requirements
-        this._isSkipScrollOnFocus = !cancelled;
-        table.focus();
-      }
+          var _handleAsyncEdit = function (updatedItemObj) {
+            this._removeSkeletonRow(tableBodyRow);
+            // recalculate the row index as it could have changed due to other tasks
+            var editableRowIndex = this._getRowIdxForRowKey(rowKey);
+            if (editable && rowIdx !== editableRowIndex) {
+              newEditRowValue.rowIndex = editableRowIndex;
+            }
+            // update editRow option
+            this._fireEditRowChangeEvent(newEditRowValue, event);
 
-      var _handleAsyncEdit = function (updatedItemObj) {
-        this._removeSkeletonRow(tableBodyRow);
-        // recalculate the row index as it could have changed due to other tasks
-        var editableRowIndex = this._getRowIdxForRowKey(rowKey);
-        if (editable && rowIdx !== editableRowIndex) {
-          newEditRowValue.rowIndex = editableRowIndex;
-        }
-        // update editRow option
-        this._fireEditRowChangeEvent(newEditRowValue, event);
-
-        if (editable) {
-          // set the editable row index
-          this._setTableActionableMode(false, true);
-          this._setEditableRowIdx(editableRowIndex);
-          table.setAttribute(Table.DOM_ATTR._ARIA_LABELLEDBY, ''); // @HTMLUpdateOK
-          // re-render the newly editable row
-          return this._refreshRow(editableRowIndex, false, !isLegacyDataSource).then(
-            function () {
-              this._setCellInRowFocus(editableRowIndex, columnIdx, forwardSearch);
-              // clear out the old editable row
-              return this._refreshPrevEditableRow(prevEditableRowKey, updatedItemObj, false);
-            }.bind(this)
-          );
-        }
-        this._focusEditCell = null;
-        this._setEditableRowIdx(null);
+            if (editable) {
+              // set the editable row index
+              this._setTableActionableMode(false, true);
+              this._setEditableRowIdx(editableRowIndex);
+              table.setAttribute(Table.DOM_ATTR._ARIA_LABELLEDBY, ''); // @HTMLUpdateOK
+              // re-render the newly editable row
+              return this._refreshRow(editableRowIndex, false, !isLegacyDataSource).then(() => {
+                this._setCellInRowFocus(editableRowIndex, columnIdx, forwardSearch);
+                // clear out the old editable row
+                return this._refreshPrevEditableRow(prevEditableRowKey, updatedItemObj, false);
+              });
+            }
+            this._focusEditCell = null;
+            this._setEditableRowIdx(null);
 
-        // clear out the old editable row
-        return this._refreshPrevEditableRow(prevEditableRowKey, updatedItemObj, cancelled);
-      }.bind(this);
+            // clear out the old editable row
+            return this._refreshPrevEditableRow(prevEditableRowKey, updatedItemObj, cancelled);
+          }.bind(this);
 
-      // handle asynchronous editing if an accept promise was provided
-      if (editAcceptPromiseArray.length !== 0) {
-        this._prepareAsyncRowEdit(tableBodyRow);
-        return this._queueTask(
-          function () {
-            this._editPromise = Promise.all(editAcceptPromiseArray).then(
-              function () {
+          // handle asynchronous editing if an accept promise was provided
+          if (editAcceptPromiseArray.length !== 0) {
+            this._prepareAsyncRowEdit(tableBodyRow);
+            Promise.all(editAcceptPromiseArray).then(
+              () => {
                 if (updatedItemPromise != null) {
-                  return updatedItemPromise.then(
-                    function (updatedItemObj) {
-                      return _handleAsyncEdit(updatedItemObj);
+                  updatedItemPromise.then(
+                    (updatedItemObj) => {
+                      _handleAsyncEdit(updatedItemObj).then(
+                        () => {
+                          this._isEditPending = false;
+                          resolve();
+                        },
+                        () => {
+                          this._isEditPending = false;
+                          resolve();
+                        }
+                      );
+                    },
+                    () => {
+                      _handleAsyncEdit().then(
+                        () => {
+                          this._isEditPending = false;
+                          resolve();
+                        },
+                        () => {
+                          this._isEditPending = false;
+                          resolve();
+                        }
+                      );
+                    }
+                  );
+                } else {
+                  _handleAsyncEdit().then(
+                    () => {
+                      this._isEditPending = false;
+                      resolve();
                     },
-                    function () {
-                      return _handleAsyncEdit();
+                    () => {
+                      this._isEditPending = false;
+                      resolve();
                     }
                   );
                 }
-                return _handleAsyncEdit();
               },
-              function () {
+              () => {
                 this._removeSkeletonRow(tableBodyRow);
                 if (editable && prevEditableRowKey == null) {
                   this._syncActiveElement();
                 }
-                this._clearEditPending();
-                return false;
-              }.bind(this)
+                this._isSkipScrollOnFocus = false;
+                this._isEditPending = false;
+                resolve(false);
+              }
             );
-            return this._editPromise;
-          }.bind(this)
-        );
-      }
+            return;
+          }
 
-      // check if an updated item promise was provided for otherwise synchronous editing
-      if (updatedItemPromise != null) {
-        this._prepareAsyncRowEdit(tableBodyRow);
-        return this._queueTask(
-          function () {
-            this._editPromise = updatedItemPromise.then(
-              function (updatedItemObj) {
-                return _handleAsyncEdit(updatedItemObj);
+          // check if an updated item promise was provided for otherwise synchronous editing
+          if (updatedItemPromise != null) {
+            this._prepareAsyncRowEdit(tableBodyRow);
+            updatedItemPromise.then(
+              (updatedItemObj) => {
+                _handleAsyncEdit(updatedItemObj).then(
+                  () => {
+                    this._isEditPending = false;
+                    resolve();
+                  },
+                  () => {
+                    this._isEditPending = false;
+                    resolve();
+                  }
+                );
               },
-              function () {
-                return _handleAsyncEdit();
+              () => {
+                _handleAsyncEdit().then(
+                  () => {
+                    this._isEditPending = false;
+                    resolve();
+                  },
+                  () => {
+                    this._isEditPending = false;
+                    resolve();
+                  }
+                );
               }
             );
-            return this._editPromise;
-          }.bind(this)
-        );
-      }
+            return;
+          }
 
-      // otherwise, handle fully synchronous editing
-      if (editable) {
-        // set the editable row index
-        this._setTableActionableMode(false, true);
-        this._setEditableRowIdx(rowIdx);
-        table.setAttribute(Table.DOM_ATTR._ARIA_LABELLEDBY, ''); // @HTMLUpdateOK
-        // re-render the newly editable row
-        this._queueTask(
-          function () {
+          // otherwise, handle fully synchronous editing
+          if (editable) {
+            // set the editable row index
+            this._setTableActionableMode(false, true);
+            this._setEditableRowIdx(rowIdx);
+            table.setAttribute(Table.DOM_ATTR._ARIA_LABELLEDBY, ''); // @HTMLUpdateOK
+            // re-render the newly editable row
             // update editRow option
             this._fireEditRowChangeEvent(newEditRowValue, event);
             // recalculate the row index as it could have changed due to other tasks
             var editableRowIndex = this._getRowIdxForRowKey(rowKey);
-            return this._refreshRow(editableRowIndex, false, !isLegacyDataSource).then(
-              function () {
+            this._refreshRow(editableRowIndex, false, !isLegacyDataSource).then(
+              () => {
                 // set focus on the column in the row
                 this._setCellInRowFocus(editableRowIndex, columnIdx, forwardSearch);
-              }.bind(this)
+                this._refreshPrevEditableRow(prevEditableRowKey, null, cancelled).then(() => {
+                  this._isEditPending = false;
+                  resolve();
+                });
+              },
+              () => {
+                this._isEditPending = false;
+                resolve();
+              }
             );
-          }.bind(this)
-        );
-      } else {
-        this._focusEditCell = null;
-        this._setEditableRowIdx(null);
-        this._queueTask(
-          function () {
-            // update editRow option
-            this._fireEditRowChangeEvent(newEditRowValue, event);
-          }.bind(this)
-        );
+            return;
+          }
+          this._focusEditCell = null;
+          this._setEditableRowIdx(null);
+          // update editRow option
+          this._fireEditRowChangeEvent(newEditRowValue, event);
+          this._refreshPrevEditableRow(prevEditableRowKey, null, cancelled).then(
+            () => {
+              this._isEditPending = false;
+              resolve();
+            },
+            () => {
+              this._isEditPending = false;
+              resolve();
+            }
+          );
+          return;
+        }
       }
-      this._queueTask(
-        function () {
-          return this._refreshPrevEditableRow(prevEditableRowKey, null, cancelled);
-        }.bind(this)
-      );
-    }
-  }
-  return undefined;
+      resolve();
+    }.bind(this)
+  );
 };
 
 Table.prototype._prepareAsyncRowEdit = function (tableBodyRow) {
-  this._isEditPending = true;
   this._insertSkeletonRow(tableBodyRow);
   this._syncScrollPosition();
 };
 
-Table.prototype._clearEditPending = function () {
-  this._isEditPending = false;
-  this._isSkipScrollOnFocus = false;
-  this._editPromise = null;
-  if (this._delayed_edit != null) {
-    this._queueTask(() => {
-      this._setEditRow(this._delayed_edit.editRow, this._delayed_edit.columnIdx);
-    });
-  }
-};
-
 Table.prototype._refreshPrevEditableRow = function (prevEditableRowKey, updatedItemObj, cancelled) {
   // make sure the row still exists before we refresh it
   const prevEditableRowIdx = this._getRowIdxForRowKey(prevEditableRowKey);
@@ -21270,11 +21397,12 @@ Table.prototype._refreshPrevEditableRow = function (prevEditableRowKey, updatedI
     }
     // re-render the previously editable row, which will be read-only now
     return this._refreshRow(prevEditableRowIdx, false, cancelled, rowItem).then(() => {
-      this._clearEditPending();
+      this._skipScrollOnFocus = false;
     });
   }
-  this._clearEditPending();
-  return Promise.resolve();
+  return Promise.resolve().then(() => {
+    this._skipScrollOnFocus = false;
+  });
 };
 
 /**
@@ -21627,13 +21755,17 @@ Table.prototype._handleKeydownTab = function (event) {
       rowElementTabIndex === tabbableElementsInRowCount - 1 &&
       !event[Table._KEYBOARD_CODES._MODIFIER_SHIFT]
     ) {
-      // last tabbable element in row so go to the next row
-      this._setAdjacentRowEditable(-1, maxIndex, true, event);
+      this._queueTask(() => {
+        // last tabbable element in row so go to the next row
+        return this._setAdjacentRowEditable(-1, maxIndex, true, event);
+      });
       event.preventDefault();
       event.stopPropagation();
     } else if (rowElementTabIndex === 0 && event[Table._KEYBOARD_CODES._MODIFIER_SHIFT]) {
-      // first tabbable element in row and Shift+Tab so go to the previous row
-      this._setAdjacentRowEditable(maxIndex, 0, false, event);
+      this._queueTask(() => {
+        // first tabbable element in row and Shift+Tab so go to the previous row
+        return this._setAdjacentRowEditable(maxIndex, 0, false, event);
+      });
       event.preventDefault();
       event.stopPropagation();
     }
@@ -21758,17 +21890,14 @@ Table.prototype._handleKeydownEnter = function (event) {
     var currentRowIdx = currentRow.rowIndex != null ? currentRow.rowIndex : -1;
     if (currentRowIdx >= 0) {
       if (this._isTableEditMode()) {
-        if (!this._hasEditableRow()) {
-          this._setTableEditable(true, false, -1, true, event);
-          return;
-        }
-        var columnIdx = this._getElementColumnIdx(event.target);
-
-        if (!event[Table._KEYBOARD_CODES._MODIFIER_SHIFT]) {
-          this._setAdjacentRowEditable(columnIdx, columnIdx, true, event);
-        } else {
-          this._setAdjacentRowEditable(columnIdx, columnIdx, false, event);
-        }
+        this._queueTask(() => {
+          if (!this._hasEditableRow()) {
+            return this._setTableEditable(true, false, -1, true, event);
+          }
+          var columnIdx = this._getElementColumnIdx(event.target);
+          var isNext = !event[Table._KEYBOARD_CODES._MODIFIER_SHIFT];
+          return this._setAdjacentRowEditable(columnIdx, columnIdx, isNext, event);
+        });
       } else {
         this._fireActionEvent(currentRowIdx, event, false);
         this._setTableActionableMode(true);
@@ -21826,14 +21955,16 @@ Table.prototype._handleKeydownF2 = function (event) {
   if (!this._isStickyLayoutEnabled()) {
     if (this._isTableEditMode()) {
       // pressing F2 toggles between editable modes.
-      if (!this._hasEditableRow()) {
-        this._setTableEditable(true, false, 0, true, event);
+      this._queueTask(() => {
         if (!this._hasEditableRow()) {
-          this._toggleTableActionableMode();
+          return this._setTableEditable(true, false, 0, true, event).then(() => {
+            if (!this._hasEditableRow()) {
+              this._toggleTableActionableMode();
+            }
+          });
         }
-      } else {
-        this._setTableEditable(false, false, 0, true, event);
-      }
+        return this._setTableEditable(false, false, 0, true, event);
+      });
     } else {
       this._toggleTableActionableMode();
     }
@@ -21852,7 +21983,9 @@ Table.prototype._handleKeydownEsc = function (event) {
   if (this._hasEditableRow()) {
     event.preventDefault();
     event.stopPropagation();
-    this._setTableEditable(false, true, 0, true, event);
+    this._queueTask(() => {
+      return this._setTableEditable(false, true, 0, true, event);
+    });
   } else if (this._isTableActionableMode()) {
     event.preventDefault();
     event.stopPropagation();
@@ -26647,19 +26780,20 @@ Table.prototype._setOptions = function (options, flags) {
     }
   }
   if (requiresRefresh) {
-    this._queueTask(
-      function () {
-        if (requiresHeaderRefresh) {
-          // clearing the rendered flag triggers a header render as part of _refresh()
-          this._renderedTableHeaderColumns = false;
-        }
-        if (requiresDataRefresh) {
-          this._beforeDataRefresh();
-        }
+    this._queueTask(() => {
+      if (requiresHeaderRefresh) {
+        // clearing the rendered flag triggers a header render as part of _refresh()
+        this._renderedTableHeaderColumns = false;
+      }
+      var beforeRefreshPromise = requiresDataRefresh
+        ? this._beforeDataRefresh()
+        : Promise.resolve();
+
+      return beforeRefreshPromise.then(() => {
         // _refresh clears the layout manager, so do not register 'refresh' update here
         return this._refresh();
-      }.bind(this)
-    );
+      });
+    });
   }
   this._superApply(arguments);
 };
@@ -26673,44 +26807,43 @@ Table.prototype._setOption = function (key, value, flags) {
     // update row index/key values in the 'selection' option
     this._syncRangeSelection(value);
     this._superApply(arguments);
-    this._queueTask(
-      function () {
-        // after applying the new selection value, sync the selection state
-        this._selectionSet = true;
-        this._validateInitialSelectionState(true);
-      }.bind(this)
-    );
+    this._queueTask(() => {
+      // after applying the new selection value, sync the selection state
+      this._selectionSet = true;
+      this._validateInitialSelectionState(true);
+    });
   } else if (key === 'selected') {
     this._superApply(arguments);
-    this._queueTask(
-      function () {
-        // after applying the new selected value, sync the selection state
-        this._selectionSet = false;
-        this._validateInitialSelectionState(true);
-      }.bind(this)
-    );
+    this._queueTask(() => {
+      // after applying the new selected value, sync the selection state
+      this._selectionSet = false;
+      this._validateInitialSelectionState(true);
+    });
   } else if (key === 'currentRow') {
     this._superApply(arguments);
-    this._queueTask(
-      function () {
-        this._setCurrentRow(value, null, true);
-      }.bind(this)
-    ).catch(function () {});
+    this._queueTask(() => {
+      var currentStatus = this._setCurrentRow(value, null, true);
+      if (currentStatus instanceof Promise) {
+        return currentStatus;
+      }
+      return Promise.resolve();
+    }).catch(function () {});
   } else if (key === 'scrollPosition') {
     this._superApply(arguments);
-    this._queueTask(
-      function () {
-        // syncScrollPosition will update with proper value
-        this._syncScrollPosition(value);
-      }.bind(this)
-    );
+    this._queueTask(() => {
+      // syncScrollPosition will update with proper value
+      this._syncScrollPosition(value);
+    });
   } else if (key === 'editRow') {
-    if (value != null && (value.rowKey != null || value.rowIndex > -1)) {
-      // clear the pending focusout editkey value if a row is becoming editable
-      this._focusoutEditKey = null;
-    }
-    // setEditRow will update with all props within value
-    this._setEditRow(value);
+    this._queueTask(() => {
+      if (value != null && (value.rowKey != null || value.rowIndex > -1)) {
+        // clear the pending focusout editkey value if a row is becoming editable
+        this._focusoutEditKey = null;
+      }
+      // setEditRow will update with all props within value
+      return this._setEditRow();
+    });
+    this._deferredEditInfo = { editRow: value };
   } else if (key === 'scrollPolicyOptions' && flags != null) {
     this._superApply(arguments);
     if (this._isStickyLayoutEnabled()) {
@@ -26725,18 +26858,14 @@ Table.prototype._setOption = function (key, value, flags) {
     }
   } else if (key === 'addRowDisplay') {
     this._superApply(arguments);
-    this._queueTask(
-      function () {
-        return this._refreshAddRowDisplay();
-      }.bind(this)
-    );
+    this._queueTask(() => {
+      return this._refreshAddRowDisplay();
+    });
   } else if (key === 'selectAllControl') {
     this._superApply(arguments);
-    this._queueTask(
-      function () {
-        return this._refreshTableHeader();
-      }.bind(this)
-    );
+    this._queueTask(() => {
+      return this._refreshTableHeader();
+    });
   } else {
     this._superApply(arguments);
   }