-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
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
React 15.0.2 "Unknown props onTouchTap" warnings #4670
Comments
I would like to send a PR but right now I don't know how to test the changes and if those changes fixes the warning ? |
Using Yeoman Webpack and
From the simple:
Packages.json:
|
Fixed by adding: // Needed for onTouchTap
import injectTapEventPlugin from 'react-tap-event-plugin';
injectTapEventPlugin(); To the component. |
I'd like to add:
Only appears in Update: Nevermind, that was with react |
I call Versions:
Some Warnings:
|
@igl try to call |
@ixrock: calling |
@Ghirigoro the same issue =\
regardless of using/not using injectTapEventPlugin() before/after render. Any ideas, guys? |
Added the suggestion by @tavurth and I got rid of the error.
I settled on placing it in the ReactDOM.render file
|
Seeing same error on CardTitle with prop titleStyle...
Code fragment: Material UI version: 0.15.2 Thanks so much. |
Hi there, Not sure if mine is related but I get this error when using
|
The same for me :
And i add : import injectTapEventPlugin from 'react-tap-event-plugin';
injectTapEventPlugin(); And it does NOT work 👎 |
I have the same warning. After some debugging I have discovered that it is raised from this code:
and
this is my package.json
|
adding |
I fixed this problem by switching from using the UMD version of react to webpack-ing everything. I suspect this fixes it because some combo of react-tap-event-plugin and material-ui needs to reach into the private internals of React, and it can't do this from the UMD version. Sigh... |
I have same warning for Overlay
Also I've read here facebook/react#436 that react-tap-event-plugin possibly may be removed since react 15. Need to investigate. |
I'm closing this issue as root issue has been identified by @tavurth. Thanks! We rely on the Going forward, we plan to remove this dependency in the |
I was using AppBar and here is the code I am using to fix this as per comments above:
|
For those like me who found this issue, applied the fixes, and had the problem persist: The suggested approach of running |
Could anyone tell me what injectTapEventPlugin() is here for please? |
@jakewins: I hade the same issue with browserify. The problem was that I am building a vendor.js and one js file with my whole client code. I found out, that the react-event-tap-plugin have to be bundled within the js file where your react lib is in. This approach fixed my problem with browserify. |
Same issue. @vaspoz Seems to be related to the react-tap-event-plugin going up to ^2.0.0 and react hitting 15.4.0. I am using UMD build and calling injectTapEventPlugin before ReactDOM.render. Using a non UMD build is unfortunately not an option. Does anyone have a working UMD version combination of react + react-tap-event-plugin + material-ui? @oliviertassinari Would it be stable enough to use the next branch which has the more native onClick? I do not care if many things are broken, as long as its semi usable. |
@vans163 Here is a 1 month-year-old version of the |
Just in case it helps someone until the new release is ready: I ran into this issue, but once I moved the import injectTapEventPlugin from 'react-tap-event-plugin';
injectTapEventPlugin(); lines to the component file where I apply I'm using an ejected webpack from |
@rdnewman This works!! Thank you so much! |
thanks all... I had this warning in tests only, and then realised that |
@rdnewman , thank you! your method works. I put it in my index.js file where it contained the ReactDOM.render method file and it also worked that way. 👍 |
I had this come up using |
hope to help someone who use typescript
it was solved by changing import style |
Thank god, these days I just have to install the react-tap-event-plugin version 2.0.1 and react 15.4.2, then I went to the file where I render app, and just include these two lines: import injectTapEventPlugin from 'react-tap-event-plugin'; And that's it 👍 |
Frustratingly vague issue. Came back after I did some code reorganization in a SSR scenario |
For "react": "15.4.2", Use "react-tap-event-plugin": "^2.0.0" or react-tap-event-plugin": "^2.0.1" and add this
|
Importing and adding injectTapEventPlugin() solved my issue.
|
Im using Next.JS This solution do not work, i have a file tap_events.js it is import each page. My file tap_events.js: |
Special hack for onTouchTap warning see mui/material-ui#4670
Special hack for onTouchTap warning see mui/material-ui#4670
Even though last release fixed a couple of issues. I am still seeing this two warnings
The text was updated successfully, but these errors were encountered: