ui5-webcomponents icon indicating copy to clipboard operation
ui5-webcomponents copied to clipboard

SideNavigation: SideNavigationSubItem count is not read

Open pppwr7 opened this issue 2 years ago • 8 comments

Bug Description

The item count information is not read when the SideNavigation ‘Tree items’ are accessed’. Only the name of the item is read when accessed. Raised as a part of accessibility testing.

Expected Behavior

The item name and the count information also should be read when the tree items are accessed.

Steps to Reproduce

Use accessibility testing software to check if the information is read correctly. SideNavigation react ui5 wiki can be used for testing https://sap.github.io/ui5-webcomponents-react/?path=/docs/layouts-floorplans-sidenavigation--default-story.

Context

  • UI5 Web Components version:"@ui5/webcomponents": "1.0.0-rc.14"
  • OS/Platform: windows
  • Browser: chrome
  • Affected component: side navigation

Priority

  • [ ] Low
  • [x] Medium
  • [ ] High
  • [ ] Very High

pppwr7 avatar Apr 12 '22 12:04 pppwr7

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

Could you take a look at this issue? Seems similar to the #5034 feature request.

Regards, Stanislav

stbodurov avatar Apr 14 '22 08:04 stbodurov

Hello @PriyaPowar , I was able to reproduce the issue only when accessing SideNavigationItem, not SideNavigationSubitem. I tested on this page https://sap.github.io/ui5-webcomponents/playground/components/SideNavigation/ with JAWS 2022. For example "4 of 4" is not read out for the "Events" item image Can you check if the issue is the same?

Best regards, Petar

dimovpetar avatar Apr 19 '22 13:04 dimovpetar

Hi dimovpetar , Yes, the issue is at SideNavigationItem level. It should specify the count of items under it. Please check.

pppwr7 avatar Apr 21 '22 14:04 pppwr7

Hello @PriyaPowar, the Side Navigation works as expected. When the focus is on some expanded tree item the count of the sub items don't need to be announced. You can see the same behavior on the ARIA best practice example: https://w3c.github.io/aria-practices/examples/treeview/treeview-navigation.html.

LidiyaGeorgieva avatar Jun 24 '22 13:06 LidiyaGeorgieva

@LidiyaGeorgieva Is it possible to find out which version would contain this fix?

samanr avatar Aug 30 '22 20:08 samanr

Hi @samanr,

There is no issue here. The reading is as expected.

"the Side Navigation works as expected. When the focus is on some expanded tree item the count of the sub items don't need to be announced"

Best, Teo

TeodorTaushanov avatar Sep 01 '22 10:09 TeodorTaushanov

Hi @samanr,

There is no issue here. The reading is as expected.

"the Side Navigation works as expected. When the focus is on some expanded tree item the count of the sub items don't need to be announced"

Best, Teo

@TeodorTaushanov We are using MacOS screenreader and it reads as "Treeitem XXX. XXX, group"

We have not tested it using JAWS latest yet. As you can see it does not read out the side navigation position, i.e. i of 2 or 2 of 2. In my case the subitem is rendered using a list.

However my request was to add an accessibility props to the sidenavigation item so we can pass custom accessibility text. I believe it will make it consistent among other screen readers.

samanr avatar Sep 22 '22 15:09 samanr

I'm reopening the ticket for further investigation.

TeodorTaushanov avatar Sep 27 '22 07:09 TeodorTaushanov

Hi,

We improved the accessibility of the ui5-side-navigation in the main branch.

Note that the different screen readers might read the control differently. For example, JAWS reads "1 of n" only the first time when enter a new tree level. After that when the arrow keys are used, the "2 of n" is not read. This is all up the to screen readers.

Best, Teo

TeodorTaushanov avatar Nov 24 '22 15:11 TeodorTaushanov