Skip to content

Commit

Permalink
Event: Add a note about a mouseenter bug in Chrome
Browse files Browse the repository at this point in the history 
Refs https://code.google.com/p/chromium/issues/detail?id=333868
  • Loading branch information
@mgol
mgol committed Jan 25, 2015
1 parent e905dcd commit a5e1c9b
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions src/event.js
View file
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -708,6 +708,10 @@ jQuery.Event.prototype = {
// Do the same for pointerenter/pointerleave and pointerover/pointerout // Do the same for pointerenter/pointerleave and pointerover/pointerout
// Support: Safari<7.0 // Support: Safari<7.0
// Safari doesn't support mouseenter/mouseleave at all. // Safari doesn't support mouseenter/mouseleave at all.
// Support: Chrome 40+

This comment has been minimized.

Sign in to view

Copy link
@markelog

markelog Feb 19, 2015

Member

In the ticket mentioned in the commit message, it says, bug is, at least, reproducible in 34 version.

This comment has been minimized.

Sign in to view

Copy link
@markelog

markelog Feb 21, 2015

Member

@mzgol ?

This comment has been minimized.

Sign in to view

Copy link
@mgol

mgol Feb 21, 2015

Author Member

It doesn't really matter. I'm fine with preserving info about compat with older browsers where we already have it but if it already says 40+, I don't think it's worth checking how far in the past that goes. Maybe it's reproducible in 33? Or earlier? I don't really care, it's not important, esp. wrt. Chrome.

cc @gibson042

This comment has been minimized.

Sign in to view

Copy link
@markelog

markelog Feb 21, 2015

Member

I'd say it's not important, but it's matter. I don't think we should investigate it but put there what we know

This comment has been minimized.

Sign in to view

Copy link
@gibson042

gibson042 Feb 23, 2015

Member

I agree, and this case should be covered by Chrome <=34 - 40 (or different range as confirmed): jquery/contribute.jquery.org#95 (comment)
// 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
jQuery.each({ jQuery.each({
mouseenter: "mouseover", mouseenter: "mouseover",
mouseleave: "mouseout", mouseleave: "mouseout",
Expand Down

0 comments on commit a5e1c9b

Please sign in to comment.