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

A11Y: Added name and instrument to accessibility labels for mute, solo, group #2639

Merged
merged 1 commit into from
May 27, 2022

Conversation

chigkim
Copy link
Contributor

@chigkim chigkim commented May 22, 2022

Short description of changes
Added name and instrument to accessibility labels for mute, solo, group

CHANGELOG: Accessibility: Added name and instrument to accessibility labels for mute, solo, group

Context: Fixes an issue?

When you tab to land on a control like mute, solo, group, screen reader just says Mute Unchecked. Now it says Mute Bob, Keyboard, Expert, Unchecked, so Users know which person to mute, solo, and etc.

Does this change need documentation? What needs to be documented and how?

Status of this Pull Request

What is missing until this pull request can be merged?

Checklist

  • I've verified that this Pull Request follows the general code principles
  • I tested my code and it does what I want
  • My code follows the style guide
  • I waited some time after this Pull Request was opened and all GitHub checks completed without errors.
  • I've filled all the content above

@chigkim chigkim force-pushed the a11y-labeled-controls branch from 4d4840e to 66e7501 Compare May 22, 2022 19:42
@hoffie hoffie added this to the Release 3.9.0 milestone May 22, 2022
Copy link
Member

@hoffie hoffie left a comment

Choose a reason for hiding this comment

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

Thanks, looks good to me.

Copy link
Member

@ann0see ann0see left a comment

Choose a reason for hiding this comment

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

Tested on Windows 11 with the integrated screenreader and it works. However the level-meter doesn't (but that's not part of this PR)

@ann0see ann0see merged commit b10cbfe into jamulussoftware:master May 27, 2022
@chigkim chigkim deleted the a11y-labeled-controls branch July 31, 2022 04:29
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