-
Notifications
You must be signed in to change notification settings - Fork 2
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
accessibility issues flagged by DubBot #1087
Comments
We've closed all the tasks listed in this issue; I'll go ahead and close this one and we can create more issues as we discover problems |
ah, the issues were checked off but I forgot that was for implementation and many of the changes are still in the PR that is causing a lighthouse error; re-opening this to remind myself to revisit — want to try cherry-picking fixes into a new branch one at a time so I can figure out what's breaking the lighthouse check |
@rlskoeser - should this be closed now or are there issues we need to keep on our radar? |
I probably never circled back to close it after the sub-tasks were completed, thanks for reminding me. Let's close it. |
reopening since the changes were made but never merged due to a failing lighthouse check; changes are in #1116 and have been updated to patch against latest version of develop branch; can be merged as soon as we resolve the lighthouse check |
Lighthouse check resolved, closing! |
The text was updated successfully, but these errors were encountered: