The input text field in the biblio search form (id="edit-keys") does not have a label element associated with it, so it fails WCAG 2.0 testing. To fix this, I made a change in biblio.pages.inc at line 791-196, to add a value to the #title attribute, which the Drupal forms API then uses to create a label for the field. To have this label hidden, I added a line to biblio.css.

Is this the best way?

Patches are attached.

CommentFileSizeAuthor
biblio.css_.patch349 bytesaalm
biblio.pages_.inc_.patch99.3 KBaalm
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mgifford’s picture

mgifford
he/him
Primary language English
CreditAttribution: mgifford commented

This doesn't look like a good patch. Should only be a few lines long.
https://drupal.org/node/707484

Also good to use git https://drupal.org/project/biblio/git-instructions and be clear which branch you are using if there are more than one.

Thanks for pointing out the missing label. It's usually an easy fix.

Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland at University of Waterloo commented
Status: Active » Closed (outdated)

This version is no longer maintained. If this issue is still relevant to the Drupal 7 version, please re-open and provide details.