-
Notifications
You must be signed in to change notification settings - Fork 0
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
Improve/accessibility #146
Conversation
Thank you for spotting and this PR @bmlancien! Please go ahead fixing the most important (!) things and I can review as soon as it is ready. |
Alright @nesnoj. In this PR I mostly did the following:
I didn't work on making the map more understandable, as this is too much of a process to do it properly. I didn't see any accessibility option directly on the MapLibre page, only this. There are also some areas where I probably missed things or didn't implement the correct way. For example I was technically not able to link the slider labels to the sliders for the screen readers. I'm also not very sure of how easy it is no navigate with a screen reader on a web app that actually consists of only one page but have different views toggled on and off... But after that PR the accessibility score should - hopefully - be a bit better. For future tools I would recommend to focus more on the accessibility issue, especially accessibility in data visualization. I will try to gather more knowledge in that field as it is very relevant to the tools developed by the institute. I would also find it interesting to test future tools with users with disabilities to see how good/bad the apps are. |
was dropped during merge
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thx @bmlancien for these improvements!
Closes #144