-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Initial support for UIA and tab navigation for a child Island #14305
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
acoates-ms
reviewed
Jan 24, 2025
vnext/Microsoft.ReactNative/Fabric/Composition/ContentIslandComponentView.cpp
Outdated
Show resolved
Hide resolved
acoates-ms
reviewed
Jan 24, 2025
vnext/Microsoft.ReactNative/Fabric/Composition/ContentIslandComponentView.cpp
Outdated
Show resolved
Hide resolved
acoates-ms
reviewed
Jan 24, 2025
vnext/Microsoft.ReactNative/Fabric/Composition/CompositionDynamicAutomationProvider.cpp
Outdated
Show resolved
Hide resolved
…islandcomponentview-uia
/azp run Secure PR |
Azure Pipelines successfully started running 1 pipeline(s). |
acoates-ms
approved these changes
Jan 29, 2025
acoates-ms
pushed a commit
to acoates-ms/react-native-windows
that referenced
this pull request
Jan 29, 2025
…oft#14305) * Basic support for stitching the UIA tree for a ContentIslandComponentView's child * Updated comment * Change files * Support shift+tab, and move Automation event handlers to ContentIslandComponentView
acoates-ms
added a commit
that referenced
this pull request
Jan 31, 2025
… factor (#14318) * Fix crash when currently focused element gets marked as enableFocusRing=false (#14306) * Fix crash when currently focused element gets marked as enableFocusRing=false * Change files * [Fabric] Use PopupWindowSiteBridge for Modal when USE_EXPERIMENTAL_WINUI3 is true (#14284) * add modal implementation with PopupWindowSiteBridge * Change files * add conditional * feedback * Initial support for UIA and tab navigation for a child Island (#14305) * Basic support for stitching the UIA tree for a ContentIslandComponentView's child * Updated comment * Change files * Support shift+tab, and move Automation event handlers to ContentIslandComponentView * Allow portals to have independent layout constraints and scale factor (#14315) * Allow portals to have independent layout constraints and scale factor * format * change files * fix * Round Focus visuals by default, fix nudge rendering (#14312) * Round Focus visuals by default, fix nudge rendering * Change files * change files * fix * fix * fix --------- Co-authored-by: Tatiana Kapos <[email protected]> Co-authored-by: JesseCol <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This change stiches together the UIA tree between a React Native content and a child island it might be hosting. It also provides initial support for tab navigation into and out of a child island.
Not working yet:
Type of Change
Erase all that don't apply.
Why
This is progress toward the goal of RNW being able to properly host a XamlIsland so a RNW fabric app can use Xaml controls.
Screenshots
When UseWinUI3Experimental, we can traverse the UIA tree downward into Xaml and upward back to RNW:
Testing
I've tested this manually using the Playground-Composition app with UseWinUI3Experimental set to true.
Changelog
Should this change be included in the release notes: no
Microsoft Reviewers: Open in CodeFlow