-
-
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
feat: Support for multiple surfaces on Fabric #6647
feat: Support for multiple surfaces on Fabric #6647
Conversation
packages/react-native-reanimated/Common/cpp/reanimated/NativeModules/NativeReanimatedModule.cpp
Outdated
Show resolved
Hide resolved
packages/react-native-reanimated/Common/cpp/reanimated/NativeModules/NativeReanimatedModule.cpp
Outdated
Show resolved
Hide resolved
packages/react-native-reanimated/Common/cpp/reanimated/NativeModules/NativeReanimatedModule.cpp
Show resolved
Hide resolved
Please merge latest main to fix macOS CI, looks like tvOS is still broken on other PRs |
I have tested it, and it crashes sometimes for me after a reload. What I did was I opened the Bokeh Example and pressed R to reload the app - on Android, it stops the animation, but on iOS, it crashes. Could you verify if this happened before as well? |
We established that this issue is not related to changes from this PR. |
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.
I've tested it and it seems to be ok 👍
Summary
This PR introduces handling multiple surfaces on Fabric.
Test plan
🚀