Skip to content
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] Vignettes lacks alt= in plots #352

Open
MichaelChirico opened this issue Feb 3, 2023 · 2 comments
Open

[Accessibility issue] Vignettes lacks alt= in plots #352

MichaelChirico opened this issue Feb 3, 2023 · 2 comments
Labels
Core Packages 📦 Discussion and planning about core packages of easystats Documentation 📚

Comments

@MichaelChirico
Copy link

We have a tool that flags images that lack an alt= attribute. See https://help.siteimprove.com/support/solutions/articles/80000863904-accessibility-image-alt-text-best-practices -- these are essential for conveying the image contents to users on screen readers.

You can add these to your vignettes by using the recent-ish fig.alt chunk option in knitr; see https://www.rstudio.com/blog/knitr-fig-alt/.

The vignettes at issue are https://github.com/easystats/insight/blob/main/vignettes/insight.Rmd (5x) and https://github.com/easystats/insight/blob/main/vignettes/export.Rmd (1x)

Unfortunately the lines are flagged in the HTML output, which won't correspond exactly to the .Rmd file lines, but I assume if you're familiar with the vignette it will be clear which images need it.

Here's a dump of the output on the HTML vignettes:

insight/inst/doc/export.html:426
# ---
insight/inst/doc/insight.html:387
insight/inst/doc/insight.html:420
insight/inst/doc/insight.html:429
insight/inst/doc/insight.html:438
insight/inst/doc/insight.html:448
@IndrajeetPatil
Copy link
Member

Thanks, @MichaelChirico! Wasn't aware of the new fig.alt chunk option.

I am moving this issue to the meta-package repo, so that we make sure that we are addressing this concern for all 10 packages in the ecosystem.

@IndrajeetPatil IndrajeetPatil transferred this issue from easystats/insight Feb 18, 2023
@IndrajeetPatil IndrajeetPatil added the Core Packages 📦 Discussion and planning about core packages of easystats label Feb 18, 2023
@IndrajeetPatil
Copy link
Member

Also related to easystats/see#127.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Core Packages 📦 Discussion and planning about core packages of easystats Documentation 📚
Projects
None yet
Development

No branches or pull requests

8 participants