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

[CHORE] Bumping axe-core to v4.8.2. Matches Kibana version. #7370

Merged
merged 3 commits into from
Nov 21, 2023
Merged

[CHORE] Bumping axe-core to v4.8.2. Matches Kibana version. #7370

merged 3 commits into from
Nov 21, 2023

Conversation

1Copenut
Copy link
Contributor

@1Copenut 1Copenut commented Nov 15, 2023

Summary

Bumping our axe-core ruleset to the current version. This matches Kibana's recent merge to latest.

PR closes #7357.

Note about EuiAccordion

I removed the EuiLink inside our accordion under test. The latest version of axe-core is throwing an error about links relying on color to distinguish them from non-clickable text. While valid, it was throwing a false positive for the component under test. I'd rather handle link underlines as their own task in the name of getting this PR merged.

Screenshot 2023-11-20 at 5 04 26 PM

QA

QA will be done in Buildkite CI. We will know PR is ready to review when all Cypress a11y tests pass green.

@1Copenut 1Copenut marked this pull request as ready for review November 20, 2023 23:09
@1Copenut 1Copenut requested a review from a team as a code owner November 20, 2023 23:09
Copy link
Contributor

@cee-chen cee-chen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM if CI passes!

@kibanamachine
Copy link

Preview staging links for this PR:

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

History

cc @1Copenut

@1Copenut 1Copenut merged commit 76749da into elastic:main Nov 21, 2023
2 checks passed
@1Copenut 1Copenut deleted the chore/axe-core-bump branch November 21, 2023 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CHORE] Bump version of axe-core to match Kibana
4 participants