DataGrid: Fix view bouncing when scrolling down when there is grouping and a summary (T1270354) (#31088)

Author: Alyar666

e2e/testcafe-devextreme/tests/dataGrid/common/etalons/T1270354-virtual-native-scrolling-with-grouping-and-summary-fluent.blue.light.png

@@ -7,6 +7,8 @@ import url from '../../../helpers/getPageUrl';
 import { createWidget } from '../../../helpers/createWidget';
 import { safeSizeTest } from '../../../helpers/safeSizeTest';
 import { salesApiMock } from './apiMocks/salesApiMock';
+import { changeTheme } from '../../../helpers/changeTheme';
+import { Themes } from '../../../helpers/themes';
 
 async function getMaxRightOffset(dataGrid: DataGrid): Promise<number> {
   const scrollWidth = await dataGrid.getScrollWidth();
@@ -1909,3 +1911,70 @@ test('DataGrid - The "row" parameter in the FocusedRowChanged event refers to a
     delete (window as TestCaseWindow).dataGridScrollableEventValues;
   }));
 });
+
+// T1270354
+[
+  { theme: Themes.genericLight, useNative: true },
+  { theme: Themes.genericLight, useNative: false },
+  { theme: Themes.materialBlue, useNative: true },
+  { theme: Themes.materialBlue, useNative: false },
+  { theme: Themes.fluentBlue, useNative: true },
+  { theme: Themes.fluentBlue, useNative: false },
+].forEach(({ theme, useNative }) => {
+  test(`Virtual ${useNative ? 'native' : 'simulated'} scrolling - Scrolling to the bottom should work correctly when there is a grouping and a summary (${theme} theme)`, async (t) => {
+    const dataGrid = new DataGrid('#container');
+    const { takeScreenshot, compareResults } = createScreenshotsComparer(t);
+
+    await t
+      .expect(dataGrid.isReady())
+      .ok();
+
+    await dataGrid.scrollBy({ y: 10000 });
+
+    await t
+      .expect(await takeScreenshot(`T1270354-virtual-${useNative ? 'native' : 'simulated'}-scrolling-with-grouping-and-summary-${theme}.png`, '#container'))
+      .ok()
+      .expect(compareResults.isValid())
+      .ok(compareResults.errorMessages());
+  })
+    .before(async () => {
+      await changeTheme(theme);
+      await createWidget('dxDataGrid', {
+        dataSource: [...new Array(50)]
+          .fill(null)
+          .map((_, index) => ({
+            id: index,
+            machine: index,
+            totalTime: 1,
+          })),
+        keyExpr: 'id',
+        showBorders: true,
+        height: 400,
+        width: '100%',
+        columns: [
+          {
+            dataField: 'machine',
+            groupIndex: 0,
+          }, {
+            dataField: 'totalTime',
+          },
+        ],
+        scrolling: {
+          mode: 'virtual',
+          useNative,
+        },
+        grouping: {
+          autoExpandAll: false,
+        },
+        summary: {
+          totalItems: [
+            {
+              column: 'totalTime',
+              summaryType: 'sum',
+            },
+          ],
+        },
+      });
+    })
+    .after(async () => changeTheme(Themes.genericLight));
+});

e2e/testcafe-devextreme/tests/dataGrid/common/etalons/T1270354-virtual-native-scrolling-with-grouping-and-summary-generic.light.png

@@ -312,7 +312,6 @@ export class ResizingController extends modules.ViewController {
     const columnsController = this._columnsController;
     const visibleColumns = columnsController.getVisibleColumns();
     const columnAutoWidth = this.option('columnAutoWidth');
-    const wordWrapEnabled = this.option('wordWrapEnabled');
     const hasUndefinedColumnWidth = visibleColumns.some((column) => !isDefined(column.width));
     let needBestFit = this._needBestFit();
     let hasMinWidth = false;
@@ -354,6 +353,8 @@ export class ResizingController extends modules.ViewController {
       return undefined;
     });
 
+    this._toggleContentMinHeight(this._hasHeight); // T1047239, T1270354
+
     this._setVisibleWidths(visibleColumns, []);
 
     const $element = this.component.$element();
@@ -366,8 +367,6 @@ export class ResizingController extends modules.ViewController {
       resetBestFitMode = true;
     }
 
-    this._toggleContentMinHeight(wordWrapEnabled); // T1047239
-
     if ($element && $element.get(0) && this._maxWidth) {
       delete this._maxWidth;
       $element[0].style.maxWidth = '';
@@ -423,9 +422,7 @@ export class ResizingController extends modules.ViewController {
           this._setVisibleWidths(visibleColumns, resultWidths);
         }
 
-        if (wordWrapEnabled) {
-          this._toggleContentMinHeight(false);
-        }
+        this._toggleContentMinHeight(false);
       });
     });
   }