-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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 issue with the Overlay functionality #44115
Comments
Hi @mikachan 👋 Could you please tag relevant folks who can provide UX/UI inputs on this one? Once this gets addressed, we could then work towards introducing this placeholder UX change for the Cover block, too. |
Yes, sure. @Ryokuhi or @alexstine may be able to offer input on this one. You may also wish to bring this issue up in the #accessibility channel on the Making WordPress Slack, as well. |
I am totally blind so I can't offer much feedback here any longer. Tagging @joedolson . |
To the best of my knowledge, if we talk about compliance with Web Content Accessibility Guidelines (WCAG), version 2.1, at level AA, which is what we aim to do according to WordPress Accessibility Coding Standards, maybe this is not even an accessibility issue, strictly speaking. The relevant success criterion here is 1.4.11 Non-text Contrast. After checking the additional content included in the guide to understand the intent of Success Criterion 1.4.11: Non-text Contrast, here are my thoughts on the topic.
Let's hear from someone else from the Accessibility Team (we can even discuss this during the next bug scrub); after there's enough feedback, we can decide about removing the |
I agree with @Ryokuhi that the biggest problem is identifying the boundaries; so a simple white border around the button should be sufficient. While testing, I noted that the upload button doesn't support the 'Show button text labels' preference, which I'd consider a more significant problem, in many ways. |
Thank you so much @Ryokuhi and @joedolson, for the detailed evaluation of the issue raised and for sharing your thoughts on the possible solution!
Understood. Do we need any input from the design team for this? Else, I can raise a PR and you can review the visuals.
Could you please elaborate a little? Are you referring to the Aside, I found a blurred effect with a slight tint on the placeholder illustration, which needs some discussion (it's related to design and not accessibility). I will create a separate issue as it's not exactly related to this. |
Gotcha! @joedolson, I have raised a PR (#44437) for the accessibility issue you pointed out, it also addresses one UI bug seen in your screenshot and adds consistency in the spacing of those items. Would be great if you could take a look and help me with any feedback. |
Description
Pull request #43838 introduces an improved UX for the Post Featured Image block by incorporating the Overlay changes reflected on the image placeholder as well.
The upload button has a blurred background which is being removed in the mentioned PR to ensure it's visible what the overlay will look like. However, this has a drawback, if a user selects an overlay that's very close to the upload button's color, it will blend in which is an accessibility or UX issue.
It's important to address this as we have plans on introducing the overlay color on the Cover block's placeholder as well.
Step-by-step reproduction instructions
Post Featured Image
block. You should see the placeholder for the image.Screenshots, screen recording, code snippet
[Overlay color set to the upload button's color with overlay opacity being 100]
Environment info
Please confirm that you have searched existing issues in the repo.
Yes
Please confirm that you have tested with all plugins deactivated except Gutenberg.
Yes
The text was updated successfully, but these errors were encountered: