I think it would be great if to have rel="bookmark" for permalinks. This usage is documented as a microformat, though it also has its detractors.

HTML5, currently a draft, explicitly describes the use of rel="bookmark" to indicate a permalink.

It should be optional, like nofollow.

Comments

andrewmacpherson’s picture

Status: Active » Needs review
StatusFileSize
new8.21 KB
new2.21 KB

Here's my patch for this. Please review it and let me know what you think.

It adds an option to the admin settings page to allow rel="bookmark" with permalinks.

If both nofollow and bookmark options are checked, permalinks will have the attribute rel="nofollow bookmark".

The option defaults to false, so as not to add the feature to someone's website automatically after a module update.

todd nienkerk’s picture

Status: Needs review » Fixed

Thanks! I have added this functionality -- with a few minor changes -- to the 6.x-1.x-dev branch.

todd nienkerk’s picture

Title: rel="bookmark" option » Add rel="bookmark" option
Assigned: Unassigned » todd nienkerk
andrewmacpherson’s picture

Status: Fixed » Needs review
StatusFileSize
new775 bytes

Hi Todd,

Thanks for including this feature, and improving my code.

However, when I tested the new code from cvs, it doesn't work - the rel attributes are not being added to the link.

A wee error has been introduced; my old choice of variable name persists in one condition. Easily fixed, see patch.

todd nienkerk’s picture

Status: Needs review » Fixed

andrewmacpherson: Sorry for the runaround there. I realized my mistake earlier this morning and fixed it. It's been committed to the dev branch (along with some handy implementations of template_preprocess functions).

Status: Fixed » Closed (fixed)

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