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

Remove support for TapEventPlugin #11689

Closed
gaearon opened this issue Nov 28, 2017 · 3 comments
Closed

Remove support for TapEventPlugin #11689

gaearon opened this issue Nov 28, 2017 · 3 comments
Milestone

Comments

@gaearon
Copy link
Collaborator

gaearon commented Nov 28, 2017

Some libraries relied on it so we kept it in 16, but it's impossible to support it forever given that we want to make changes to the event system. We should deprecate the injection it relies on and remove it in React 17. The migration path is to turn it into a userland <Tappable> component or something like that. Hopefully somebody using TapEventPlugin can work on this, as we don't actively use it.

@rafgraph
Copy link

What about using the React Interactive component? It has events for tap (without delay), long press, two finger tap, etc, and a touchActive state for styling touch interactions.
https://github.com/rafrex/react-interactive

@gaearon
Copy link
Collaborator Author

gaearon commented Nov 28, 2017

I think people would probably be looking for a very targeted API compatible replacement that is more or less exact port in terms of features. This component seems to serve a broader use case.

@gaearon
Copy link
Collaborator Author

gaearon commented Mar 24, 2021

We did that.

@gaearon gaearon closed this as completed Mar 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants