Download & Extend

addition of option for target="_blank" to l()

Project:Drupal core
Version:x.y.z
Component:Code
Category:feature request
Priority:normal
Assigned:moshe weitzman
Status:closed (fixed)

Issue Summary

One thing I realised when coding the weblinks module is that the option of adding target="_blank" to the link is missing.

Comments

#1

i think there should be added not only the "target" attribute, but a general purpose (array of) attributes parameter. which could be called like

l("linktext",
  $args,
  array(
    "title" => "the title"
    , "target" => "_blank"
    , "name" => "an_anchor"
    //, "class" => "a-css-class"
    //, "style" => "text-decoration: none; "
    // there are 30 more attributes possible for <a> ;)
  ),
  $script)
)

implementation is straightforward (should i provide a patch?)

where is the <pre> allowed html tag?!

#2

should i provide a patch?

If you have the time, it would be apprieciated.

#3

Assigned to:Anonymous» moshe weitzman

as ax suggested, l(), la(), and lm() now accept an associative array called $attribs which will be used as attributes in the tag.

#4

Priority:major» normal

Closed.

nobody click here