For many sites, this variable contains a large amount of data - and in the case of a large number of items of the menu item page is renders too heavy.

CommentFileSizeAuthor
#7 taxonomy_menu-DRUPAL-6--2_description.patch3.27 KBdboulet
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

indytechcook’s picture

indytechcook CreditAttribution: indytechcook commented
Title: Please - add switch(OFF)able option to set ($term->description) as "title=" of the item of menu » Ability to turn off description
Version: 6.x-1.02 » 6.x-1.x-dev
Assigned: Ne_L@drupal.ru » indytechcook

I will add this to the list of features for the module rewrite that is currently underway.

indytechcook’s picture

indytechcook CreditAttribution: indytechcook commented
Version: 6.x-1.x-dev » 6.x-2.x-dev
Component: User interface » Code

I will create a plugin with this ability.

indytechcook’s picture

indytechcook CreditAttribution: indytechcook commented
Assigned: indytechcook » Unassigned
temp’s picture

temp CreditAttribution: temp commented

where i can disable description?

indytechcook’s picture

indytechcook CreditAttribution: indytechcook commented

You currently can't.

dboulet’s picture

dboulet CreditAttribution: dboulet commented

Marked #771266: Allow suppression of tooltip / popup / link title text on menus as a duplicate.

dboulet’s picture

dboulet CreditAttribution: dboulet commented
Status: Active » Needs review
FileSize
taxonomy_menu-DRUPAL-6--2_description.patch3.27 KB

Here's a quick patch for review.

indytechcook’s picture

indytechcook CreditAttribution: indytechcook commented

Thanks dboulet,

I do have a question about intended functionality. Would you want to default to just the term name or leave the attribute blank?

dboulet’s picture

dboulet CreditAttribution: dboulet commented

To me it doesn't make much sense to use the term name as a default, it's already being used for the link text.

indytechcook’s picture

indytechcook CreditAttribution: indytechcook commented
Status: Needs review » Fixed

This as been added with a slight modification. http://drupal.org/cvs?commit=379810

Setting the field to true does not set the title attribute. Otherwise it uses the description or title.

Status: Fixed » Closed (fixed)

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

spgd01’s picture

spgd01 CreditAttribution: spgd01 commented
Status: Closed (fixed) » Active

The title should be populated for accessibility. It makes perfect sense for the term name to be the title for this. It should have a check box to use term name for title and a check to just disable title for those that don't care about accessibility.
Thanks

dboulet’s picture

dboulet CreditAttribution: dboulet commented

The title should be populated for accessibility. It makes perfect sense for the term name to be the title for this. It should have a check box to use term name for title and a check to just disable title for those that don't care about accessibility.

spgd01, how does adding the term as a title attribute for the link improve accessibility? How does not not adding a title to a text link mean that a person does not “care about accessibility” ?

dstol’s picture

dstol
Primary language English
CreditAttribution: dstol commented
Status: Active » Closed (duplicate)

I want to address this in 7.x first, which already has an open ticket. Closing as duplicate of #1283914: Don't Include Taxonomy Description in Menu Title Tags