argilla-io / argilla

Argilla is a collaboration tool for AI engineers and domain experts to build high-quality datasets
https://docs.argilla.io
Apache License 2.0
4.04k stars 381 forks source link

Feature - Improve Accessibility for Screenreaders #5634

Closed paulbauriegel closed 3 weeks ago

paulbauriegel commented 4 weeks ago

Improve Accessibility for Screenreaders - Description & some structure via Aria-Attributes

We are currently in the internal review in Deutsche Telekom how to enable people with disabilities for our annotation process. One key-part will be better usability with screen-readers. This PR tries to enable basic functionality via aria attributes. I tried to not touch the current HTML structure too much, but it might make sense to change some of the aria-role="button" to actual html button. This would be the first of a couple of PR that we are proposing. In future PR we would also want to add a MR for a high contrast mode, and some optimizations for keyboard navigation for blind people.

Scope of this one: Description & some structure via Aria-Attributes.

Next steps - one at a time:

We will most likely receive more feedback in our internal review process that we will also try to integrate then.

Type of change

How Has This Been Tested Local Frontend Build w. Firefox tested w. VoiceOver

Checklist

leiyre commented 4 weeks ago

Hi @paulbauriegel, this is great and the proposed future MRs sound awesome. Feel free to move from draft to ready for review whenever you want. Thanks!

paulbauriegel commented 4 weeks ago

@leiyre It's ready to be reviewed from my side. Let me know what you think.

leiyre commented 3 weeks ago

Hi @paulbauriegel, It looks great to me. I will review the HTML and change some tags in other PR. Thank you for this contribution