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

[SF][AnalyticalTable]: A11y - Table header text should not be empty #6209

Closed
1 task done
summer2200 opened this issue Aug 15, 2024 · 2 comments · Fixed by #6408
Closed
1 task done

[SF][AnalyticalTable]: A11y - Table header text should not be empty #6209

summer2200 opened this issue Aug 15, 2024 · 2 comments · Fixed by #6408

Comments

@summer2200
Copy link

Describe the bug

We've received another A11y issue found by axe tool, the "Select All" checkbox is a table header but it does not have any visible text.
The solution could be either add aria-hidden="true" to the table header cell of checkbox, or add some invisible text to it so screen reader can detect.
Screenshot 2024-08-15 at 16 55 26

Isolated Example

No response

Reproduction steps

...

Expected Behaviour

No response

Screenshots or Videos

No response

UI5 Web Components for React Version

1.29.8

UI5 Web Components Version

1.24.8

Browser

Chrome

Operating System

No response

Additional Context

No response

Relevant log output

No response

Organization

No response

Declaration

  • I’m not disclosing any internal or sensitive information.
@ui5-webcomponents-react-bot
Copy link
Contributor

🎉 This issue has been resolved in version v2.2.0 🎉

The release is available on v2.2.0

Your semantic-release bot 📦🚀

@summer2200
Copy link
Author

summer2200 commented Oct 21, 2024

Hi @Lukas742 ,

Thanks for fixing this issue! Now the check box has aria-hidden="true" and it's expected. But actually aria-hidden="true" needs to be added to the table header cell element of the checkbox too, because the cell element is a table header and it does not have any text inside and it's being detected by AXE tool.
I have raised another issue #6530 to track it, please help check, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 🆕 New
Development

Successfully merging a pull request may close this issue.

3 participants