-
Notifications
You must be signed in to change notification settings - Fork 3
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 updates #366
base: develop
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for vico-design ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for recogito-cloud-staging ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@rsimon Sorry. Lots of format only changes here as well. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! I made a few tweaks & added DE labels.
@@ -17,6 +17,8 @@ export const RecogitoLogo = (props: Props) => ( | |||
viewBox='0 0 644 121' | |||
fill='none' | |||
xmlns='http://www.w3.org/2000/svg' | |||
role='img' | |||
aria-label={'Recogito Studio Logo'} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could be "Recogito Studio Logo"
directly
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I fixed that already while I was at it...
Summary
Analyzed all screens and ran axe DevTools scans and corrected what issues I could.