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

Enabling nested-interactive a11y check to check the status of failures #154005

Closed
wants to merge 14 commits into from

Conversation

bhavyarm
Copy link
Contributor

We had disable nested-interactive a11y validation check because there were a lot of failures. This PR is to check how many failures so I can log them.

@bhavyarm bhavyarm self-assigned this Mar 29, 2023
@bhavyarm bhavyarm added Project:Accessibility v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v8.8.0 and removed v8.0.0 labels Mar 29, 2023
@bhavyarm
Copy link
Contributor Author

Keeping this open to keep track of failures.

@bhavyarm
Copy link
Contributor Author

bhavyarm commented Jun 9, 2023

@Heenawter when you get a moment can you please validate if this is a valid a11y error?
Thanks very much!

@Heenawter
Copy link
Contributor

Looks like this is the same error that's been around for ages - I've actually already been working on a fix for this, which should hopefully be in soon :)

@bhavyarm
Copy link
Contributor Author

@Heenawter Yay. Awesome. Thanks!

@bhavyarm bhavyarm added v8.10.0 and removed v8.8.0 labels Aug 1, 2023
@bhavyarm
Copy link
Contributor Author

bhavyarm commented Aug 1, 2023

Closing this to open a fresh PR.

@bhavyarm bhavyarm closed this Aug 1, 2023
@kibana-ci
Copy link
Collaborator

kibana-ci commented Aug 1, 2023

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] x-pack/test/accessibility/config.ts / Maps app Accessibility loads maps workpads
  • [job] [logs] x-pack/test/accessibility/config.ts / Maps app Accessibility loads maps workpads

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @bhavyarm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Project:Accessibility release_note:skip Skip the PR/issue when compiling release notes v8.10.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants