Skip to content

Conversation

protikbiswas100
Copy link
Contributor

@protikbiswas100 protikbiswas100 commented Sep 3, 2025

Description

Screen reader fails to announce the expand/collapse state of the navigation button

Type of Change_

  • New feature (non-breaking change which adds functionality)

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

@protikbiswas100 protikbiswas100 requested a review from a team as a code owner September 3, 2025 09:36
@protikbiswas100
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines successfully started running 2 pipeline(s).


// Raise UIA event if clients are listening
if (UiaClientsAreListening()) {
winrt::Microsoft::ReactNative::implementation::UpdateUiaProperty(
Copy link
Contributor

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Got it

Copy link
Contributor Author

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

@protikbiswas100 protikbiswas100 self-assigned this Sep 5, 2025
@protikbiswas100
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines successfully started running 2 pipeline(s).

Copy link
Contributor

@vineethkuttan vineethkuttan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@protikbiswas100 protikbiswas100 removed the request for review from anupriya13 September 8, 2025 15:02
@protikbiswas100 protikbiswas100 merged commit 42d0d8c into microsoft:main Sep 8, 2025
75 checks passed
protikbiswas100 added a commit to protikbiswas100/react-native-windows that referenced this pull request Sep 9, 2025
…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]>
protikbiswas100 added a commit to protikbiswas100/react-native-windows that referenced this pull request Sep 9, 2025
…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]>
acoates-ms pushed a commit to acoates-ms/react-native-windows that referenced this pull request Sep 11, 2025
…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]>
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.

3 participants