I started with the 4.5.0 version of this module. I've changed how theme_print_link() outputs the image so that it works in 4.6.1 and I changed print.tpl.php for XHTML compliance.

This should now allow Print Friendly Pages to completely work with Drupal 4.6.

CommentFileSizeAuthor
#1 print.tpl.php1.64 KBEglish
print.module5.21 KBEglish
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Eglish’s picture

Eglish CreditAttribution: Eglish commented
FileSize
print.tpl.php1.64 KB

print.tpl.php

km’s picture

km CreditAttribution: km commented
Assigned: Unassigned » km

Thanks a lot!

I will investigate and commit your changes in two weeks (after a vacation).

markus_petrux’s picture

markus_petrux CreditAttribution: markus_petrux commented

Tidy reported a warning related to the site link on the top. The following:

l($base_url,'')

Generates an empty href, which can be fixed link this:

l($base_url,$base_url)

;)

deekayen’s picture

deekayen CreditAttribution: deekayen commented
Priority: Critical » Normal
Status: Needs review » Fixed

Comitted to CVS for Drupal 4.7 (but not tagged for 4.7 packaging). Needs backport for older versions.

Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)