From 07cc2d20887db0674e407506b7591436346f410d Mon Sep 17 00:00:00 2001
From: Oleg Utitsyn <utitsyn@gmail.com>
Date: Fri, 5 Apr 2019 22:13:17 +0300
Subject: [PATCH] fix recalculation coordinates of element every time when spy
 handler is invoked

recalculation coordinates must happen only if isDynamic prop is passed!
---
 modules/mixins/scroll-link.js | 18 +++++++++++++++---
 1 file changed, 15 insertions(+), 3 deletions(-)

diff --git a/modules/mixins/scroll-link.js b/modules/mixins/scroll-link.js
index c4e2286d..5546e035 100644
--- a/modules/mixins/scroll-link.js
+++ b/modules/mixins/scroll-link.js
@@ -31,6 +31,12 @@ export default (Component, customScroller) => {
   const scroller = customScroller || defaultScroller;
 
   class Link extends React.PureComponent {
+    element = null;
+    elementBounds = {
+      topBound: 0,
+      bottomBound: 0,
+    }
+
     constructor(props) {
       super(props);
       this.state = {
@@ -75,9 +81,9 @@ export default (Component, customScroller) => {
       }
 
       let to = this.props.to;
-      let element = null;
-      let elemTopBound = 0;
-      let elemBottomBound = 0;
+      let element = this.element;
+      let elemTopBound = this.elementBounds.topBound;
+      let elemBottomBound = this.elementBounds.bottomBound;
       let containerTop = 0;
 
       if (scrollSpyContainer.getBoundingClientRect) {
@@ -92,6 +98,12 @@ export default (Component, customScroller) => {
         let cords = element.getBoundingClientRect();
         elemTopBound = (cords.top - containerTop + y);
         elemBottomBound = elemTopBound + cords.height;
+
+        this.element = element
+        this.elementBounds = {
+          topBound: elemTopBound,
+          bottomBound: elemBottomBound
+        }
       }
 
       let offsetY = y - this.props.offset;