-
Notifications
You must be signed in to change notification settings - Fork 155
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
[DO NOT MERGE] a11y fixes #5512
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
…tion A11y #5381 table role presentation for version table
Pascal Wengerter seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
A lot of merge conflicts as the code base changed heavily over the last few months. @pascalwengerter what do you think, does it make sense to keep this open? |
I'll close here since the codebase has moved on (as in "improved") a lot in the last 2,5 years, and a11y has gotten its way into many new or refactored components. While the project most likely would need another audit before picking up certification, this PR can still be found if need arises, but isn't actionable enough for now. |
DO NOT MERGE - WIP PR
Description
Implementing feedback from the a11y audit
Related Issue
Also, owncloud/ocis#2309 fixes two more issues (#5376 and #5377)
Open tasks: