Skip to content

[SF][AnalyticalTable]: ARIA attribute is not allowed: aria-selected="false" #5074

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

Closed
1 task done
zhangliangyi opened this issue Sep 20, 2023 · 1 comment · Fixed by #5080
Closed
1 task done

[SF][AnalyticalTable]: ARIA attribute is not allowed: aria-selected="false" #5074

zhangliangyi opened this issue Sep 20, 2023 · 1 comment · Fixed by #5080
Labels

Comments

@zhangliangyi
Copy link

Describe the bug

The aria-selected property is not allowed in the selection checkbox as the its role is cell. This issue is reported by axe-tools.

Based on the description of aria-selected in MDN, the property is only for the element with role gridcell, option, row and tab. Check https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Attributes/aria-selected for more information.

Isolated Example

No response

Reproduction steps

  1. Go to https://sap.github.io/ui5-webcomponents-react/iframe.html?viewMode=docs&id=data-display-analyticaltable--docs#display-indicator-for-navigated-rows
  2. Open dev tools and select any checkbox in selection column
  3. Also you can open axe-tools to see this issue

Expected Behaviour

No response

Screenshots or Videos

image

UI5 Web Components for React Version

1.19.0

UI5 Web Components Version

1.17.0

Browser

Edge

Operating System

No response

Additional Context

No response

Relevant log output

No response

Declaration

  • I’m not disclosing any internal or sensitive information.
@github-actions
Copy link

github-actions bot commented Oct 2, 2023

🎉 This issue has been resolved in version v1.20.0 🎉

The release is available on v1.20.0

Your semantic-release bot 📦🚀

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

Successfully merging a pull request may close this issue.

1 participant