One thing I realised when coding the weblinks module is that the option of adding target="_blank" to the link is missing.
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?!
should i provide a patch?
If you have the time, it would be apprieciated.
as ax suggested, l(), la(), and lm() now accept an associative array called $attribs which will be used as attributes in the tag.
Closed.
Comments
Comment #1
ax commentedi 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?!
Comment #2
barryorourke commentedshould i provide a patch?
If you have the time, it would be apprieciated.
Comment #3
moshe weitzman commentedas ax suggested, l(), la(), and lm() now accept an associative array called $attribs which will be used as attributes in the tag.
Comment #4
barryorourke commentedClosed.