Add title attribute to taxonomy links - use term description as text [#35178]

We're missing a great opportunity to make our taxonomy links richer and more useful. Currently, the taxonomy links only have the href attribute (required), and nothing else. The patch changes that so that taxonomy links now look like this:

<a href="/path/to/term" title="description of term">term name</a>

The added benefit for the "semantic" web is clear; so is the added search engine optimization potential.

Comment	File	Size	Author
#8	taxonomy_description_2.patch	991 bytes	robertDouglass
#8
#3	taxonomy_description_1.patch	1017 bytes	robertDouglass
#3
#1	taxonomy_description_0.patch	985 bytes	robertDouglass
#1
	taxonomy_description.patch	985 bytes	robertDouglass

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

robertDouglass CreditAttribution: robertDouglass commented 25 October 2005 at 10:01

File	Size
taxonomy_description_0.patch	985 bytes

While we're at it, I think we should add rel="tag" as well. I think this will make us friendlier to sites like technorati, but still have to research whether it alone is enough to make Drupal taxonomy links technorati tags.

Comment #2

robertDouglass CreditAttribution: robertDouglass commented 25 October 2005 at 10:04

Sorry, uploaded the old patch again. Here's the correct one for rel="tag".

Comment #3

robertDouglass CreditAttribution: robertDouglass commented 25 October 2005 at 10:04

File	Size
taxonomy_description_1.patch	1017 bytes

too many buttons to push *blush*

Comment #4

sethcohn CreditAttribution: sethcohn commented 25 October 2005 at 11:16

Simple, adds user friendliness and search engine juice.

Comment #5

breyten CreditAttribution: breyten commented 25 October 2005 at 11:16

Status:

Needs review

» Reviewed & tested by the community

Patch applies cleanly, big +1 on relTag support.

Comment #6

pfaocle

English

CreditAttribution: pfaocle commented 25 October 2005 at 11:36

Good ideas, patch works fine.

Comment #7

Steven CreditAttribution: Steven commented 25 October 2005 at 18:20

Status:

Reviewed & tested by the community

» Needs work

Input validation thingie: the values for HTML attributes are plain-text and are already escaped in drupal_attributes() in l(). So the check_plain() here is unnecessary.

Other than that, looks fine.