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

Keyboard Controls requires additional ARIA markup to assist screen reader users in understanding what a section is for, and where it begins and ends. #4997

Closed
cryptasian opened this issue Dec 29, 2023 · 1 comment
Labels
area-accessibility Bot Services Required for internal Azure reporting. Do not delete. Do not change color. bug Indicates an unexpected problem or an unintended behavior. customer-reported Required for internal Azure reporting. Do not delete.

Comments

@cryptasian
Copy link

Is it an issue related to Adaptive Cards?

No.

What is the PWD impact?

No response

What browsers and screen readers do this issue affect?

No response

Are there any code-based customization done to Web Chat?

No, I am using Web Chat without any customizations except "styleOptions".

What version of Web Chat are you using?

Latest production

Which area does this issue affect?

ARIA roles

What is the public URL for the website?

https://edfinancial.com

How to reproduce the issue?

This is feedback from the accessibility company Allyant who has been testing our website and has people who use screenreaders and other assistive technologies as a part of their testing team.

Keyboard controls require additional ARIA markup to assist screen reader users in understanding what a section is for, and where it begins and ends.
keyboard-controls-arias

What do you expect?

Add a role="region" and aria-label="Keyboard controls" to the element wrapping this content.

What actually happened?

See other responses.

Do you have any screenshots or recordings to repro the issue?

No response

Did you find any DOM elements that might have caused the issue?

No response

MAS reference

No response

WCAG reference

https://www.w3.org/WAI/WCAG22/Understanding/info-and-relationships

WAI-ARIA reference

No response

Adaptive Card JSON

No response

Additional context

No response

@cryptasian cryptasian added area-accessibility Bot Services Required for internal Azure reporting. Do not delete. Do not change color. bug Indicates an unexpected problem or an unintended behavior. customer-reported Required for internal Azure reporting. Do not delete. labels Dec 29, 2023
@OEvgeny
Copy link
Collaborator

OEvgeny commented Jan 2, 2024

I'm looking through accessibility tree and see the control already has the role and name set. The dialog role seems more suitable for the control as it has a close button and resides on top of the chat contents

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-accessibility Bot Services Required for internal Azure reporting. Do not delete. Do not change color. bug Indicates an unexpected problem or an unintended behavior. customer-reported Required for internal Azure reporting. Do not delete.
Projects
None yet
Development

No branches or pull requests

2 participants