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

deps(axe-core): update axe-core to latest (beta) release #4595

Merged
merged 3 commits into from
Mar 2, 2018

Conversation

benschwarz
Copy link
Contributor

There's a number of important performance fixes in the latest axe-core release that will significantly improve the speed of LH (and in many cases allow the accessibility audits to complete successfully for sites with big dom).

Ref: dequelabs/axe-core/issues/696
Ref: dequelabs/axe-core/issues/701
Ref: dequelabs/axe-core/issues/702

Top work, @paulirish 💓

@benschwarz benschwarz changed the title Update axe-core to latest (beta) release perf(axe-core): Update axe-core to latest (beta) release Feb 21, 2018
@benschwarz benschwarz changed the title perf(axe-core): Update axe-core to latest (beta) release deps(axe-core): update axe-core to latest (beta) release Feb 21, 2018
@patrickhulce
Copy link
Collaborator

These performance improvements are indeed amazing 💯, are we comfortable shipping a beta release of axe long-term though? If it's buggy we'll hear about it for 18+ weeks until it finally leaves Chrome stable 😆

How vetted is it?

@benschwarz
Copy link
Contributor Author

I’m not super familiar with the release schedule of axe, but after it hits a non beta we’d want to move to the stable ASAP.

@paulirish
Copy link
Member

with lighthouse --config-path=./lighthouse-core/config/a11y.js http://mlb.com --fast -G

the a11y gatherer went from taking 31s to 9s on my machine. \o/

@paulirish
Copy link
Member

Let's wait for dequelabs/axe-core#731 to land and the beta it ships in.

@paulirish
Copy link
Member

Let's wait for dequelabs/axe-core#731 to land and the beta it ships in.

done.

@GoogleChrome GoogleChrome deleted a comment from googlebot Mar 2, 2018
@paulirish paulirish merged commit 54225c4 into GoogleChrome:master Mar 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants