DataGrid - Scrolling position is reset to far right on an attempt to scroll left if the most right cell is focused using the keyboard (T1277214) (DevExpress#29373)

Author: tongsonbarbs

e2e/testcafe-devextreme/tests/dataGrid/common/scrolling.ts

@@ -1765,7 +1765,7 @@ test('DataGrid - Gray boxes appear when the push method is used to remove rows i
   });
 });
 
-// T1262288
+// T1262288 - Left to Right
 test('DataGrid - Focused cell moves to the end of the table after horizontal scrolling', async (t) => {
   const dataGrid = new DataGrid('#container');
 
@@ -1794,3 +1794,33 @@ test('DataGrid - Focused cell moves to the end of the table after horizontal scr
     enabled: false,
   },
 }));
+
+// T1277214 - Right to Left
+test('DataGrid - Scrolling position is reset to far right on an attempt to scroll left if the most right cell is focused using the keyboard', async (t) => {
+  const dataGrid = new DataGrid('#container');
+  await dataGrid.scrollTo(t, { x: 1000 });
+  await t
+    .click(dataGrid.getDataCell(0, 20).element)
+    .pressKey('right');
+
+  await t.expect(dataGrid.getDataCell(0, 20).isFocused).ok();
+
+  await dataGrid.scrollBy({ x: -1000 });
+  await dataGrid.scrollBy({ x: 1000 });
+
+  await t.expect(dataGrid.getDataCell(0, 20).isFocused).ok();
+}).before(async () => createWidget('dxDataGrid', {
+  dataSource: getData(1, 20).map((item, index) => ({ ...item, id: index })),
+  keyExpr: 'id',
+  columnWidth: 100,
+  showBorders: true,
+  focusedRowEnabled: true,
+  scrolling: {
+    columnRenderingMode: 'virtual',
+    mode: 'virtual',
+    showScrollbar: 'always',
+  },
+  paging: {
+    enabled: false,
+  },
+}));

packages/devextreme/js/__internal/grids/grid_core/data_controller/m_data_controller.ts

@@ -499,6 +499,7 @@ export class DataController extends DataHelperMixin(modules.Controller) {
       that._columnsController.columnsChanged.remove(updateItemsHandler);
       that.updateItems({
         repaintChangesOnly: false,
+        event: change?.changeTypes?.event,
         virtualColumnsScrolling: change?.changeTypes?.virtualColumnsScrolling,
       });
     };

packages/devextreme/js/__internal/grids/grid_core/keyboard_navigation/m_keyboard_navigation.ts

@@ -303,7 +303,9 @@ export class KeyboardNavigationController extends modules.ViewController {
         ? !isAppend
         : this._isHiddenFocus && isFullUpdate && !e?.virtualColumnsScrolling;
       if (needUpdateFocus) {
-        this._updateFocus(true);
+        const isScrollEvent = !!e?.event?.type;
+        const skipFocusEvent = e?.virtualColumnsScrolling && isScrollEvent;
+        this._updateFocus(true, skipFocusEvent);
       }
     }
   }
@@ -315,9 +317,7 @@ export class KeyboardNavigationController extends modules.ViewController {
 
     if (!$focusedElement.length && e?.virtualColumnsScrolling) {
       const focusedColumnIndex = this._focusedCellPosition?.columnIndex ?? -1;
-      const focusedColumnIndexWithoutOffset = focusedColumnIndex - this._getFocusedColumnIndexOffset(focusedColumnIndex);
-
-      return focusedColumnIndexWithoutOffset >= 0;
+      return this._isColumnRendered(focusedColumnIndex);
     }
 
     return true;
@@ -1567,7 +1567,7 @@ export class KeyboardNavigationController extends modules.ViewController {
     }
   }
 
