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

Discussion: React 17+ might remove the internal this depends on #112

Open
gaearon opened this issue Nov 28, 2017 · 0 comments
Open

Discussion: React 17+ might remove the internal this depends on #112

gaearon opened this issue Nov 28, 2017 · 0 comments

Comments

@gaearon
Copy link

gaearon commented Nov 28, 2017

Hi!

Just wanted to leave a note that the internals this plugin depends on will likely be removed in React 17: facebook#11689. It would be great to start migrating the ecosystem that has come to depend on this plugin.

Now is a good time to start thinking about alternatives. I would probably suggest creating a <Tapable> component that uses only public API as a replacement. We could maybe do this ourselves, but it's challenging since we don't actively use TapEvenPlugin and can’t evaluate if the replacement is working well.

Would somebody who uses this be interested in working on it?

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

1 participant