-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
Audit error: Required Accessibility gatherer encountered an error when using almond.js #2505
Comments
This seems to be an issue with axe-core interacting with your site. Would you mind filing this issue upstream? I get this stack trace when just trying to run axe on your site without Lighthouse
It seems their UMD handling might have to be updated if (typeof define === 'function' && define.amd) {
define([], function() { // axe.js:24
'use strict';
return axe;
});
} |
@mjpuser can you post the URL of your site? |
|
Sorry for the late feedback here but it's easy to forget stuff after a few weeks of vacation. The thing is that it works fine with their own Developer Tools plugin so I feel that it might be up to how you invoke it. I've solved this on my end using an IIFE around my code anyway so if anyone run into this on their own site then this is a possible workaround. |
Ok. closing this for now as it's resolved in your case. If someone else runs into this we'll have to investigate if this is a bug with AXE's UMD or how we're running it. |
If it works with axe's extension, it's likely because they're running it inside a content script that won't be affected by the globals of the page. Migrating our page evaluations to |
in that case, This is blocked by #2505 |
We are using https://github.com/requirejs/almond in our builds instead of the whole requirejs package and this seems to break lighthouse accessibility tests. I've tried it both via the node package and via the Audit tab in Chrome Developer Tools.
The result is a 0 score for accessibility and all tests have failed with the below message:
Almond has 30 000 downloads the last week on npm and is used in lots of places around the web so a solution or at least a workaround that we can use would be great!
The text was updated successfully, but these errors were encountered: