-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Screen reader fails to announce the expand/collapse state of the navigation button #15106
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
/azp run |
Azure Pipelines successfully started running 2 pipeline(s). |
|
||
// Raise UIA event if clients are listening | ||
if (UiaClientsAreListening()) { | ||
winrt::Microsoft::ReactNative::implementation::UpdateUiaProperty( |
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.
This would only notify Uia of the property change if the expand was triggered through Uia, not if it was triggered through the UI or some other means.
The property change notification should be triggered when the state changes on the UI object, not here.
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.
Got it
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.
@acoates-ms In the latest commit added expand/collapse state handling in CompositionViewComponentView.cpp. added UpdateUiaProperty call for UIA_ExpandCollapseExpandCollapseStateProperty Id, please review
/azp run |
Azure Pipelines successfully started running 2 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.
LGTM!
…gation button (microsoft#15106) * adding UIA event handler changes for navigator * Change files * adding ViewComponent chnages for accessibility * removing unnecessary change file * Remove vnext/fmt/packages.lock.json --------- Co-authored-by: Protik Biswas <[email protected]>
…gation button (microsoft#15106) * adding UIA event handler changes for navigator * Change files * adding ViewComponent chnages for accessibility * removing unnecessary change file * Remove vnext/fmt/packages.lock.json --------- Co-authored-by: Protik Biswas <[email protected]>
…gation button (microsoft#15106) * adding UIA event handler changes for navigator * Change files * adding ViewComponent chnages for accessibility * removing unnecessary change file * Remove vnext/fmt/packages.lock.json --------- Co-authored-by: Protik Biswas <[email protected]>
Description
Screen reader fails to announce the expand/collapse state of the navigation button
Type of Change_
Why
Screen reader fails to announce the expand/collapse state of the navigation button
Resolves [https://github.com/microsoft/react-native-gallery/issues/612]
What
Screen reader fails to announce the expand/collapse state of the navigation button
Added UIA event handling
Screenshots
After changes
Narrator.mp4
Testing
yarn install & yarn build passed
Launched playground-fabric app
Attached screenshot
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.
Microsoft Reviewers: Open in CodeFlow