]> source.dussan.org Git - jquery.git/commitdiff
Event: Remove the event.which shim
authorMichał Gołębiowski-Owczarek <m.goleb@gmail.com>
Wed, 26 Aug 2020 12:10:33 +0000 (14:10 +0200)
committerGitHub <noreply@github.com>
Wed, 26 Aug 2020 12:10:33 +0000 (14:10 +0200)
All supported browsers implement this property by themselves. The shim was only
needed for IE <9.

Fixes gh-3235
Closes gh-4765
Ref gh-4755

src/event.js

index 96056a13815f6b1710221fcf678a581a2fee007a..b8c5e96fe3da2ab1ef2fbec82b32c59829b66408 100644 (file)
@@ -11,10 +11,7 @@ import nodeName from "./core/nodeName.js";
 import "./core/init.js";
 import "./selector.js";
 
-var
-       rkeyEvent = /^key/,
-       rmouseEvent = /^(?:mouse|pointer|contextmenu|drag|drop)|click/,
-       rtypenamespace = /^([^.]*)(?:\.(.+)|)/;
+var rtypenamespace = /^([^.]*)(?:\.(.+)|)/;
 
 function returnTrue() {
        return true;
@@ -717,34 +714,7 @@ jQuery.each( {
        targetTouches: true,
        toElement: true,
        touches: true,
-
-       which: function( event ) {
-               var button = event.button;
-
-               // Add which for key events
-               if ( event.which == null && rkeyEvent.test( event.type ) ) {
-                       return event.charCode != null ? event.charCode : event.keyCode;
-               }
-
-               // Add which for click: 1 === left; 2 === middle; 3 === right
-               if ( !event.which && button !== undefined && rmouseEvent.test( event.type ) ) {
-                       if ( button & 1 ) {
-                               return 1;
-                       }
-
-                       if ( button & 2 ) {
-                               return 3;
-                       }
-
-                       if ( button & 4 ) {
-                               return 2;
-                       }
-
-                       return 0;
-               }
-
-               return event.which;
-       }
+       which: true
 }, jQuery.event.addProp );
 
 jQuery.each( { focus: "focusin", blur: "focusout" }, function( type, delegateType ) {