Hi everyone,

Thanks for the module. Just a minor note on an issue that tripped me up and could be improved.

The instructions for the Rel= attribute are currently:

The most common usage is rel="nofollow" which prevents

So I entered exactly that value in the field, i.e. instead of just typing nofollow, I typed the full tag rel="nofollow", which resulted in my anchors getting the value of rel="rel="nofollow"".

It's the help text that is misleading, perhaps an example of what is acceptable input in the string would help future users of the module? I.e. "Please don't include the rel="" part..."

Thanks,
Mark

Comments

quicksketch’s picture

How about if we themed the field so that the text field was displayed like this?

Rel Attribute:
rel = [ textfield here ]
When output, this link will have this rel attribute. The most common usage is rel="nofollow" which prevents some search engines from spidering entered links.

That way we wouldn't break translations, and the field would be more clear to match the help text.

Mark Theunissen’s picture

Sounds good! ;)

quicksketch’s picture

Status: Active » Fixed

Turns out this was much easier than I thought. Just use the practically unknown "#field_prefix" and "#field_suffix" FAPI options :)

Committed and fixed.

http://cvs.drupal.org/viewvc.py/drupal/contributions/modules/link/link.m...

Anonymous’s picture

Status: Fixed » Closed (fixed)

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