-
Notifications
You must be signed in to change notification settings - Fork 52
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
No alt text on photo galleries #666
Comments
Adding the alt text doesn't look straightforward https://photoswipe.com/caption/ |
@jeroen does https://photoswipe.com/caption/ make sense to you, i.e., would you know how to edit the changes added in https://github.com/ropensci/roweb3/pull/487/files so that photo galleries feature alternative text? |
@jeroen friendly reminder in case you have time |
I think they are actually there where a screenreader can find them. Try running this in the chrome devtools: $('figure img').map((x,y) => console.log(y)) Perhaps they get grayed out when the lightbox is opened, but this won't affect a screenreader because the screenreader should not interact with the lightbox. It just reads the page as is. |
After checking with a screen reader user in two different browsers, the screen reader does not find the alt-text. See ropensci-org/blog-guidance#216 for the details. |
The photo galleries don't have the alt text (as far as I can tell using right-click Inspect Accessibility Properties).
For example: https://ropensci.org/blog/2023/02/27/runiverse-discovering/
Code with alt:
roweb3/content/blog/2023-02-27-runiverse-discovering/index.md
Line 40 in 3764b0a
The text was updated successfully, but these errors were encountered: