calcite-action ARIA attributes

[vivzhang]

Check existing issues

• [X] I have checked for existing issues to avoid duplicates and reviewed the Accessibility page for guidance.

Summary

Remove the aria-disabled, aria-busy and aria-pressed attributes on calcite-action. Please ignore the request for aria-label, the Hub team will address that by passing in a unique value for the label property for their use case.

Original AMP issues: 11242 11244

Actual Behavior

aria-disabled, aria-busy and aria-pressed attributes rendered on the button inside calcite-action

Expected Behavior

No aria-disabled, aria-busy and aria-pressed rendered on the button inside calcite-action

Reproduction Sample

https://codepen.io/vivzhang/pen/rNXVWBo

Reproduction Steps

1. Open the devtool console
2. Inspect the calcite-action component
3. You should see the aria-disabled and an aria-busy attributes rendered on the button element

Reproduction Version

2.12.1

Working W3C Example/Tutorial

No response

Relevant Info

related: https://github.com/Esri/calcite-design-system/issues/10706

Regression?

No response

Priority impact

impact - p3 - not time sensitive

Calcite package

• [x] @esri/calcite-components
• [ ] @esri/calcite-components-angular
• [ ] @esri/calcite-components-react
• [ ] @esri/calcite-design-tokens
• [ ] @esri/eslint-plugin-calcite-components

Esri team

Arcgis Hub

[jcfranco]

Some notes:

• aria-disabled – this should be addressed in today's release via https://github.com/Esri/calcite-design-system/pull/10367
• aria-busy – this was added here https://github.com/Esri/calcite-app-components/issues/357

[driskull]

I think we need to research more why aria-busy on an action is a problem. Would we need another property added to action to allow aria-busy to be set? like a loadable property?

I would expect this would apply to other components where aria-busy is being used.