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

a11y: Fix keyboard navigation and focus display issues on the home screen #12776

Open
nwmac opened this issue Dec 4, 2024 · 8 comments · Fixed by #12966
Open

a11y: Fix keyboard navigation and focus display issues on the home screen #12776

nwmac opened this issue Dec 4, 2024 · 8 comments · Fixed by #12966
Assignees
Labels
area/accessibility kind/enhancement priority/0 QA/manual-test Indicates issue requires manually testing size/2 Size Estimate 2
Milestone

Comments

@nwmac
Copy link
Member

nwmac commented Dec 4, 2024

a11y Issues: 62, 64, 65, 69, 60

Fix keyboard navigation and focus issues on the home screen:

In the clusters table - sorting icons do not support keyboard navigation or show focus - these should be fixed in the Sortable Table component.

image

Preferences link in the banner - link does not support keyboard navigation or show focus

image

Banner Close Button - close button does not support keyboard navigation or show focus

image

Banner Image Close Button - close button does not support keyboard navigation or show focus

image

** Issue 60 - Textual description missing for interactive image**

These two images are missing the aria-label attribute:

image
image

@nwmac nwmac added this to the v2.11.0 milestone Dec 4, 2024
@github-actions github-actions bot added the QA/dev-automation Issues that engineers have written automation around so QA doesn't have look at this label Dec 4, 2024
@aalves08 aalves08 self-assigned this Jan 2, 2025
@yonasberhe23 yonasberhe23 added QA/manual-test Indicates issue requires manually testing and removed QA/dev-automation Issues that engineers have written automation around so QA doesn't have look at this labels Jan 9, 2025
@yonasberhe23
Copy link
Contributor

Needs manual validation at this time as we dont have existing accessiblity tests. Follow-up ticket to create automated tests coverage rancher/qa-tasks#1639

@aalves08
Copy link
Member

@yonasberhe23 there's already an issue for creating automated tests: #8986

where there's an associated PR with a WIP inclusion of cypress-axe: #12900

We wanted to knock out the priority 0 issues before spending time on tests, since I imagine that's going to take a while to configure and run.

@yonasberhe23
Copy link
Contributor

@aalves08 ah thanks for pointing that out. I'll close the ticket I created.

@IsaSih IsaSih self-assigned this Jan 13, 2025
@IsaSih
Copy link
Contributor

IsaSih commented Jan 15, 2025

Navigation using tab and return work as expected in
Rancher | v2.11-1e40bc40c55a020fc67dc3c3dde47b41dc94caf1-head
Dashboard | master 574248d
Rancher CLI | v2.11.0-rc.1
Helm | dev
Machine | v0.15.0-rancher125

However, the space bar does not perform click actions in elements inside the table. It works for elements outside tables.
Same as in #12786, want to confirm @nwmac if this is working as expected? I reopened the issue because I believe that space should also perform the same actions as the return/enter keyboard shortcut.

@aalves08
Copy link
Member

The decision made is that "links" that appear as buttons are to have the space key as trigger as simple links, anchors, non-button links should not have the space key as trigger, as per #12959 (comment) + #12959 (comment)

@IsaSih
Copy link
Contributor

IsaSih commented Jan 29, 2025

I'm reopening this issue as I identified a regression from the version I tested last.
The closing X icon of the image in the home screen is not being selected by tab navigation. It's selected by tab just if the user first clicks the image with the mouse. This is demonstrated in the screenshot:

Screen.Recording.2025-01-28.at.16.28.34.mov

Tested in

Rancher v2.11-a1bda936c809e0394520c9d40d2998529223401e-head
Dashboard master c987984
Rancher CLI v2.11.0-rc.1

@aalves08
Copy link
Member

aalves08 commented Jan 29, 2025

I'm reopening this issue as I identified a regression from the version I tested last. The closing X icon of the image in the home screen is not being selected by tab navigation. It's selected by tab just if the user first clicks the image with the mouse. This is demonstrated in the screenshot:

Screen.Recording.2025-01-28.at.16.28.34.mov
Tested in

Rancher v2.11-a1bda936c809e0394520c9d40d2998529223401e-head
Dashboard master c987984
Rancher CLI v2.11.0-rc.1

Hi @IsaSih

I forgot about this. Thanks for reporting it.

I've opened #13228 to make the close button always visible (it was only visible on mouse hover). To note that the home page banner closeability is tied to a user preference as well, meaning that if that preference is set to false a user won't see the X to close it. I think it comes true (closable) as default. FYI @rak-phillip

@aalves08
Copy link
Member

Since the solution is more complex, according to #13228 (comment) , I've created #13251 to track the reported situation from @IsaSih .

Can this issue be closed or is this a blocker @IsaSih @nwmac ? FYI @rak-phillip

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment