Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Navigation perf profiling #1658

Draft
wants to merge 1 commit into
base: development
Choose a base branch
from
Draft

Conversation

savelichalex
Copy link
Contributor

No description provided.

@savelichalex savelichalex requested a review from SAnatoliiS July 29, 2022 15:42
@savelichalex savelichalex marked this pull request as draft July 29, 2022 15:42
@@ -35,7 +37,11 @@ protected List<ReactPackage> getPackages() {
// Packages that cannot be autolinked yet can be added manually here, for example:
// packages.add(new MyReactNativePackage());

return packages;
packages = PatchedReanimatedPackage.patchPackages(packages);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It might be a good idea to apply it in Surf anyway as it completely remove patched UIManager from reanimated that they use for layout animations and it has useless overhead

@sertony sertony force-pushed the development branch 3 times, most recently from a7c8130 to 90c9e60 Compare November 28, 2022 14:00
@sertony sertony force-pushed the development branch 3 times, most recently from 6c5f6ae to 5833230 Compare April 19, 2023 13:15
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant