-
-
Notifications
You must be signed in to change notification settings - Fork 755
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
Image gallery listing variation has empty alt attributes #6302
Comments
Don't abuse the description for this. In many cases the description is shown as a caption, which is not the same purpose as alt text. The caption is text which is shown (or read) to all users and explains what the image is. The alt text is only shown (read) to users who can't see the image visually, and it adds any additional description that is needed for someone who can't see the image visually to understand it. As long as the caption is always shown along with the image, the alt text can be empty in some cases when the caption already provides all necessary description. Repeating the same description in the alt text would be a worse UX for users with screen readers. I would support adding alt text as a field on the Image content type. It'll also need to be a catalog metadata column in order to be included in the summary serialization for listings. |
@davisagli In volto core we do not show any image caption at all in the image gallery. We do have it in vlt though. We could also go ahead and add the image caption in core. I added an issue at plone.app.contentypes here: plone/plone.app.contenttypes#700 |
I had a chat with a11y expert Jacopo Deyla (@jdeyla) about this and he explained that if images have a description next to them (like we have in vlt, see screenshot), as long as it is clear that the image is described by this text (e.g. the |
@jackahl One more thing to consider: Plone also shows the description field in listings, and it might not always make sense to show the same text in a listing, vs. next to or below an image. |
@davisagli hm... I cant really see many situations where that would be the case. As the description of an image should usually describe the image. But on the other hand, I had a look at the current implementation of the image gallery and it seems not really possible to add any other labels than the alt tag to the image due to technical limits of the gallery. I think I will push for the alt field on the content type tiself then, as I think this would be a useful addition for other cases as well |
Then maybe we need a better gallery. A gallery that shows images without their descriptions/captions is less useful. |
I was thinking about maybe wanting different descriptions for different contexts. like "main image for the fall 2024 marketing campaign" in the listing vs "Members of the Plone community gather to collaborate at the Salamina Sprint in Ferrara, Italy." as a caption. Maybe this is too niche a requirement to include in core. |
I do agree in general, though this would mean a significant effort from our side. Another idea could be to add the option to add the |
Image gallery listing variation has empty alt attributes. I think one could argue that they are not necessarily of decorative nature and should therefore have decent alt tags.
As you cannot edit the contents within the slideshow, the source for the alt-text should come from the source object itself. I see two ways to do this:
The text was updated successfully, but these errors were encountered: