Skip to content

[EuiCollapsibleNavBeta] Consider adding aria-hidden="true" to the icon button for screen reader UX #7353

@1Copenut

Description

@1Copenut

Is your feature request related to a problem? Please describe.
The current refactor of EuiCollapsibleNavBeta ( #7337 ) makes tremendous improvements for visual and keyboard UX. I'd like to add on to that work and propose hiding accordions' icon button from screen readers to avoid two buttons in Form Controls menus with the same accessible names.

Describe the solution you'd like
The expand/collapse accordions in EuiCollapsibleNavBeta is composed of two EuiButton components as sibling elements. The second button has a tabindex="-1" to remove it from the tab order. I'd like to take this a step further and add aria-hidden="true" to this button to remove a duplicate announcement for screen reader users.

Additional context
All three screen readers (VoiceOver, NVDA, JAWS) announced each button twice in their respective Form Controls menu and when traversing using the virtual cursor.

Metadata

Metadata

Assignees

No one assigned
    No fields configured for Enhancement.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions