Permalink
Browse files

Event: Update support comments for mouseenter/mouseleave implementation

Custom mouseenter/mouseleave implementation was needed because of:
1. Safari 6 not implementing mouseenter/mouseleave at all.
2. Chrome sending mouseenter too often.

The second issue has been fixed in Chrome but exists now in Safari 7
(it's fixed in Safari 8) so we have to keep it for now, unfortunately.
  • Loading branch information...
mgol committed Jul 27, 2015
1 parent 93bee47 commit 2792845534e36c39dbb9c8369ed96aaefa560081
Showing with 4 additions and 7 deletions.
  1. +4 −7 src/event.js
View
@@ -772,13 +772,10 @@ jQuery.Event.prototype = {
// so that event delegation works in jQuery.
// Do the same for pointerenter/pointerleave and pointerover/pointerout
//
// Support: Safari<7.0
// Safari doesn't support mouseenter/mouseleave at all.
//
// Support: Chrome 34+
// Mouseenter doesn't perform while left mouse button is pressed
// (and initiated outside the observed element)
// https://code.google.com/p/chromium/issues/detail?id=333868
// Support: Safari 7 only
// Safari sends mouseenter too often; see:
// https://code.google.com/p/chromium/issues/detail?id=470258
// for the description of the bug (it existed in older Chrome versions as well).
jQuery.each({
mouseenter: "mouseover",
mouseleave: "mouseout",

2 comments on commit 2792845

@mgol

This comment has been minimized.

Show comment
Hide comment
@mgol

mgol Jul 27, 2015

Member

@jquery/core For a moment I thought we'll be able to finally rip out our custom mouseenter/mouseleave implementation... but no, Safari 7 needs it. :(

Member

mgol replied Jul 27, 2015

@jquery/core For a moment I thought we'll be able to finally rip out our custom mouseenter/mouseleave implementation... but no, Safari 7 needs it. :(

@markelog

This comment has been minimized.

Show comment
Hide comment
@markelog

markelog Jul 27, 2015

Member

Soon @mzgol soon :-)

Member

markelog replied Jul 27, 2015

Soon @mzgol soon :-)

Please sign in to comment.