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

Incorrect aria-hidden for non-focusable buttons #2444

Open
rajsite opened this issue Oct 22, 2024 · 0 comments
Open

Incorrect aria-hidden for non-focusable buttons #2444

rajsite opened this issue Oct 22, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@rajsite
Copy link
Member

rajsite commented Oct 22, 2024

🐛 Bug Report

Some button interactions are non-focusable due to aria patterns where alternate keyboard interactions are expected:

  • inc/dec buttons on numeric
  • dropdown button on select / combobox
  • next/prev buttons on tabs

Our current pattern has been to remove those buttons from the aria tree via aria-hidden. However based on discussions in the scrollable tabs PR it seems like removing the buttons from aria all together is not the right direction.

Specifically as an example, the spin button pattern example describes of the inc/dec buttons as the following:

The increase and decrease buttons are not contained within the div with role spinbutton so they can be separately focusable by users of touch screen readers. However, they are excluded from the page Tab sequence with tabindex="-1" because they are redundant with the arrow key support provided to keyboard users.

This issue is to cover our other non-focusable buttons to make sure they are exposed to aria correctly.

💻 Repro or Code Sample

See above.

🤔 Expected Behavior

Non-focusable buttons are still in the aria tree and have correct aria labels

😯 Current Behavior

Non-focusable buttons are aria hidden.

💁 Possible Solution

See above.

🔦 Context

N/A

🌍 Your Environment

N/A

@rajsite rajsite added bug Something isn't working triage New issue that needs to be reviewed labels Oct 22, 2024
@rajsite rajsite mentioned this issue Oct 22, 2024
1 task
@m-akinc m-akinc removed the triage New issue that needs to be reviewed label Oct 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Backlog
Development

No branches or pull requests

2 participants