-
Notifications
You must be signed in to change notification settings - Fork 1.2k
[Fabric] Added UIA Event Triggers for accessibilityAnnotation Property Changes #15039
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
Conversation
change/react-native-windows-8aaaff79-a2a7-4bd6-af5f-ecee696abe44.json
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add the requested change.
/azp run PR |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run PR |
Azure Pipelines successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds UIA (UI Automation) event triggers for the accessibilityAnnotation property in React Native Windows. When accessibilityAnnotation property values change, the system will now properly raise UIA change events to notify accessibility tools.
Key changes:
- Added new overloaded UpdateUiaProperty function for long values
- Implemented UpdateUiaPropertiesForAnnotation function to handle all annotation property updates
- Integrated annotation property change notifications into the accessibility update workflow
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
File | Description |
---|---|
UiaHelpers.h | Added function declarations for long value UIA updates and annotation property handling |
UiaHelpers.cpp | Implemented UIA property update functions for annotation properties |
CompositionViewComponentView.cpp | Integrated annotation property change notifications into accessibility update flow |
change/react-native-windows-a648cb52-86ff-47b1-b2be-04d168a7f110.json | Added change file for version tracking |
/azp run PR |
Azure Pipelines successfully started running 1 pipeline(s). |
@gsaran25 Please don't remove the changelog section of PR description and add values appropriately. |
… Changes (#15039) * Added UIA Event Triggers for accessibilityAnnotation Property Changes * Added UIA Event Triggers for accessibilityAnnotation Property Changes * wrapped to single call - UpdateUIAPropertiesForAnnotation * Annotation Props UIA Event method call * removed yarn change file * Review Changes done * UpdateUiaPropertyForAnnotation method removed. * UpdateUiaProperty long method param added. * yarn format changes * AccessibilityAnnotation default valued assigned. --------- Co-authored-by: Vineeth <[email protected]>
Description
The accessibilityAnnotation property was introduced in RNW. However, we did not implement logic to raise a UIA change event when its values are updated.
Type of Change
Why
The accessibilityAnnotation property was introduced in RNW. However, we did not implement logic to raise a UIA change event when its values are updated.
Resolves [#14999]
What
Added events for the below properties.
UIA_AnnotationAnnotationTypeIdPropertyId
UIA_AnnotationAnnotationTypeNamePropertyId
UIA_AnnotationAuthorPropertyId
UIA_AnnotationDateTimePropertyId
Screenshots
Developed in mac, Could not able to test.
Testing
Developed in mac, Could not able to test.
Changelog
Should this change be included in the release notes: yes
Add a brief summary of the change to use in the release notes for the next release.
Added UIA update event for change in annotation properties.
Microsoft Reviewers: Open in CodeFlow