window.print() should be in a Drupal.behaviors function instead of in onload="" so that other modules' jQuery stuff can run.
I've attached a patch with a really quick edit to the template. This should be moved into a drupal_add_js() call, but I wasn't sure what the side-effects of removing $print['sendtoprinter'] would be.

CommentFileSizeAuthor
print-jquery.patch1.29 KBJamesK

Comments

JamesK’s picture

JamesK commented

Note: The above issue occurs in IE7/8. Unsure for other browsers.

jcnventura’s picture

jcnventura
Primary language English
commented
Status: Needs review » Fixed

Hi,

Thanks for the patch. I've just committed it to CVS.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

kristen pol’s picture

kristen pol
she/her
Primary language English
Location Santa Cruz, CA, USA
commented
Status: Closed (fixed) » Active

I am reopening this because the dialog does not show up for me in Firefox or Chrome on Linux with this change. It does show up if I put the window.print() back into the body onload.

Anyone able to see it for either of these 2 browsers? I tried the change from #958896: window.print() dialog not showing in Chrome (http://drupal.org/node/958896) but it did not work for me.

Thanks,
Kristen

jcnventura’s picture

jcnventura
Primary language English
commented
Status: Active » Closed (fixed)

I've tested this working, so this is not the problem. Also, please don't reopen closed issues when you've already reported the problem in another issue.