fix: move virtual scrolling logic to GridSizeStore for convenience

Author: yordan-st

packages/pluggableWidgets/datagrid-web/src/model/hooks/useInfiniteControl.tsx

@@ -42,21 +42,7 @@ export function useInfiniteControl(): [trackTableScrolling: ((e: any) => void) |
     );
 
     useEffect(() => {
-        const timer = setTimeout(() => {
-            if (!isVisible) {
-                return;
-            }
-            const wasLocked = gridSizeStore.gridContainerHeight !== undefined;
-            gridSizeStore.lockGridContainerHeight();
-            const justLocked = !wasLocked && gridSizeStore.gridContainerHeight !== undefined;
-
-            if (justLocked) {
-                const gridBody = gridSizeStore.gridBodyRef.current;
-                if (gridBody && gridSizeStore.hasMoreItems && gridBody.scrollHeight <= gridBody.clientHeight) {
-                    gridSizeStore.bumpPage();
-                }
-            }
-        }, 100);
+        const timer = setTimeout(() => isVisible && gridSizeStore.lockGridContainerHeight(), 100);
         return () => clearTimeout(timer);
     });
 

packages/pluggableWidgets/datagrid-web/src/model/stores/GridSize.store.ts

@@ -112,5 +112,10 @@ export class GridSizeStore {
         const overflows = gridContainer.scrollHeight > fullHeight;
         this.gridContainerHeight = fullHeight - (overflows ? 0 : VIRTUAL_SCROLLING_OFFSET);
         this.lockedAtPageSize = currentPageSize;
+
+        const gridBody = this.gridBodyRef.current;
+        if (this.hasMoreItems && gridBody && gridBody.scrollHeight <= gridBody.clientHeight) {
+            this.bumpPage();
+        }
     }
 }