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

aMustHaveTitle accessibility test not necessary #281

Closed
edtechdev opened this issue May 20, 2017 · 1 comment
Closed

aMustHaveTitle accessibility test not necessary #281

edtechdev opened this issue May 20, 2017 · 1 comment

Comments

@edtechdev
Copy link

edtechdev commented May 20, 2017

(Forgive me, I don't have UDOIT running. Please delete/ignore this if aMustHaveTitle is already disabled) There doesn't seem to be a basis for the aMustHaveTitle accessibility test. Screen readers ignore the title attribute on links by default, for example.

The code for the test is on line 371 here: https://github.com/ucfopen/UDOIT/blob/master/lib/quail/quail/common/accessibility_tests.php

It's been requested to be removed from the quail library UDOIT uses, but development on quail has ceased for a long time now, so the issue is still open:
quailjs/quail#363

W3C HTML5 recommends against using the title attribute:
https://www.w3.org/TR/html/dom.html#the-title-attribute

And here are a few articles about not using the title attribute in links for accessibility purposes:
https://silktide.com/i-thought-title-text-improved-accessibility-i-was-wrong/
http://www.mediacurrent.com/blog/dont-rely-title-attribute-accessibility-seo
https://www.paciellogroup.com/blog/2013/01/using-the-html-title-attribute-updated/
http://webaim.org/articles/gonewild/#title

On the quail github page, since they are no longer actively developing it, they recommend to use the axe library instead. I don't see any evidence that the axe library tests for a title attribute in links:
https://github.com/dequelabs/axe-core/blob/develop/doc/rule-descriptions.md

@emilysachs
Copy link

Hi Doug, that test is indeed disabled for UDOIT and only exists in the code from the quail library. Thanks for the concern!

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

No branches or pull requests

2 participants