fix: calculation for {force: false, offset: nonZero} (#139)

Undo the pre-applied offset when calculating whether the element is already visible (and therefore we shouldn't scroll). Fixes #138
rigor789 · Sep 10, 2019 · f01ce23 · f01ce23
1 parent cbe0c1a
commit f01ce23
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 511 deletions.
diff --git a/src/scrollTo.js b/src/scrollTo.js
@@ -196,7 +196,7 @@ export const scroller = () => {
                     : container.offsetHeight;
             const containerTop = initialY;
             const containerBottom = containerTop + containerHeight;
-            const elementTop = targetY;
+            const elementTop = targetY - offset;
             const elementBottom = elementTop + element.offsetHeight;
             if (
                 elementTop >= containerTop &&