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
Refactor NativeProxy on Android #4042
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.
Thanks @graszka22 for this refactor, it really makes things simpler and better-organized. I've left a few suggestions, let me know if they are doable or not.
android/src/fabric/java/com/swmansion/reanimated/NativeProxy.java
Outdated
Show resolved
Hide resolved
android/src/fabric/java/com/swmansion/reanimated/NativeProxy.java
Outdated
Show resolved
Hide resolved
android/src/fabric/java/com/swmansion/reanimated/NativeProxy.java
Outdated
Show resolved
Hide resolved
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.
Tested on Example and FabricExample (Android). Well done @graszka22!
<!-- 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 Refactor of `NativeProxy.java` to avoid some code duplication. ## Test plan Run Example app and FabricExample app. Make sure things like keyboard, sensors, layout animations still work.
Summary
Refactor of
NativeProxy.java
to avoid some code duplication.Test plan
Run Example app and FabricExample app. Make sure things like keyboard, sensors, layout animations still work.