Permalink
Browse files

Event: Remove pageX/pageY fill for event object

Fixes gh-3092
CLoses gh-3106

IE8 was the last major browser missing these.
  • Loading branch information...
1 parent e61fccb commit 931f45fc387d68cfce004f786330d18f74cf03e5 @dmethvin dmethvin committed May 5, 2016
Showing with 2 additions and 34 deletions.
  1. +2 −34 src/event.js
View
@@ -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 );

0 comments on commit 931f45f

Please sign in to comment.