Skip to content

Conversation

@ritoban23
Copy link

@ritoban23 ritoban23 commented Dec 3, 2025

Description

Fixes the UIA_LiveSettingPropertyId event to use the correct VT_I4 datatype instead of VT_BSTR for accessibility live region updates in Fabric. This ensures compliance with UIA requirements and improves accessibility event handling.

Type of Change

Erase all that don't apply.

  • Bug fix (non-breaking change which fixes an issue)

Why

The motivation for this change is to resolve an issue where the UIA_LiveSettingPropertyId event was being raised with the wrong datatype (VT_BSTR instead of VT_I4). This caused accessibility tools to misinterpret the live region setting, impacting screen reader behavior and overall accessibility compliance.

Resolves #15050

What

  • Updated CompositionViewComponentView.cpp to use the GetLiveSetting() conversion function for UIA_LiveSettingPropertyId.
  • The event now passes the correct integer value for live region settings ("polite", "assertive", "off") instead of a string.

Screenshots

N/A (no UI changes)

Testing

  • Verified that the correct datatype is now used for UIA_LiveSettingPropertyId events.
  • Ran linting and formatting as required by the repository.
  • No new tests were added, but the change was validated by manual inspection and build checks.

Optional: Ran local builds and checked accessibility event output for correctness.

Changelog

Should this change be included in the release notes: yes

Release Notes Summary:
Fixes accessibility event datatype for UIA_LiveSettingPropertyId in Fabric, ensuring proper screen reader support and UIA compliance.

Microsoft Reviewers: Open in CodeFlow

…of VT_BSTR

Fixes microsoft#15050

Use GetLiveSetting() conversion function to properly convert accessibilityLiveRegion string values to long (VT_I4) datatype as required by UIA_LiveSettingPropertyId.
@ritoban23 ritoban23 requested a review from a team as a code owner December 3, 2025 02:50
@ritoban23
Copy link
Author

@microsoft-github-policy-service agree

@vineethkuttan
Copy link
Contributor

/azp run PR

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 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.

This is a manually created file, this won't work. Delete this

Run yarn change in your local repo, follow this doc

@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs: Author Feedback The issue/PR needs activity from its author (label drives bot activity) label Dec 3, 2025
@acoates-ms
Copy link
Contributor

This is a manually created file, this won't work. Delete this

Run yarn change in your local repo, follow this doc

Looks like the format is ok. Its likely ok as is. It does look like you are missing parts of your changes though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs: Author Feedback The issue/PR needs activity from its author (label drives bot activity)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants