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

Sidebar: Close button is missing discernible text #14838

Closed
WTW-Steve-Martin opened this issue Feb 19, 2024 · 6 comments
Closed

Sidebar: Close button is missing discernible text #14838

WTW-Steve-Martin opened this issue Feb 19, 2024 · 6 comments
Labels
Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible

Comments

@WTW-Steve-Martin
Copy link

Describe the bug

The Axe Dev Tools for Accessibility is reporting that the close button for the p-sidebar doesn't have any discernible text.
image

Environment

Windows 10

Reproducer

No response

Angular version

17.2.1

PrimeNG version

17.7.0

Build / Runtime

Angular CLI App

Language

TypeScript

Node version (for AoT issues node --version)

18.18.0

Browser(s)

Chrome and Edge

Steps to reproduce the behavior

  1. Install the Axe Dev Tools for Accessibility plug-in.
  2. Open primeng sidebar page (https://primeng.org/sidebar) on Chrome/Edge browser.
  3. Scan the page with Axe

Expected behavior

The sidebar passes all Axe tests.

@WTW-Steve-Martin WTW-Steve-Martin added the Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible label Feb 19, 2024
@maruthumj
Copy link
Contributor

@WTW-Steve-Martin I will start working on this one

@WTW-Steve-Martin
Copy link
Author

Thanks very much

@maruthumj
Copy link
Contributor

@WTW-Steve-Martin there is one @input property is there called "ariaLabel". can you please pass value to that property and check?

@WTW-Steve-Martin
Copy link
Author

WTW-Steve-Martin commented Feb 19, 2024

I'm afraid adding the ariaLabel property (highlighted in yellow) to the p-sidebar did not change it's associated close button.
image

As you can see the close button (highlighted in grey) does not include an aria label.

Ideally, I would like it to have the aria-label of 'sidebar aria-label' + '_close'. Or something like that.

@maruthumj
Copy link
Contributor

@WTW-Steve-Martin sorry didn't notice it was a sidenav. you can pass the aria-label by using "ariaCloseLabel" property from sidenav. I hope this will resolve your issue!

@WTW-Steve-Martin
Copy link
Author

That solved it. Thanks very much

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible
Projects
None yet
Development

No branches or pull requests

2 participants