Skip to content

Flip the order of component descriptor registration#53953

Closed
rozele wants to merge 2 commits into
facebook:mainfrom
rozele:export-D83273376
Closed

Flip the order of component descriptor registration#53953
rozele wants to merge 2 commits into
facebook:mainfrom
rozele:export-D83273376

Conversation

@rozele
Copy link
Copy Markdown
Contributor

@rozele rozele commented Sep 25, 2025

Summary:
For TurboModule registration, we register custom / app-specific TurboModules before core TurboModules to ensure that app-specific module overrides are preferred over core / default behaviors.

This provides a similar behavior for Fabric component descriptor registration.

Differential Revision: D83273376

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Sep 25, 2025
@facebook-github-bot
Copy link
Copy Markdown
Contributor

@rozele has exported this pull request. If you are a Meta employee, you can view the originating diff in D83273376.

Summary:

In React Native Android, you can technically override core components like ScrollView to add additional props and behaviors, but doing so in the new architecture requires ensuring that these props are tracked in the C++ props object.

This change unseals ScrollViewProps so they can be overridden in (rather rare and generally unsupported) cases where you need to customize ScrollView behaviors for a specific app.

Reviewed By: Abbondanzo

Differential Revision: D83268457
Summary:

For TurboModule registration, we register custom / app-specific TurboModules before core TurboModules to ensure that app-specific module overrides are preferred over core / default behaviors.

This provides a similar behavior for Fabric component descriptor registration.

Differential Revision: D83273376
@facebook-github-bot
Copy link
Copy Markdown
Contributor

@rozele has exported this pull request. If you are a Meta employee, you can view the originating diff in D83273376.

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Sep 26, 2025
@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request has been merged in b9dcc1b.

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. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants