You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the problem
Per the latest W3C specifications and accessibility guidelines, the aria-sort attribute should only have values of “ascending”, “descending”, or “other” to indicate the sort order. If sorting does not exist for the column, no aria-sort attribute should be added.
currently outputs a value of "none" for columns that are not sortable but should not.
How do you reproduce the problem?
In OpenShift Management Console, upgrade axe-core and cypress-axe to the latest versions, run other-routes.cy.ts, and note the aria-sort="none" accessibility failures.
Expected behavior
No aria-sort="none" accessibility issues should be present after upgrading axe-core and cypress-axe to the latest versions in OpenShift Management Console.
rhamilto
changed the title
Bug - Table - columns that are not sortable should not have aria-label
Bug - Table - columns that are not sortable should not have aria-sortSep 18, 2024
Describe the problem
Per the latest W3C specifications and accessibility guidelines, the
aria-sort
attribute should only have values of “ascending”, “descending”, or “other” to indicate the sort order. If sorting does not exist for the column, noaria-sort
attribute should be added.patternfly-react/packages/react-table/src/components/Table/utils/decorators/sortable.tsx
Line 55 in 1850cf4
How do you reproduce the problem?
In OpenShift Management Console, upgrade
axe-core
andcypress-axe
to the latest versions, runother-routes.cy.ts
, and note thearia-sort="none"
accessibility failures.Expected behavior
No
aria-sort="none"
accessibility issues should be present after upgradingaxe-core
andcypress-axe
to the latest versions in OpenShift Management Console.Is this issue blocking you?
Yes. We cannot upgrade
axe-core
andcypress-axe
in OpenShift Management Console as a result of this bug, which in turn is blocking the ability to upgrademonaco-editor
.Screenshots
cc: @logonoff
The text was updated successfully, but these errors were encountered: