-
Notifications
You must be signed in to change notification settings - Fork 438
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
Accesibility issue in edit/access-control #2775
Comments
@CrisGuzmanS : Sounds good. I'll assign back to you then and move this to the 7.6.x board. Please submit a PR when you are ready and we'll get it applied to 8.0 and 7.6.x |
@tdonohue & @CrisGuzmanS: This should already be fixed by PR #2717 |
@alexandrevryghem : Thanks for catching that. It's very difficult to keep all these accessibility issues separate :) @CrisGuzmanS: If you want to test out PR #2717 to see if it's already solved there, that would be appreciated. If it is solved, then there's no need for you to do any work. If it is NOT solved, then you can claim this as a separate task (#2717 is also on my list to test, but I have a very long list right now) |
Hello @tdonohue & @alexandrevryghem , i have already verified that the issue has been solved in #2717 |
Thanks @CrisGuzmanS for your help in testing that PR! In that case, I'm going to link this ticket up to #2717 and close the #2776 PR as a duplicate. Apologies again for the confusion. (I'll try to move #2717 up my list too in order to avoid this happening again, as it seems to fix a lot of small accessibility issues) |
Describe the bug
In the edit section for access control there are two accessibility errors regarding empty labels.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
There should be no error
@tdonohue i can take this activity
The text was updated successfully, but these errors were encountered: