Change to INSTALL.TXT for _blank target
puredrivel - September 20, 2005 - 07:23
| Project: | Printer, e-mail and PDF versions |
| Version: | 4.6.x-1.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Jump to:
Description
Can i suggest the code in INSTALL.TXT is modified from this;
function theme_print_link($node) {
$attributes = array("target"=> "_blank");
return l(t('printer friendly page'), "node/$node->nid/print",$attributes);
}to the following to keep the title tag for the link
function theme_print_link($node) {
$attributes = array("target"=> "_blank", "title" => "Display a printer friendly version of this page.");
$links = l(t('printer friendly version'), "node/$node->nid/print",$attributes );
return $links;
}
#1
Comitted to CVS for Drupal 4.7 (but not tagged for 4.7 packaging). Also modified print.module's function to match INSTALL.txt's example. Needs backport for older versions.
#2
didn't change status