New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ScrollLeft and ScrollTop reinitializing themselves in an iframe contentDocument #1945

Closed
hilnius opened this Issue Dec 17, 2014 · 6 comments

Comments

Projects
None yet
5 participants
@hilnius

hilnius commented Dec 17, 2014

When scrolling manually in an iframe using $($('#iframe').contentWindow).scrollTop(y).scrollLeft(x), the scrollLeft reinitializes the scrollTop
For a demo, see this plunker:
http://plnkr.co/edit/tbn2UJ72frQG16vA9Z5y?p=preview

@dmethvin

This comment has been minimized.

Show comment
Hide comment
@dmethvin

dmethvin Dec 17, 2014

Member

Oh that's a good one. It's funny, we go to extra lengths to get the correct window and then we don't use it when setting the values. So what's happening is that the iframe gets the parent window's value.

Member

dmethvin commented Dec 17, 2014

Oh that's a good one. It's funny, we go to extra lengths to get the correct window and then we don't use it when setting the values. So what's happening is that the iframe gets the parent window's value.

@dmethvin dmethvin added this to the 3.0.0 milestone Dec 17, 2014

@timmywil

This comment has been minimized.

Show comment
Hide comment
@timmywil

timmywil Dec 17, 2014

Member

Yea, we're good like that.

Member

timmywil commented Dec 17, 2014

Yea, we're good like that.

@timmywil

This comment has been minimized.

Show comment
Hide comment
@timmywil

timmywil Dec 17, 2014

Member

Easy fix, but it seems we need an iframe test.

Member

timmywil commented Dec 17, 2014

Easy fix, but it seems we need an iframe test.

@dmethvin

This comment has been minimized.

Show comment
Hide comment
@dmethvin

dmethvin Dec 17, 2014

Member

Oh and you're probably better using the iframe window's .scrollTo() when setting both.

Member

dmethvin commented Dec 17, 2014

Oh and you're probably better using the iframe window's .scrollTo() when setting both.

@araghava

This comment has been minimized.

Show comment
Hide comment
@araghava

araghava Dec 20, 2014

Contributor

Is anyone working on this? If so can I take it? @hilnius

Contributor

araghava commented Dec 20, 2014

Is anyone working on this? If so can I take it? @hilnius

@mgol

This comment has been minimized.

Show comment
Hide comment
@mgol

mgol Dec 20, 2014

Member

@araghava If you want, go for it!

Member

mgol commented Dec 20, 2014

@araghava If you want, go for it!

araghava added a commit to araghava/jquery that referenced this issue Dec 21, 2014

araghava added a commit to araghava/jquery that referenced this issue Dec 21, 2014

araghava added a commit to araghava/jquery that referenced this issue Dec 23, 2014

@markelog markelog closed this in d21edb5 Dec 23, 2014

@dmethvin dmethvin modified the milestones: 1.12/2.2, 3.0.0 Jan 7, 2016

@cssmagic cssmagic referenced this issue May 18, 2016

Open

jQuery #5

@lock lock bot locked as resolved and limited conversation to collaborators Jun 19, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.