offset broken on dom update

[RyanMulready]

When scrolling to an element and using offset it appears the calculation is made based on the first view of the dom. Any changes to the dom and the offset behaves incorrectly.

[flesler]

Not 100% sure what you mean, the plugin won't auto update the scroll if you move elements around or change the dom in any way. Think of this plugin as a wrapper to $().animate()

[RyanMulready]

It's hard to explain outside of the context but this jsfiddle may help: http://jsfiddle.net/r8WZt/8/

Essentially the object I'm scrolling to is inside a toggled div. When the first scrollTo is triggered the offset works fine but subsequent scrollTo to the same object the offset is wrong.

[flesler]

Not sure what this demo tries to achieve, you are scrolling to the element
with id "6" that is hidden unless you expand, so there's no point in
scrolling unless it's visible (btw ids shouldn't start with a number). Once
you "expand" and scroll, the plugin seems to be respecting the offset
correctly. Trying this on Firefox Mac.

On Wed, Jul 31, 2013 at 12:16 PM, Residual Envy notifications@github.comwrote:

It's hard to explain outside of the context but this jsfiddle may help:
http://jsfiddle.net/r8WZt/8/

Essentially the object I'm scrolling to is inside a toggled div. When the
first scrollTo is triggered the offset works fine but subsequent scrollTos
to the same div the offset is wrong.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/49#issuecomment-21870558
.

Ariel Flesler