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

Neuroglancer Viewer has many accessibility issues #659

Open
jingdid opened this issue Nov 4, 2024 · 2 comments
Open

Neuroglancer Viewer has many accessibility issues #659

jingdid opened this issue Nov 4, 2024 · 2 comments

Comments

@jingdid
Copy link

jingdid commented Nov 4, 2024

Upon running an Axe Devtools scan of our neuroglancer page, we surfaced a very large number of accessibility issues that seem inbuilt to the Neuroglancer library.

For example, class .neuroglancer-side-panel-titlebar.neuroglancer-layer-side-panel-title gets assigned background-color: #393; while the text on it in .neuroglancer-layer-side-panel-name gets assigned color: white; and this is not a high enough contrast ratio, resulting in this message: Element has insufficient color contrast of 3.65 (foreground color: #ffffff, background color: #339933, font size: 9.0pt (12px), font weight: bold). Expected contrast ratio of 4.5:1

Most other issues surfaced were either due to "Form elements must have labels" or " elements with an img role must have an alternative text."

@jbms
Copy link
Collaborator

jbms commented Nov 13, 2024

Potentially we could tweak the colors a bit. I would indeed consider issues like that in-scope. As far as I am aware Neuroglancer, being a visualization tool, is not usable anyway by users that rely on a screen reader, for example, and therefore accessibility issues related to screen reader usage are irrelevant.

@jingdid
Copy link
Author

jingdid commented Nov 13, 2024

Tweaking the colors would be very helpful! Not catering to screen reader usage seems pretty reasonable.

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

No branches or pull requests

2 participants