Skip to content

[SideNavigationItem][A11y]: Duplicate text announced by screen readers for some icons in SideNavigationItem #11304

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

Closed
1 task done
AlekseiSch opened this issue Apr 10, 2025 · 4 comments
Assignees
Labels

Comments

@AlekseiSch
Copy link

Bug Description

When using the SideNavigationItem component with certain icons, such as action-settings, screen readers (JAWS and NVDA) announce the element's text twice. For instance, the element is read as "settings settings". This issue occurs because some icons include an aria-label attribute, which causes redundant text to be read by screen readers.

<SideNavigationItem
icon="action-settings"
text="Settings"
onClick={() => {}}
selected={}
onSelect={() => {}}
/>

Affected Component

SideNavigationItem

Expected Behaviour

Screen readers should announce the element's text only once, e.g., "Settings", without duplicating the name from the icon.

Isolated Example

No response

Steps to Reproduce

  1. Use the SideNavigationItem with an icon "action-settings".
  2. Open the application in a web browser.
  3. Launch a screen reader (e.g., JAWS or NVDA).
  4. Navigate to the SideNavigationItem in the UI by using navigation keys.
  5. Observe how the screen reader announces the element:
    Expected: "Settings" (once).
    Actual: "Settings settings" (text is duplicated).
  6. Test with other icons in the SideNavigationItem to see if the issue persists.

Log Output, Stack Trace or Screenshots

No response

Priority

None

UI5 Web Components Version

2.8.0

Browser

Chrome

Operating System

Windows

Additional Context

No response

Organization

No response

Declaration

  • I’m not disclosing any internal or sensitive information.
@plamenivanov91
Copy link
Contributor

Hello @SAP/ui5-webcomponents-topic-rd ,

Could you please look into that ACC issue ?

Regards,
Plamen Ivanov

@LidiyaGeorgieva LidiyaGeorgieva self-assigned this Apr 14, 2025
@LidiyaGeorgieva
Copy link
Contributor

Hello @AlekseiSch ,

I could not reproduce the issue in isolated example - link.
Jaws output:
"Navigation List Tree
Home
To move through or expand items use the Arrow keys"

Best Regards,
Lidiya

@AlekseiSch
Copy link
Author

It seems that this bug was fixed in the latest version. Thanks, I'll close it

@github-project-automation github-project-automation bot moved this from New Issues to Completed in Maintenance - Topic RD Apr 15, 2025
Copy link

This issue has been closed. To reopen, just leave a comment!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Completed
Development

No branches or pull requests

3 participants