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

[Switch]: [Accessibility issue - aria-required] #9916

Closed
1 task done
IlianaB opened this issue Sep 20, 2024 · 2 comments
Closed
1 task done

[Switch]: [Accessibility issue - aria-required] #9916

IlianaB opened this issue Sep 20, 2024 · 2 comments
Assignees
Labels
ACC bug This issue is a bug in the code Low Prio TOPIC B

Comments

@IlianaB
Copy link
Member

IlianaB commented Sep 20, 2024

Bug Description

This issue is originally reported in 9872

According to Access Assistant tool:
The aria-required attribute is not allowed on this DIV

Affected Component

ui5-switch

Expected Behaviour

No warnings with Access Assistant tool.

Isolated Example

https://stackblitz.com/edit/github-qgw27d-nln9nx?file=index.html,main.js

Steps to Reproduce

  1. Open sample https://stackblitz.com/edit/github-qgw27d-nln9nx?file=index.html,main.js
  2. Run Access Assistant and check the Notes

Refered to the switch:
The aria-required attribute is not allowed on this DIV

Log Output, Stack Trace or Screenshots

No response

Priority

Low

UI5 Web Components Version

1.24.6

Browser

Chrome

Operating System

No response

Additional Context

No response

Organization

No response

Declaration

  • I’m not disclosing any internal or sensitive information.
@IlianaB IlianaB added bug This issue is a bug in the code ACC TOPIC B Low Prio labels Sep 20, 2024
@IlianaB
Copy link
Member Author

IlianaB commented Sep 20, 2024

Hello @ui5-webcomponents-topic-b , please check out this accessibility issue about ui5-switch.

Regards,
Iliana

@unazko unazko self-assigned this Sep 21, 2024
@unazko
Copy link
Contributor

unazko commented Sep 24, 2024

Hello @IlianaB,

The aria-required attribute is allowed over an element with role switch since wai-aria-1.2:
https://www.w3.org/TR/wai-aria-1.2/#switch
Also the switch role is present in the list of role for the aria-required attribute:
https://www.w3.org/TR/wai-aria-1.2/#aria-required

The access assistant doesn't yet correspond to all the updates from wai-aria-1.2.

Best regards,
Boyan

@unazko unazko closed this as completed Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ACC bug This issue is a bug in the code Low Prio TOPIC B
Projects
Status: Completed
Development

No branches or pull requests

2 participants