Merge pull request #54881 from callstack-internal/remove-forcing-re-l…

…ayout-on-useViewportOffsetTop
Expensify · Jan 12, 2025 · f6774ca · f6774ca
2 parents 0349d96 + a68a091
commit f6774ca
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/hooks/useViewportOffsetTop/index.ts b/src/hooks/useViewportOffsetTop/index.ts
@@ -40,7 +40,8 @@ export default function useViewportOffsetTop(shouldAdjustScrollView = false): nu
     useEffect(() => addViewportResizeListener(updateOffsetTop), [updateOffsetTop]);
 
     useEffect(() => {
-        if (!shouldAdjustScrollView) {
+        // We don't want to trigger window.scrollTo when we are already at the target position. It causes unnecessary style recalculations.
+        if (!shouldAdjustScrollView || viewportOffsetTop === window.scrollY) {
             return;
         }
         window.scrollTo({top: viewportOffsetTop, behavior: 'smooth'});