Download & Extend

Add title attribute to taxonomy links - use term description as text

Project:Drupal core
Version:x.y.z
Component:taxonomy.module
Category:feature request
Priority:normal
Assigned:robertDouglass
Status:closed (fixed)

Issue Summary

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.

AttachmentSizeStatusTest resultOperations
taxonomy_description.patch985 bytesIgnored: Check issue status.NoneNone

Comments

#1

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.

AttachmentSizeStatusTest resultOperations
taxonomy_description_0.patch985 bytesIgnored: Check issue status.NoneNone

#2

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

#3

too many buttons to push *blush*

AttachmentSizeStatusTest resultOperations
taxonomy_description_1.patch1017 bytesIgnored: Check issue status.NoneNone

#4

+1

Simple, adds user friendliness and search engine juice.

#5

Status:needs review» reviewed & tested by the community

Patch applies cleanly, big +1 on relTag support.

#6

+!

Good ideas, patch works fine.

#7

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.

#8

without check_plain.

AttachmentSizeStatusTest resultOperations
taxonomy_description_2.patch991 bytesIgnored: Check issue status.NoneNone

#9

+1 clean, functional and transparent

#10

Status:needs work» reviewed & tested by the community

+1, deals with Steven's above validation concern.
Any other reason not to commit?

#11

Status:reviewed & tested by the community» fixed

Committed to HEAD. Thanks.

#12

Status:fixed» closed (fixed)
nobody click here