Skip to content
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

use events with better support #1211

Merged
merged 3 commits into from
Jun 28, 2017

Conversation

stevenleadbeater
Copy link
Contributor

mouseover and mouseout are supported across all major browsers. mouseenter is a little patchy in its support and can cause performance issues in large structures:

https://developer.mozilla.org/en/docs/Web/Events/mouseenter

mouseover and mouseout are supported across all major browsers. mouseenter is a little patchy in its support and can cause performance issues in large structures:

https://developer.mozilla.org/en/docs/Web/Events/mouseenter
@codecov-io
Copy link

codecov-io commented Nov 7, 2016

Codecov Report

Merging #1211 into development will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##           development    #1211   +/-   ##
============================================
  Coverage        87.16%   87.16%           
============================================
  Files               85       85           
  Lines             2267     2267           
  Branches           293      293           
============================================
  Hits              1976     1976           
  Misses             188      188           
  Partials           103      103
Impacted Files Coverage Δ
src/utils/triggers.ts 96.42% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ad2da54...24c782a. Read the comment docs.

@valorkin
Copy link
Member

valorkin commented Nov 8, 2016

hey, thanks for info and PR. I will merge it a bit later,
because I am in a middle of complete module restructure

@valorkin
Copy link
Member

@Betrozov I can merge now but I need your approval

@IlyaSurmay IlyaSurmay requested a review from valorkin June 27, 2017 15:20
@valorkin
Copy link
Member

Thanks for PR ;)

@valorkin valorkin merged commit 46419e3 into valor-software:development Jun 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants