Currently, _taxonomy_menu_save() will save a "title" attribute by attempting to use the term description, but degrading to the term name if no description is present. This results in the title attribute being identical to the link text, which besides being redundant, is an accessibility alert.

See the following:

The attached patch only adds the title text if/when the taxonomy term description is present, retaining the ability to provide "advisory" text while avoiding obviously redundant text.

CommentFileSizeAuthor
#3 taxonomy_menu-accessibilty-avoid-duplicate-title-text-2.patch632 bytesmark_fullmer
taxonomy_menu-accessibilty-avoid-duplicate-title-text-1.patch796 bytesmark_fullmer
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mark_fullmer created an issue. See original summary.

Status: Needs review » Needs work

The last submitted patch, taxonomy_menu-accessibilty-avoid-duplicate-title-text-1.patch, failed testing.

mark_fullmer’s picture

mark_fullmer
he/him
Primary language English
Location Tucson
CreditAttribution: mark_fullmer as a volunteer and at University of Texas at Austin commented
FileSize
taxonomy_menu-accessibilty-avoid-duplicate-title-text-2.patch632 bytes
mark_fullmer’s picture

mark_fullmer
he/him
Primary language English
Location Tucson
CreditAttribution: mark_fullmer as a volunteer and at University of Texas at Austin commented
Status: Needs work » Needs review
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Assigned: mark_fullmer » Unassigned
Parent issue: » #3261307: Plan for Taxonomy Menu 7.x-1.7
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Title: Accessibility : Redundant Title Text » Accessibility: Redundant Title Text
Status: Needs review » Fixed

This is a reasonable fix, thank you.

Committed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.