-
Notifications
You must be signed in to change notification settings - Fork 1.3k
This issue was moved to a discussion.
You can continue the conversation there. Go to discussion →
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 Review #190
Comments
Hopefully the new theme helps with this, but I'd love if someone more read up on web accessibility could take a look at the site and let us know what we're missing! |
Hi @austinlparker,
|
Here are some details for home page :
I can take a look at other pages too if you think this is helpful. |
I can suggest some UI fixes for the same and additionally submit a pr. Could you assign me the issue? |
Sure thing. |
@austinlparker Sorry but could you guide me as to where the code corresponding to the navbar on the homepage is? I cant seem to find it. |
The navigation bar is generated automatically from the content directory - if you look at the There's more info here: https://www.docsy.dev/docs/language/ |
|
This issue was moved to a discussion.
You can continue the conversation there. Go to discussion →
Per open-telemetry/community#370 (comment) and other D&I measures, we'll need to do some work on making the website ARIA friendly and ensuring that we're using best practices for accessibility. This issue will hold the review from myself and others on figuring out what needs to be changed.
The text was updated successfully, but these errors were encountered: