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

Remove image/inacessible #1157

Closed
ChristopherChudzicki opened this issue Mar 24, 2022 · 0 comments · Fixed by #1158
Closed

Remove image/inacessible #1157

ChristopherChudzicki opened this issue Mar 24, 2022 · 0 comments · Fixed by #1158

Comments

@ChristopherChudzicki
Copy link
Contributor

Remove all instances of

![This resource may not render correctly in a screen reader.](/images/inacessible.gif)

as described in comment

FYI, I just met with the OL accessibility expert and she told me that we should not continue using the hidden image with alt tag to flag "inaccessible" PDFs. She claims that many more PDFs are accessible than we think, and that it's no safe for us to assume what a user can or wants to get out of file. It's sufficient to indicate that it's a PDF. And, if we really wanted to be helpful, we would have a note that explains the content of the PDF, rather then just tell users "this isn't for you."

#1130 (comment)

These links:

  • show missing image icon in studio
  • are bad for accessibility
  • are noise for link fixing / broken link detection
  • are extremely low effort to remove, so lets do it.
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 a pull request may close this issue.

1 participant