Bug 1888430 - Cypress: Fix 'aria-hidden-focus' accesibility violations
Summary: Cypress: Fix 'aria-hidden-focus' accesibility violations
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Management Console
Version: 4.6.z
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: 4.6.z
Assignee: David Taylor
QA Contact: Yadan Pei
URL:
Whiteboard:
Depends On: 1885702 1888985
Blocks:
TreeView+ depends on / blocked
 
Reported: 2020-10-14 20:22 UTC by OpenShift BugZilla Robot
Modified: 2021-01-11 10:24 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2021-01-11 10:24:00 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description OpenShift BugZilla Robot 2020-10-14 20:22:05 UTC
+++ This bug was initially created as a clone of Bug #1885702 +++

Created attachment 1719478 [details]
Cypress a11y violations info.

The 3 'aria-hidden-focus' a11y viloations in these Cypress tests:

*Running: crud/namespace-crud.spec.ts*

1 accessibility violation was detected for Create Namespace modal

'aria-hidden-focus' │ 'serious' │ 'Ensures aria-hidden elements do not contain focusable elements' │ 1 │
       delete the Namespace

1 accessibility violation was detected for Delete Namespace modal

'aria-hidden-focus' │ 'serious' │ 'Ensures aria-hidden elements do not contain focusable elements' │ 1 │
       delete project

 
Running: monitoring/monitoring.spec.ts

1 accessibility violation was detected for Expire Silence modal

'aria-hidden-focus' │ 'serious' │ 'Ensures aria-hidden elements do not contain focusable elements' │ 1 │{code}
    delete project

You can run these tests in the Cypress Test Runner and click on the a11y violations to see more info. in the Dev-Console, Console. (see attachement)

Comment 1 David Taylor 2020-12-04 14:59:19 UTC
Decided not to backport this fix due to (a) turns out this was only a partial fix (https://github.com/openshift/console/pull/7382 truly addresses this issue), and (b) the PR also turns on 'fail CI upon A11y violations detected' which is something we do not want to do so late in the 4.6 release cycle.

Comment 2 Jakub Hadvig 2021-01-11 10:24:00 UTC
Check: https://bugzilla.redhat.com/show_bug.cgi?id=1888430#c1


Note You need to log in before you can comment on or make changes to this bug.