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

onTouchTap doesn't work with React 15.6.0 #7129

Closed
bogdansoare opened this issue Jun 14, 2017 · 8 comments
Closed

onTouchTap doesn't work with React 15.6.0 #7129

bogdansoare opened this issue Jun 14, 2017 · 8 comments
Labels
bug 🐛 Something doesn't work

Comments

@bogdansoare
Copy link

The onTouchTap from Buttons isn't working with React 15.6.0 https://facebook.github.io/react/warnings/unknown-prop.html
I think it's finally time to move to the standard onClick and remove tap event plugin

@oliviertassinari
Copy link
Member

think it's finally time to move to the standard onClick and remove tap event plugin

That's what we did on the next branch.

@oliviertassinari oliviertassinari added the bug 🐛 Something doesn't work label Jun 14, 2017
@lantenac
Copy link

any idea of the next release date that includes that change ? thanks in advance.

@oliviertassinari
Copy link
Member

That might need to be fixed at the react-tap-event-plugin level. I'm not sure we can do anything here.

@radix
Copy link

radix commented Jun 21, 2017

Hi, just to confirm: is there absolutely no way to use material-ui with recent versions of react?

I'm trying it out for the first time and I tried setting up react-tap-event-plugin as instructed in the README, but I'm still getting the warning in my console about unknown props, and the IconMenu that I put in my app isn't responding to clicks.

@radix
Copy link

radix commented Jun 21, 2017

I filed a ticket on react-tap-event-plugin to see if there's a workaround there: zilverline/react-tap-event-plugin#103

@oliviertassinari
Copy link
Member

I'm considering the option to move away from onTouchTap for onClick in the v0.x releases in order to fix that issue. That's going to be an important breaking change. Hopefully, the upgrade path should be pretty easy. Also, I haven't seen any limitation with onClick yet.

@oliviertassinari
Copy link
Member

oliviertassinari commented Jul 11, 2017

I have been setting up a create-react-app project (that use 15.6.1 internally). I had no issue, the touch callbacks are still working. I'm assuming it's another limitation of react-tap-event-plugin. If it's a blocker for you, consider using the v1-alpha branch. We are even supporting the 16.0.0-alpha.13 release of React on that branch.

@CharlieIGG
Copy link

I have been setting up a create-react-app project (that use 15.6.1 internally). I had no issue, the touch callbacks are still working. I'm assuming it's another limitation of react-tap-event-plugin. If it's a blocker for you, consider using the v1-alpha branch. We are even supporting the 16.0.0-alpha.13 release of React on that branch.

Can you elaborate a little bit here @oliviertassinari ?
How should I go about doing that upgrade in order to fix this?

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work
Projects
None yet
Development

No branches or pull requests

5 participants