Skip to content

Commit 10966bb

Browse files
committed
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.
1 parent 7e6b20e commit 10966bb

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

‎src/event.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -329,10 +329,12 @@ jQuery.event = {
329329
} else if ( !event.isDefaultPrevented() ) {
330330
var target = event.target, old,
331331
isClick = jQuery.nodeName(target, "a") && type === "click",
332+
isScroll = target === window && type === "scroll",
332333
special = jQuery.event.special[ type ] || {};
333334

334335
if ( (!special._default || special._default.call( elem, event ) === false) &&
335-
!isClick && !(target && target.nodeName && jQuery.noData[target.nodeName.toLowerCase()]) ) {
336+
!isClick && !isScroll &&
337+
!(target && target.nodeName && jQuery.noData[target.nodeName.toLowerCase()]) ) {
336338

337339
try {
338340
if ( target[ type ] ) {

0 commit comments

Comments
 (0)