-
Notifications
You must be signed in to change notification settings - Fork 129
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
Accessbility analysis #82
Comments
Hey! I am interested in doing this ticket! |
Okay so, great. 😃 Please notice this is mostly a ticket for investigation, with work mostly in docs etc., and does not have to result in a PR. As such, please (also) explain what you tested and how. And what you noticed, of course. In the end, you may also have a look on how to fix potentially discovered issues. This could then be a PR. 😃 |
BTW https://ffoodd.github.io/a11y.css/errors.html may be useful. |
@renatavmaraj If you need any pointers/help or have questions, feel free to ask. |
Also good to read/have a look is https://a11yfirst.netlify.com/… @renatavmaraj Are you still interested in it? If not, that's no problem, just add a notice, so I know what the current status of this issue is. |
Can I work on this? I will need a few days to go over this and come up with something solid... that sound good? |
@Rish-P Sure and thanks a lot! |
One may have a deep look on whether stuff for accessibility can be improved. Considering that we use Photon colors and other standard stuff, which has been tested for accessibility already, I hope it should be quite okay.
To do so, you may have a look at the accessibility inspector of Firefox 61 or add-ons such as axe devtools.
I had a quick look at the settings and they seem to be okay:
But because of (WebExtension) limitations in my Firefox 61 instance I could not get it to actually get it to analyze the popup though. And even the options page needs a check with axe e.g.
Also generally it may be better to run the tool outside of the browser: https://github.com/dequelabs/axe-core https://www.axe-core.org/
The text was updated successfully, but these errors were encountered: