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.

CommentFileSizeAuthor
#8 taxonomy_description_2.patch991 bytesrobertdouglass
#3 taxonomy_description_1.patch1017 bytesrobertdouglass
#1 taxonomy_description_0.patch985 bytesrobertdouglass
taxonomy_description.patch985 bytesrobertdouglass

Comments

robertdouglass’s picture

robertdouglass commented
StatusFileSize
new taxonomy_description_0.patch985 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.

robertdouglass’s picture

robertdouglass commented

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

robertdouglass’s picture

robertdouglass commented
StatusFileSize
new taxonomy_description_1.patch1017 bytes

too many buttons to push *blush*

sethcohn’s picture

sethcohn commented

+1

Simple, adds user friendliness and search engine juice.

breyten’s picture

breyten commented
Status: Needs review » Reviewed & tested by the community

Patch applies cleanly, big +1 on relTag support.

pfaocle’s picture

pfaocle
He/Him
Primary language English
commented

+!

Good ideas, patch works fine.

Steven’s picture

Steven commented
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.

robertdouglass’s picture

robertdouglass commented
StatusFileSize
new taxonomy_description_2.patch991 bytes

without check_plain.

lennart’s picture

lennart commented

+1 clean, functional and transparent

sethcohn’s picture

sethcohn commented
Status: Needs work » Reviewed & tested by the community

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

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD. Thanks.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)