]> source.dussan.org Git - jquery.git/commitdiff
Event: Remove pageX/pageY fill for event object
authorDave Methvin <dave.methvin@gmail.com>
Thu, 5 May 2016 01:36:11 +0000 (21:36 -0400)
committerDave Methvin <dave.methvin@gmail.com>
Fri, 6 May 2016 19:32:12 +0000 (15:32 -0400)
Fixes gh-3092
CLoses gh-3106

IE8 was the last major browser missing these.

src/event.js

index ba2a9dca6d8e530389c60364fa15f6de21921f3f..ffaec578df04a140f8951dffa40d619e40f95e7b 100644 (file)
@@ -592,6 +592,8 @@ jQuery.each( {
        detail: true,
        eventPhase: true,
        metaKey: true,
+       pageX: true,
+       pageY: true,
        shiftKey: true,
        view: true,
        "char": true,
@@ -622,40 +624,6 @@ jQuery.each( {
                }
 
                return event.which;
-       },
-
-       pageX: function( event ) {
-               var eventDoc, doc, body;
-
-               // Calculate pageX if missing and clientX available
-               if ( event.pageX == null && event.clientX != null ) {
-                       eventDoc = event.target.ownerDocument || document;
-                       doc = eventDoc.documentElement;
-                       body = eventDoc.body;
-
-                       return event.clientX +
-                               ( doc && doc.scrollLeft || body && body.scrollLeft || 0 ) -
-                               ( doc && doc.clientLeft || body && body.clientLeft || 0 );
-               }
-
-               return event.pageX;
-       },
-
-       pageY: function( event ) {
-               var eventDoc, doc, body;
-
-               // Calculate pageY if missing and clientY available
-               if ( event.pageY == null && event.clientY != null ) {
-                       eventDoc = event.target.ownerDocument || document;
-                       doc = eventDoc.documentElement;
-                       body = eventDoc.body;
-
-                       return event.clientY +
-                               ( doc && doc.scrollTop || body && body.scrollTop || 0 ) -
-                               ( doc && doc.clientTop || body && body.clientTop || 0 );
-               }
-
-               return event.pageY;
        }
 }, jQuery.event.addProp );