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

Add ability to override UpdateLayoutMetrics and VisualToMountChildrenInto #14182

Merged
merged 5 commits into from
Dec 9, 2024

Conversation

acoates-ms
Copy link
Contributor

@acoates-ms acoates-ms commented Dec 9, 2024

Description

Adding ability for custom native components to provide a custom UpdateLayoutMetrics method, and to override VisualToMountChildrenInto.

I also did some cleanup of how override methods work to reduce the size of ComponentViews in memory.

Fixes #14172 and #14174.

Microsoft Reviewers: Open in CodeFlow

@acoates-ms acoates-ms requested review from a team as code owners December 9, 2024 18:36
@microsoft-github-policy-service microsoft-github-policy-service bot added Area: Component Views New Architecture Broad category for issues that apply to the RN "new" architecture of Turbo Modules + Fabric labels Dec 9, 2024
@acoates-ms acoates-ms merged commit 91ad48e into microsoft:main Dec 9, 2024
59 checks passed
@acoates-ms acoates-ms deleted the visualtoaddto branch December 9, 2024 21:45
acoates-ms added a commit to acoates-ms/react-native-windows that referenced this pull request Dec 9, 2024
…Into (microsoft#14182)

* Add ability to override UpdateLayoutMetrics and VisualToMountChildrenInto

* Change files

* format

* fix
acoates-ms added a commit to acoates-ms/react-native-windows that referenced this pull request Dec 9, 2024
…Into (microsoft#14182)

* Add ability to override UpdateLayoutMetrics and VisualToMountChildrenInto

* Change files

* format

* fix
acoates-ms added a commit to acoates-ms/react-native-windows that referenced this pull request Dec 9, 2024
…Into (microsoft#14182)

* Add ability to override UpdateLayoutMetrics and VisualToMountChildrenInto

* Change files

* format

* fix
acoates-ms added a commit that referenced this pull request Dec 10, 2024
…hildrenInto (#14187)

* Add linter exception to generated headers for Office builds (#14157)

* Add linter exception to generated headers for Office builds

* Add ability to override UpdateLayoutMetrics and VisualToMountChildrenInto (#14182)

* Add ability to override UpdateLayoutMetrics and VisualToMountChildrenInto

* Change files

* format

* fix

* fix

---------

Co-authored-by: Sarang Gokhale <[email protected]>
acoates-ms added a commit that referenced this pull request Dec 10, 2024
…hildrenInto (#14188)

* Add linter exception to generated headers for Office builds (#14157)

* Add linter exception to generated headers for Office builds

* Add ability to override UpdateLayoutMetrics and VisualToMountChildrenInto (#14182)

* Add ability to override UpdateLayoutMetrics and VisualToMountChildrenInto

* Change files

* format

* fix

* change files

* fix

---------

Co-authored-by: Sarang Gokhale <[email protected]>
acoates-ms added a commit that referenced this pull request Dec 10, 2024
…hildrenInto (#14186)

* Add linter exception to generated headers for Office builds (#14157)

* Add linter exception to generated headers for Office builds

* Add ability to override UpdateLayoutMetrics and VisualToMountChildrenInto (#14182)

* Add ability to override UpdateLayoutMetrics and VisualToMountChildrenInto

* Change files

* format

* fix

* fix

---------

Co-authored-by: Sarang Gokhale <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Component Views New Architecture Broad category for issues that apply to the RN "new" architecture of Turbo Modules + Fabric
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Custom components should be able to override VisualToMountChildrenInto
2 participants