New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix relatedTarget in mouseenter and mouseleave #2697

Merged
merged 1 commit into from Feb 24, 2015

Conversation

Projects
None yet
3 participants
@SergioCrisostomo
Member

SergioCrisostomo commented Feb 1, 2015

basically using @arian's suggestion

fixes #2348

@arian

This comment has been minimized.

Show comment
Hide comment
@arian

arian Feb 2, 2015

Member

Is there a way to test this?

Member

arian commented Feb 2, 2015

Is there a way to test this?

@SergioCrisostomo

This comment has been minimized.

Show comment
Hide comment
@SergioCrisostomo

SergioCrisostomo Feb 2, 2015

Member

I tried to use Syn.js to reproduce this but didn't seem to work.
So I made a jsFiddle that works fine in FF with this patch, but has to be a human testing.

Member

SergioCrisostomo commented Feb 2, 2015

I tried to use Syn.js to reproduce this but didn't seem to work.
So I made a jsFiddle that works fine in FF with this patch, but has to be a human testing.

@SergioCrisostomo

This comment has been minimized.

Show comment
Hide comment
@SergioCrisostomo

SergioCrisostomo Feb 17, 2015

Member

@arian updated with Spec and rebased

Member

SergioCrisostomo commented Feb 17, 2015

@arian updated with Spec and rebased

@SergioCrisostomo SergioCrisostomo added this to the 1.5.2 milestone Feb 22, 2015

@SergioCrisostomo

This comment has been minimized.

Show comment
Hide comment
@SergioCrisostomo

SergioCrisostomo Feb 24, 2015

Member

Can we merge this?

Member

SergioCrisostomo commented Feb 24, 2015

Can we merge this?

ibolmo added a commit that referenced this pull request Feb 24, 2015

Merge pull request #2697 from SergioCrisostomo/fix-2348
fix relatedTarget in mouseenter and mouseleave

@ibolmo ibolmo merged commit f91048f into mootools:master Feb 24, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@ibolmo

This comment has been minimized.

Show comment
Hide comment
@ibolmo

ibolmo Feb 24, 2015

Member

LGTM

Member

ibolmo commented Feb 24, 2015

LGTM

@SergioCrisostomo SergioCrisostomo deleted the SergioCrisostomo:fix-2348 branch Feb 24, 2015

@mootools mootools deleted a comment Mar 14, 2018

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