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

False negative: 'image-alt' rule does not fail for alt=" " #1174

Closed
iamrafan opened this issue Oct 3, 2018 · 1 comment · Fixed by #1260
Closed

False negative: 'image-alt' rule does not fail for alt=" " #1174

iamrafan opened this issue Oct 3, 2018 · 1 comment · Fixed by #1260
Assignees
Labels

Comments

@iamrafan
Copy link
Contributor

iamrafan commented Oct 3, 2018

  • Demo: <img id="test" src="" alt=" ">

  • AT Tests:

  AT/Browser alt=" " (space)
1 Edge/Narrator Image
2 Edge/JAWS Graphic
3 Edge/NVDA Graphic
4 Chrome/NVDA Graphic
5 Chrome/JAWS Ignored by AT
6 IE/JAWS Ignored by AT
7 IE/NVDA Graphic
8 Safari/voiceover Ignored by AT
9 Firefox/NVDA Graphic
axe-core version: 3.1.1
axe-Coconut version: 3.5.0
@WilcoFiers
Copy link
Contributor

This should probably be a separate check so that we can report the issue accurately, and so that its easy to turn off.

@marcysutton marcysutton self-assigned this Nov 26, 2018
marcysutton pushed a commit that referenced this issue Nov 27, 2018
The image-alt rule now checks for space characters, and fails if they are present. Empty alt attributes are still fine, and text content is still fine. This fixes cases where ATs do not skip decorative images because of the space characters in the alt attribute.

Closes #1174
marcysutton pushed a commit that referenced this issue Nov 27, 2018
The image-alt rule now checks for space characters, and fails if they are present. Empty alt attributes are still fine, and text content is still fine. This fixes cases where ATs do not skip decorative images because of the space characters in the alt attribute.

Closes #1174
WilcoFiers pushed a commit that referenced this issue Dec 7, 2018
* feat(image-alt): require alt text or empty strings

The image-alt rule now checks for space characters, and fails if they are present. Empty alt attributes are still fine, and text content is still fine. This fixes cases where ATs do not skip decorative images because of the space characters in the alt attribute.

Closes #1174

* chore: include updated build files

(there are no actual changes)

* chore: rename to alt-space-value, focus on images

* fix: PR feedback
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants