Bug Tracker

#6170 closed bug (fixed)

Opened February 25, 2010 03:55AM UTC

Closed September 19, 2011 07:43PM UTC

Last modified March 13, 2012 01:47PM UTC

jQuery(window).scroll(); causes IE* to scroll to 0,0

Reported by: rlrosario Owned by: dmethvin
Priority: high Milestone: 1.7
Component: event Version: 1.4.1
Keywords: Cc:
Blocked by: Blocking:
Description

Triggering the scroll event on the window ('jQuery(window).scroll();')

should be considered a special case because the window object has

a scroll method that takes two parameters: x,y. On non-IE browsers,

the call gets ignored when parameters aren't passed in. But on IE*,

the call takes on 0,0 as a default and scrolls the page. This issue

is preventing me from triggering scroll handlers this way because

of the undesirable side effect.

I wrote a patch on github:

http://github.com/rlr/jquery/commit/10966bb9b0c46c69697ebf4703b8ddc9906648f3

Attachments (0)
Change History (10)

Changed February 26, 2010 03:20PM UTC by rlrosario comment:1

Any way to edit this bug? It really should be Version 1.4.2 Milestone 1.4.3 but that wasnt available when I submitted it?

Changed November 14, 2010 03:07AM UTC by dmethvin comment:2

priority: → undecided
status: new → open

Changed April 07, 2011 01:20AM UTC by dmethvin comment:3

milestone: 1.4.2 → 1.6
owner: → dmethvin
priority: undecided → low
status: open → assigned

Changed April 17, 2011 09:22PM UTC by john comment:4

milestone: 1.6 → 1.next
priority: low → high

Changed July 11, 2011 09:37PM UTC by dmethvin comment:5

This is the problem where we execute methods on things that are not .nodeType===1 DOM elements, such as window or plain Object. The solution is to not run those methods for such cases.

Changed July 12, 2011 04:04PM UTC by john comment:6

(This was confirmed during the bug triage meeting.)

Changed September 08, 2011 02:01PM UTC by dmethvin comment:7

Since this behavior may be desired (or at least currently used) for plain objects, I'm only planning to exclude window in the 1.7 patch.

Changed September 08, 2011 05:38PM UTC by dmethvin comment:8

milestone: 1.next → 1.7

Changed September 19, 2011 07:43PM UTC by Dave Methvin comment:9

resolution: → fixed
status: assigned → closed

Fix #6170. Don't try to run DOM methods on window.

Changeset: 6872d31f5a7d6162161b61bfd35ad0e8aa545395

Changed September 28, 2011 12:56AM UTC by dmethvin comment:10

#10355 is a duplicate of this ticket.