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

Expose private node management methods in UIManager #33688

Closed
wants to merge 5 commits into from

Conversation

ShikaSD
Copy link
Contributor

@ShikaSD ShikaSD commented Apr 21, 2022

Summary:
These methods used to be public in the legacy implementation, and hiding them significantly reduces amount of customization available to other clients outside Fabric core.

Changelog: [Internal] Allow external callers to call UIManager methods

Differential Revision: D35818114

Andrei Shikov added 5 commits April 21, 2022 09:15
Summary:
Minimal set of changes to intercept events in external modules. Current intended use-case is for Reanimated to handle events for the Animated properties.

Changelog: [Added] Add listeners to allow intercepting events in C++ core.

Differential Revision: D35312534

fbshipit-source-id: 26e505942d7fe34eb41c161e01d8b8af2d0b2d3a
Differential Revision: D35313399

fbshipit-source-id: 3c0d96c6e3ea035aee3d3e05d7570d1746930790
Summary:
Exposes event listener through iOS scheduler wrapper (`RCTScheduler`) and exposes scheduler itself through `RCTSurfacePresenter`.

Changelog: [Changed] Exposed `RCTScheduler` to allow setting event listeners.

Differential Revision: D35313398

fbshipit-source-id: 35143cdbbe74e8c0ba15abcf27ecee33593f4d45
Summary:
Pull Request resolved: facebook#33545

Exposes `UIManager` instance for access from third-party modules.

Changelog: [Changed] Exposes `UIManager` instance for third-party access.

Differential Revision: D35314058

fbshipit-source-id: b80c5520185f272f66788c3392eccecbb2eb1def
Summary:
These methods used to be public in the legacy implementation, and hiding them significantly reduces amount of customization available to other clients outside Fabric core.

Changelog: [Internal] Allow external callers to call UIManager methods

Differential Revision: D35818114

fbshipit-source-id: ba2fb3ba20f95259bb9f1ed9b30d0969cbfe698a
@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner fb-exported labels Apr 21, 2022
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D35818114

@analysis-bot
Copy link

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 7,775,958 +3,043
android hermes armeabi-v7a 7,181,683 +2,725
android hermes x86 8,085,263 +3,330
android hermes x86_64 8,065,457 +3,622
android jsc arm64-v8a 9,649,959 +3,050
android jsc armeabi-v7a 8,424,079 +2,730
android jsc x86 9,599,453 +3,321
android jsc x86_64 10,197,049 +3,619

Base commit: aeac6ab
Branch: main

tomekzaw added a commit to software-mansion/react-native-reanimated that referenced this pull request Apr 21, 2022
@analysis-bot
Copy link

Platform Engine Arch Size (bytes) Diff
ios - universal n/a --

Base commit: aeac6ab
Branch: main

@react-native-bot
Copy link
Collaborator

This pull request was successfully merged by @cortinico in 59385e8.

When will my fix make it into a release? | Upcoming Releases

@react-native-bot react-native-bot added the Merged This PR has been merged. label Apr 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. p: Facebook Partner: Facebook Partner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants