Removed unneeded reference to `window.event`. #12610 #968

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
@saiwong
Contributor

saiwong commented Oct 15, 2012

The need for window.event does not seem to be present for the current
set of support browser/platforms. Use of event || window.event syntax
seems only relevant when attaching event listeners via the following
syntax:

document.body.onclick = function(event){
  var e = event || window.event; // event is null in the case of IE6
  ...
}
Sai Wong
Removed unneeded reference to `window.event`. #12610
The need for `window.event` does not seem to be present for the current
set of support browser/platforms. Use of `event || window.event` syntax
seems only relevant when attaching event listeners via the following
syntax:

    document.body.onclick = function(event){
      var e = event || window.event; // event is null in the case of IE6
      ...
    }
@dmethvin

This comment has been minimized.

Show comment Hide comment
@dmethvin

dmethvin Oct 20, 2012

Owner

/botio test

Owner

dmethvin commented Oct 20, 2012

/botio test

@dmethvin dmethvin closed this in 5228f0a Oct 20, 2012

mescoda pushed a commit to mescoda/jquery that referenced this pull request Nov 4, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment