fix: retry scrolls to ensure unmounted sizes are not estimated

Author: Lucki2g

Website/components/datamodelview/List.tsx

@@ -159,9 +159,7 @@ export const List = ({ setCurrentIndex }: IListProps) => {
             return;
         }
 
-        rowVirtualizer.scrollToIndex(sectionIndex, { 
-            align: 'start'
-        });
+        smartScrollToIndex(sectionIndex);
 
     }, [flatItems]);
 
@@ -175,9 +173,7 @@ export const List = ({ setCurrentIndex }: IListProps) => {
             return;
         }
 
-        rowVirtualizer.scrollToIndex(groupIndex, { 
-            align: 'start'
-        });
+        smartScrollToIndex(groupIndex);
     }, [flatItems]);
 
     const restoreSection = useCallback(() => {
@@ -202,6 +198,23 @@ export const List = ({ setCurrentIndex }: IListProps) => {
         }
     }, [rowVirtualizer]);
 
+    const smartScrollToIndex = useCallback((index: number) => {
+        rowVirtualizer.scrollToIndex(index, { align: 'start' });
+
+        const tryFix = () => {
+            const mounted = rowVirtualizer.getVirtualItems().some(v => v.index === index);
+            if (!mounted) {
+                requestAnimationFrame(tryFix);
+                return;
+            }
+
+            requestAnimationFrame(() => {
+                rowVirtualizer.scrollToIndex(index, { align: 'start' });
+            });
+        };
+        requestAnimationFrame(tryFix);
+        }, [rowVirtualizer]);
+
     return (
         <>
             <Box className={`absolute w-full h-full flex items-center justify-center z-[100] transition-opacity duration-300 ${loadingSection ? 'opacity-100' : 'opacity-0 pointer-events-none'}`}>