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

Hover module experimental event API delay approach #15285

Closed
behzad888 opened this issue Apr 1, 2019 · 3 comments
Closed

Hover module experimental event API delay approach #15285

behzad888 opened this issue Apr 1, 2019 · 3 comments

Comments

@behzad888
Copy link
Contributor

behzad888 commented Apr 1, 2019

To implement delayHoverStart and delayHoverEnd there are many situations such as below:

First
When start delay timeout greater than or equal to the end delay timeout. no worries about this, because both of mouseover and mouseout events will be run perfectly.

Second
When start delay timeout less than end delay. In this situation, there are three options

  • We can clear start delay and JUST run end delay but we will lose mouseover event.
  • We can keep both of them but maybe mouseout event runs before mouseover
  • We can check by end >= start as always

I was wondering which one could be more useful.

@trueadm
Copy link
Contributor

trueadm commented Apr 1, 2019

Pinging @necolas as he mentioned something along these lines last week. We should consider these for the hover module and make a task on what we want to do going forward in #15257.

@necolas
Copy link
Contributor

necolas commented Apr 1, 2019

Thanks. The delays are only additive as in the React Native touchables.

We should close this and keep conversations in the umbrella issue

@behzad888
Copy link
Contributor Author

Ok then

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants