-
Notifications
You must be signed in to change notification settings - Fork 65
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
The event "mousedown" is not fired on mobile properly #11
Comments
I am experiencing the same issue on iPad. I made a workaround for the issue by doing something like this in my component:
where |
So, I had this conversation with @aarongloege when the change was made to
@ljharb @garrettberg do you have any thoughts on this? |
Unless react changed its behavior (which would be a breaking change we’d have heard about) i would expect click to be sufficient to cover touch devices. |
@ljharb we're no longer using click for the reasons I've cited above. We are using mousedown because the spec suggests that that should also trigger on touch devices. |
@majapw @ljharb I'm having the same issue with the pdp bookit calendar. Here's the related jira ticket. Should I use the workaround posted above to fix this? |
I'm using 'react-dates' library and on my iphone5 handler
onOutsideClick
is not fired properly.It's fired when I click on the button, but not if I click on an empty space background.
I've added 'touchstart' event for my component and now it's fine.
Can we extend current event listener with
touchstart
?The text was updated successfully, but these errors were encountered: