Skip to content

Improve gallery#1065

Draft
maelle wants to merge 4 commits intomainfrom
gallery
Draft

Improve gallery#1065
maelle wants to merge 4 commits intomainfrom
gallery

Conversation

@maelle
Copy link
Copy Markdown
Member

@maelle maelle commented Jun 3, 2025

  • I made each div in the gallery (each figure) aria-hidden.
  • Along each aria-hidden div I added an sr-only (screenreader only in the Bootstrap version we use) paragraph that says "a picture:" and then the alternative text, plus the caption between parentheses if there is one.

@maelle

This comment was marked as resolved.

@maelle
Copy link
Copy Markdown
Member Author

maelle commented Jun 3, 2025

👋 @LizHareDogs! Would you mind trying https://deploy-preview-1065--ropensci.netlify.app/blog/2023/02/27/runiverse-discovering/ for instance? The pesky divs are aria-hidden, but I wonder if you get all the "sr-only" paragraphs, and whether the text in those is ok. Since the "sr-only" paragraphs are not images I added "A picture" to the text but that might not be the best way to provide context for the alternative text.

Thank you and no hurry!

@LizHareDogs
Copy link
Copy Markdown

LizHareDogs commented Jun 3, 2025 via email

@maelle
Copy link
Copy Markdown
Member Author

maelle commented Jun 3, 2025

Thank you Liz!

@maelle
Copy link
Copy Markdown
Member Author

maelle commented Jun 26, 2025

@LizHareDogs just a reminder for when you have time, but really there is no hurry. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants