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

[react-events] Rename hook exports #16533

Merged
merged 1 commit into from
Aug 22, 2019

Conversation

necolas
Copy link
Contributor

@necolas necolas commented Aug 21, 2019

For example, 'useHoverResponder' becomes 'useHover'

@sizebot
Copy link

sizebot commented Aug 21, 2019

No significant bundle size changes to report.

Generated by 🚫 dangerJS

@necolas necolas force-pushed the react-events/renaming branch from a56ef65 to 0496d99 Compare August 21, 2019 23:25
@trueadm
Copy link
Contributor

trueadm commented Aug 22, 2019

You missed the RN Press responder too.

@necolas
Copy link
Contributor Author

necolas commented Aug 22, 2019

Maybe we should remove the RN responder for now

For example, 'useHoverResponder' becomes 'useHover'
@necolas necolas force-pushed the react-events/renaming branch from 0496d99 to df5dfe4 Compare August 22, 2019 20:22
@necolas necolas merged commit 474b650 into facebook:master Aug 22, 2019
@necolas necolas deleted the react-events/renaming branch December 20, 2019 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants