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

Comments

ax’s picture

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?!

barryorourke’s picture

should i provide a patch?

If you have the time, it would be apprieciated.

moshe weitzman’s picture

Assigned: Unassigned » 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.

barryorourke’s picture

Priority: Major » Normal

Closed.