-  public _updateFocus(isRenderView?) {
+  public _updateFocus(isRenderView?, skipFocusEvent = false) {
     this._updateFocusTimeout = setTimeout(() => {
       if (this._needFocusEditingCell()) {
         this._editingController._focusEditingCell();
@@ -1606,12 +1606,12 @@ export class KeyboardNavigationController extends modules.ViewController {
                 );
                 return;
               }
-              !isFocusedElementDefined && this._focus($cell);
+              !isFocusedElementDefined && this._focus($cell, false, skipFocusEvent);
             } else if (
               !isFocusedElementDefined
               && (this._isNeedFocus || this._isHiddenFocus)
             ) {
-              this._focus($cell, this._isHiddenFocus);
+              this._focus($cell, this._isHiddenFocus, skipFocusEvent);
             }
             if (isEditing && !column?.showEditorAlways) {
               this._focusInteractiveElement.bind(this)($cell);
@@ -2007,7 +2007,6 @@ export class KeyboardNavigationController extends modules.ViewController {
       const isShowWhenGrouped = column && column.showWhenGrouped;
       const isDataCell = column && !$cell.hasClass(COMMAND_EXPAND_CLASS) && isDataRow($row);
       const isValidGroupSpaceColumn = function () {
-        // eslint-disable-next-line radix
         return (
           (!isMasterDetailRow
             && column

packages/devextreme/js/__internal/grids/grid_core/virtual_columns/m_virtual_columns.ts

@@ -50,7 +50,7 @@ const rowsView = (Base: ModuleType<RowsView>) => class VirtualColumnsRowsViewExt
     }
 
     // @ts-expect-error
-    that._columnsController.setScrollPosition(left);
+    that._columnsController.setScrollPosition(left, e.event);
   }
 
   protected _renderCore(e) {
@@ -181,11 +181,13 @@ const columns = (Base: ModuleType<ColumnsController>) => class VirtualColumnsCon
     return this.option('scrolling.columnPageSize');
   }
 
-  private _fireColumnsChanged() {
+  private _fireColumnsChanged(event?) {
     const date: any = new Date();
     this.columnsChanged.fire({
       optionNames: { all: true, length: 1 },
-      changeTypes: { columns: true, virtualColumnsScrolling: true, length: 2 },
+      changeTypes: {
+        columns: true, virtualColumnsScrolling: true, length: 2, event,
+      },
     });
     this._renderTime = (new Date()) as any - date;
   }
@@ -202,25 +204,25 @@ const columns = (Base: ModuleType<ColumnsController>) => class VirtualColumnsCon
     return scrollingTimeout;
   }
 
-  private setScrollPosition(position) {
+  private setScrollPosition(position, event?) {
     const scrollingTimeout = this.getScrollingTimeout();
 
     if (scrollingTimeout > 0) {
       clearTimeout(this._changedTimeout);
 
       this._changedTimeout = setTimeout(() => {
-        this._setScrollPositionCore(position);
+        this._setScrollPositionCore(position, event);
       }, scrollingTimeout);
     } else {
-      this._setScrollPositionCore(position);
+      this._setScrollPositionCore(position, event);
     }
   }
 
   private resize() {
     this._setScrollPositionCore(this._position);
   }
 
-  private _setScrollPositionCore(position) {
+  private _setScrollPositionCore(position, event?) {
     const that = this;
 
     if (that.isVirtualMode()) {
@@ -232,7 +234,7 @@ const columns = (Base: ModuleType<ColumnsController>) => class VirtualColumnsCon
       if (needColumnsChanged) {
         that._beginPageIndex = beginPageIndex;
         that._endPageIndex = endPageIndex;
-        that._fireColumnsChanged();
+        that._fireColumnsChanged(event);
       }
     }
   }

packages/devextreme/testing/tests/DevExpress.ui.widgets.dataGrid/virtualColumns.tests.js

@@ -385,7 +385,7 @@ QUnit.module('Scrolling', { beforeEach: setupModule, afterEach: teardownModule }
         this.columnsController.columnsChanged.add(e => {
             assert.deepEqual(e, {
                 optionNames: { all: true, length: 1 },
-                changeTypes: { columns: true, virtualColumnsScrolling: true, length: 2 }
+                changeTypes: { columns: true, virtualColumnsScrolling: true, length: 2, event: undefined }
             }, 'columnsChanged args');
             columnsChangedPositions.push(pos);
         });
@@ -413,7 +413,7 @@ QUnit.module('Scrolling', { beforeEach: setupModule, afterEach: teardownModule }
         this.columnsController.columnsChanged.add(e => {
             assert.deepEqual(e, {
                 optionNames: { all: true, length: 1 },
-                changeTypes: { columns: true, virtualColumnsScrolling: true, length: 2 }
+                changeTypes: { columns: true, virtualColumnsScrolling: true, length: 2, event: undefined }
             }, 'columnsChanged args');
             columnsChangedPositions.push(pos);
         });