Implement accessibilityLevel for Fabric #14593
Merged
Conversation
This file contains hidden or 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
There was a problem hiding this comment.
Pull Request Overview
This pull request adds support for the new accessibilityLevel prop for Fabric, enabling enhanced UI automation properties for accessibility.
- Added the new accessibilityLevel property to view props.
- Updated construction, property setting, and UI automation providers to handle the new level.
- Modified the RNTesterApp to include and display the new accessibilityLevel.
Reviewed Changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| HostPlatformViewProps.h | Added the accessibilityLevel property to the view props. |
| HostPlatformViewProps.cpp | Initialized and set the accessibilityLevel property during props construction and via the prop setter. |
| CompositionViewComponentView.cpp | Updated UI Automation properties to include the new accessibilityLevel. |
| CompositionDynamicAutomationProvider.cpp | Extended the property provider to return the accessibilityLevel value for UI automation. |
| RNTesterApp-Fabric.cpp | Extended the test app to retrieve and verify the accessibilityLevel property. |
Files not reviewed (1)
- change/react-native-windows-5aa167ea-3ec8-4b11-87ed-14f728f2eec6.json: Language not supported
|
Please update the "What", "Testing", and "Changelog" sections of the PR to include relevant information, once you're ready for the PR to be reviewed. |
vineethkuttan
added
enhancement
Area: Accessibility
Area: Fabric
...eact-native-windows/tester/src/js/examples-win/Accessibility/AccessibilityExampleWindows.tsx
Outdated
Show resolved
Hide resolved
vineethkuttan
force-pushed
the
impAccessibilityLevel
branch
from
56739f6 to
9f34321
Compare
chiaramooney
approved these changes
Apr 23, 2025
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
see #14550 for the behavior of this prop.
Type of Change
Why
Add Support for accessibilityLevel Prop
Resolves [#14550 , #14551]
What
accessibilityLevel/ aria-levelas UIA propertylevelaccessibilityLevel/ aria-levelvalueScreenshots
The below screenshots will show to difference using the Accessibility Insights for Windows tool
Testing
If you added tests that prove your changes are effective or that your feature works, add a few sentences here detailing the added test scenarios.
Tested using Playground and RNTester fabric App . Also added test cases to ensure the changes will work.
Changelog
Should this change be included in the release notes: yes
Add Support for
accessibilityLevelPropMicrosoft Reviewers: Open in CodeFlow