The issue I'm facing is that calling update changes the props for the instance, but does not seem to update the actual scroll behavior. Taking a look at the method:
I can see that computeScrollOffsets is called BEFORE the props are updated. It seems to me that computing scroll offsets ought to be done AFTER changing the props.
Manually calling computeScrollOffsets after using update in my own implementation results in the desired change to scroll behavior.
I'm happy to submit a PR to move computeScrollOffsets below prop changes if this logic seems correct.
Incidentally, I found the issue where this code was introduced (#412) - I suspect the problem I'm seeing is hidden by the example where update is called on resize, where update is probably called multiple times as the browser resize event fires multiple times (thus recomputing the previous prop updates, but never recomputing the very last prop update).
The issue I'm facing is that calling
update
changes the props for the instance, but does not seem to update the actual scroll behavior. Taking a look at the method:https://github.com/dollarshaveclub/stickybits/blob/62627976d67bdd4b22247a664d97fb00b2cb5e57/src/stickybits.js#L443-L455
I can see that
computeScrollOffsets
is called BEFORE the props are updated. It seems to me that computing scroll offsets ought to be done AFTER changing the props.Manually calling
computeScrollOffsets
after usingupdate
in my own implementation results in the desired change to scroll behavior.I'm happy to submit a PR to move
computeScrollOffsets
below prop changes if this logic seems correct.Incidentally, I found the issue where this code was introduced (#412) - I suspect the problem I'm seeing is hidden by the example where
update
is called onresize
, whereupdate
is probably called multiple times as the browser resize event fires multiple times (thus recomputing the previous prop updates, but never recomputing the very last prop update).