-
Notifications
You must be signed in to change notification settings - Fork 71
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
File Upload Dropzone Accessibility Fixes #1759
Conversation
✅ Deploy Preview for modus-webcomponents ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
daf08f9
to
6b20582
Compare
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.
thanks for spotting that. That was an error on my part, the updated SVG I added should have viewport 24 x 24, not 32x32 as it was before. Fixed now locally and committing |
20bbc68
to
d89e283
Compare
d89e283
to
6719258
Compare
6719258
to
ef2b961
Compare
Azure Static Web Apps: Your stage site is ready! Visit it here: https://icy-ground-0374a1310-1759.centralus.1.azurestaticapps.net |
Description
role="alert"
(REF: https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/alert_role) - FIXEDType of change
How Has This Been Tested?
Edge v117
Checklist