When Disqus is setup at the bottom of the page my Wave plugin says it's missing some alt text as well as a label correctly set. Here's a screenshot of the warning. Not sure if this is an issue with the module implementation or the service itself. Conducting an audit for accessibility and noticed it.

CommentFileSizeAuthor
Wave46.23 KBbtopro

Comments

Everett Zufelt’s picture

Everett Zufelt commented
Issue tags: +Accessibility

Irony... your bug posting is not accessible :)

I could try to give some insight if I could have a better explanation of the errors that WAVE is providing you. Or, the URL of the site where this is enabled, with directions on how to reproduce.

btopro’s picture

btopro commented

ha. Here's the site I've got it running on that I noticed the issue -- http://drupal.psu.edu/

Run the Wave test on the site after it's loaded and there are two errors. I'm pretty sure one is that the anonymous / poster picture doesn't have alt/title tags.

The other one I believe is that the "sort by popular now" select box doesn't have a label like CCK / Views generates their form elements in the style of 

<label for="cool-field">Cool Field</label>
robloach’s picture

robloach
he/him
commented
Version: 6.x-1.7 » 7.x-1.x-dev
Priority: Normal » Minor
Status: Active » Postponed (maintainer needs more info)

You might want to bring this up to them, as there's not much we can do on the client-side. Thanks!

btopro’s picture

btopro commented

will do, wasn't sure if it was in scope of this project (figured it wasn't).

mgifford’s picture

mgifford
he/him
Primary language English
commented

Is there an issue that you could point to on their site? I couldn't find anything interesting in http://docs.disqus.com/search/?q=accessibility

I searched within the results here without anything useful coming up. I also tweeted them a link to this problem.

btopro’s picture

btopro commented

Did the same, I don't think they've gotten to the issue as of yet. It's a pretty small change, probably just an oversight at the moment. I think it just needed a label field or something.

mgifford’s picture

mgifford
he/him
Primary language English
commented

If anyone cares to RT this - http://twitter.com/#!/mgifford/status/83262776772009985

Ultimately it's about being the squeaky wheel.

bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
commented
Status: Postponed (maintainer needs more info) » Closed (works as designed)

Hey guys, considering that the markup in question is brought in by Disqus and not this module, I'm closing the thread. Feel free to chime in if anyone hears anything from Disqus.

mgifford’s picture

mgifford
he/him
Primary language English
commented

Or perhaps we should engage with Disqus and ask them to improve their markup for accessibility.