Skip to content

Do not create m_childrenContainer when using a custom visual to mount children into#15877

Merged
acoates-ms merged 3 commits intomicrosoft:mainfrom
acoates-ms:noclipCustomMountChildren
Mar 31, 2026
Merged

Do not create m_childrenContainer when using a custom visual to mount children into#15877
acoates-ms merged 3 commits intomicrosoft:mainfrom
acoates-ms:noclipCustomMountChildren

Conversation

@acoates-ms
Copy link
Copy Markdown
Contributor

@acoates-ms acoates-ms commented Mar 30, 2026

Description

The implementation of overflow:"hidden" creates a new container visual, which would bypass a custom visual provided by SetVisualToMountChildrenIntoHandler. It would also crash trying to remove existing visuals from Visual(), when they are actually in VisualToMountChildrenInto().

Resolves #15875

Microsoft Reviewers: Open in CodeFlow

@acoates-ms acoates-ms requested review from a team as code owners March 30, 2026 22:55
@acoates-ms acoates-ms merged commit 0c3f2fb into microsoft:main Mar 31, 2026
32 checks passed
@acoates-ms acoates-ms deleted the noclipCustomMountChildren branch March 31, 2026 16:21
acoates-ms added a commit to acoates-ms/react-native-windows that referenced this pull request Mar 31, 2026
… children into (microsoft#15877)

* Do not create m_childrenContainer when using a custom visual to mount children into

* Change files

* fix
acoates-ms added a commit that referenced this pull request Mar 31, 2026
…o mount children into (#15884)

* Do not create m_childrenContainer when using a custom visual to mount children into (#15877)

* Do not create m_childrenContainer when using a custom visual to mount children into

* Change files

* fix

* fix change file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Crash when applying a clip to a component with custom VisualToMountChildrenInto

2 participants