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

fix: replace hardcoded checked/ unchecked names with toggle pattern #1529

Merged
merged 3 commits into from
Jan 13, 2023

Conversation

sfoslund
Copy link
Member

Details

Fixes the checked/ unchecked instances in #1514

Motivation

Addresses issue #1514

Context

While checking that this did not disrupt any existing styling, I noticed/ filed #1528 to be addressed separately.

Pull request checklist

@sfoslund sfoslund requested a review from a team as a code owner January 12, 2023 23:40
Copy link
Contributor

@DaveTryon DaveTryon left a comment

Choose a reason for hiding this comment

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

Very nice! Approved with one small question

@sfoslund sfoslund merged commit f750072 into microsoft:main Jan 13, 2023
@sfoslund sfoslund deleted the FixToggle branch January 13, 2023 21:27
sfoslund added a commit that referenced this pull request Jan 19, 2023
#### Details

Expand on fix in
#1529. I
found another set of radio buttons which were not properly announcing
their state/ implementing the toggle pattern. This PR leverages with
ToggleMenuItem control added in
#1529 to
ensure that checked/ unchecked states are announced.

Steps to repro:
- Live inspect any app with AI win
- Go to "More options" and "listen to events"
- Select event settings button (beside recording button)
- Expand "Event recording scope" menu item and observe 3 radio buttons
that do not announce their state to screenreaders

##### Motivation

Address another instance of improper implementation of UIA patterns that
was missed in
#1514

#### Pull request checklist
<!-- If a checklist item is not applicable to this change, write "n/a"
in the checkbox -->

- [ ] Run through of all [test
scenarios](https://github.com/Microsoft/accessibility-insights-windows/blob/main/docs/Scenarios.md)
completed?
- [n/a] Does this address an existing issue? If yes, Issue# - 
- [n/a] Includes UI changes?
- [ ] Run the production version of Accessibility Insights for Windows
against a version with changes.
  - [ ] Attach any screenshots / GIF's that are applicable.
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.

2 participants