Skip to content
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

Update ControlType values to match corresponding accessibilityRole values #14215

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

Yajur-Grover
Copy link
Contributor

@Yajur-Grover Yajur-Grover commented Dec 19, 2024

Description

Update the control type values of components to corresponding accessibilityRole value.

Type of Change

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

Why

When the accessibilityRole property is set on a component, the controlType property should be updated to match the corresponding accessibilityRole value. Currently, for most components, this was not happening, and was shown in the following bug:

What

Add functionality to update corresponding control type value based on components' accessibilityRole value.

Screenshots

Before:
image

After:
image

Testing

N/A

Changelog

Should this change be included in the release notes: yes

Update the control type values of components to corresponding accessibilityRole value.

Microsoft Reviewers: Open in CodeFlow

@Yajur-Grover Yajur-Grover marked this pull request as ready for review January 7, 2025 21:52
@Yajur-Grover Yajur-Grover requested review from a team as code owners January 7, 2025 21:52
@Yajur-Grover Yajur-Grover changed the title Update ControlType values testing Update ControlType values to match corresponding accessibilityRole values Jan 7, 2025
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.

1 participant