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

[DO NOT MERGE] a11y fixes #5512

Closed
wants to merge 20 commits into from
Closed

[DO NOT MERGE] a11y fixes #5512

wants to merge 20 commits into from

Conversation

pascalwengerter
Copy link
Contributor

@pascalwengerter pascalwengerter commented Jul 12, 2021

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:

  • Should fix some more issues between # 5375 and # 5398

@update-docs
Copy link

update-docs bot commented Jul 12, 2021

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.

@kulmann kulmann changed the title [DO NOT MERGE] First a11y fixes [DO NOT MERGE] a11y fixes Oct 26, 2021
@CLAassistant
Copy link

CLAassistant commented Oct 24, 2022

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
3 out of 4 committers have signed the CLA.

✅ lookacat
✅ pascalwengerter
✅ kulmann
❌ Pascal Wengerter


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.

@JammingBen
Copy link
Collaborator

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?

@pascalwengerter
Copy link
Contributor Author

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment