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

Make screen reader announce status changes in userlist #11376

Merged
merged 1 commit into from Feb 11, 2021

Conversation

KDSBrowne
Copy link
Collaborator

@KDSBrowne KDSBrowne commented Feb 11, 2021

What does this PR do?

This makes the screen reader announce the status changes in the user list so user's don't have to navigate to get the info.

sr-status-alert

The caveat with this solution is the status notifications are not done through the toasts. For the screen reader this means to announce the status changes, the user-list must be open.

Closes Issue(s)

Closes #11372

@antobinary antobinary added this to the Release 2.3 milestone Feb 11, 2021
@antobinary antobinary merged commit 78b21c9 into bigbluebutton:develop Feb 11, 2021
@basisbit
Copy link
Collaborator

basisbit commented Feb 11, 2021

@KDSBrowne can this be disabled or filtered somehow (besides hiding the users list)? I'd imagine otherwise it could be very annoying for blind attendees when there are hundreds of users in the session, and some people join or leave every few seconds?

@KDSBrowne
Copy link
Collaborator Author

@basisbit At the moment there isn't a way to disable apart from closing the user-list. The announcements could surely get annoying if a large number of users continually change their status or join / leave (due to the potential noise).

The element that announces the user is currently set to aria-live="assertive". Changing this to "polite" would make the announcement less intrusive but still probably as annoying. In either case, the screen readers have a shortcut to stop any on going announcement.

Alternatively, a setting could be used to toggle the aria-live property to "off" for disabling.

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

Successfully merging this pull request may close these issues.

Announce status changes
3 participants