fix: prevent bumpPage from firing after filter changes

Author: yordan-st

packages/pluggableWidgets/datagrid-web/e2e/DataGrid.spec.js-snapshots/datagrid-virtual-scrolling-chromium-linux.png

@@ -46,11 +46,16 @@ export function useInfiniteControl(): [trackBodyScrolling: ((e: any) => void) |
             if (!isVisible) {
                 return;
             }
+
+            const wasLocked = gridSizeStore.gridBodyHeight !== undefined;
             gridSizeStore.lockGridBodyHeight();
+            const justLocked = !wasLocked && gridSizeStore.gridBodyHeight !== undefined;
 
-            const gridBody = gridSizeStore.gridBodyRef.current;
-            if (gridBody && gridSizeStore.hasMoreItems && gridBody.scrollHeight <= gridBody.clientHeight) {
-                gridSizeStore.bumpPage();
+            if (justLocked) {
+                const gridBody = gridSizeStore.gridBodyRef.current;
+                if (gridBody && gridSizeStore.hasMoreItems && gridBody.scrollHeight <= gridBody.clientHeight) {
+                    gridSizeStore.bumpPage();
+                }
             }
         }, 100);
         return () => clearTimeout(timer);

packages/pluggableWidgets/datagrid-web/e2e/filtering/DataGridFilteringIntegration.spec.js-snapshots/datagridFilteringIntegration-chromium-linux.png

@@ -122,19 +122,17 @@ export class GridSizeStore {
         const viewportHeight = this.computeBodyViewport();
 
         // Don't lock height before the grid body has rendered content.
-        // viewportHeight is 0 when cells have no layout yet, which would
+        // clientHeight is 0 when the element has no layout yet, which would
         // produce a negative height and break scrolling.
         if (viewportHeight <= 0) {
             return;
         }
 
-        // Compare scrollHeight and clientHeight from the same element to correctly
-        // detect whether content already overflows a fixed-height container.
-        // If it does, do not subtract the pre-fetch offset — that would hide the
-        // last rows and trigger the next page too early. Only subtract when the
-        // grid does not yet overflow (auto-height grid) to create a small synthetic
-        // overflow that makes the body scrollable.
-        const overflows = gridBody.scrollHeight > gridBody.clientHeight;
+        // If content already overflows the container (fixed-height grid), do not subtract the
+        // pre-fetch offset — that would hide the last rows and trigger the next page too early.
+        // Only subtract the offset when the grid does not yet overflow (auto-height grid) so
+        // that we create a small synthetic overflow that makes the body scrollable.
+        const overflows = gridBody.scrollHeight > viewportHeight;
         this.gridBodyHeight = viewportHeight - (overflows ? 0 : VIRTUAL_SCROLLING_OFFSET);
         this.lockedAtPageSize = currentPageSize;
     }