-
Notifications
You must be signed in to change notification settings - Fork 125
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
Add a11y support to the landscape #842
Comments
i'd also love to see this prioritized. the idea of a landscape is very powerful and i'd love if the breadth of our landscapes could be accessible to as many as possible. |
I think that is fixed, at least the alt text |
Yes, can you link to the commits?
I think we can link to an a11y test showing decent results and continue to
iterate and improve
On Thu, Sep 29, 2022 at 3:46 PM Andrey Kozlov ***@***.***> wrote:
I think that is fixed, at least the alt text
—
Reply to this email directly, view it on GitHub
<#842 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAPSIPA334PZRM73RFLZFLWAX5ZDANCNFSM6AAAAAAQMGUO74>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
--
Cheers,
Chris Aniszczyk
https://aniszczyk.org
|
that would be great. and great for our internal discussions too. many thanks! |
Adding alt text without fixing keyboard navigation doesn't do much, unfortunately. The contrast on the text within the tiles, which actually contains the information, is the thing that can be addressed independently. While the WebAIM contrast checker indicates that it may pass in some situations, at the size of the text involved it fails AAA. Additionally, the alt text only works on card mode. For landscape mode, the alt text is insufficient. See the functional images guidance. |
All landscapes are lacking accessibility support, including but not limited to:
A basic lighthouse audit gives landscapes a sub-60 score.
The text was updated successfully, but these errors were encountered: