-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
doc: added important tip when using android device #4357
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @Chinedu19, thanks for the PR, I've left some suggestions in the comments.
Co-authored-by: Tomek Zawadzki <tomekzawadzki98@gmail.com>
Co-authored-by: Tomek Zawadzki <tomekzawadzki98@gmail.com>
I've implemented the changes you requested |
Co-authored-by: Krzysztof Piaskowy <krzysztof.piaskowy@swmansion.com>
@piaskowyk suggestion implemented |
@Chinedu19 Thanks, can you please propagate this change to |
Done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👏
…4357) <!-- Thanks for submitting a pull request! We appreciate you spending the time to work on these changes. Please follow the template so that the reviewers can easily understand what the code changes affect. --> ## Summary <!-- Explain the motivation for this PR. Include "Fixes #<number>" if applicable. --> This pull request aims to improve the documentation on [Events Fundamentals](https://docs.swmansion.com/react-native-reanimated/docs/fundamentals/events/) by adding instructions for how to set up event handlers on Android devices. Many developers, including myself, have had issues practicing the example on Android devices, which can be confusing for those who are not familiar with *react-native-gesture-handler*. This addition would make it easier for developers to practice using event handlers with *react-native-reanimated* on Android devices directly from the docs and help avoid spending time researching why it isn't working. ## Changes Made This pull request adds a new section to the Events Fundamentals documentation that explains how to enable event handlers on Android devices. It includes simple instructions and code example to help developers enable the event handlers. ## Test plan <!-- Provide a minimal but complete code snippet that can be used to test out this change along with instructions how to run it and a description of the expected behavior. --> As this is a documentation update, tests are not applicable. However, reviewers can follow the instructions provided in the new section and verify that the event handlers are working correctly on Android devices. --------- Co-authored-by: Tomek Zawadzki <tomekzawadzki98@gmail.com> Co-authored-by: Krzysztof Piaskowy <krzysztof.piaskowy@swmansion.com>
Summary
This pull request aims to improve the documentation on Events Fundamentals by adding instructions for how to set up event handlers on Android devices. Many developers, including myself, have had issues practicing the example on Android devices, which can be confusing for those who are not familiar with react-native-gesture-handler.
This addition would make it easier for developers to practice using event handlers with react-native-reanimated on Android devices directly from the docs and help avoid spending time researching why it isn't working.
Changes Made
This pull request adds a new section to the Events Fundamentals documentation that explains how to enable event handlers on Android devices. It includes simple instructions and code example to help developers enable the event handlers.
Test plan
As this is a documentation update, tests are not applicable. However, reviewers can follow the instructions provided in the new section and verify that the event handlers are working correctly on Android devices.