245483 - Hook up print preview support

Status: RESOLVED FIXED

(Thunderbird :: Mail Window Front End, defect)

Description

[Alexander Edgren]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.7) Gecko/20040601 Firefox/0.8.0+
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.7) Gecko/20040601 Firefox/0.8.0+

Mozilla Thunderbird version 0.6+ (20040603) branch build.

Print preview toolbar is missing.

Reproducible: Always
Steps to Reproduce:
1. Click on a message
2. Choose print preview
3. Toolbar is missing



Expected Results:  
Shown the toolbar with print, close, zoom and so on.

Reported here first http://forums.mozillazine.org/viewtopic.php?p=557921#557921

Comment 1

[Scott MacGregor]

the print preview window is failing to find the binding for the preview toolbar.

We need to add the following CSS blurb to a CSS file that gets loaded into the
print preview window.

toolbar[printpreview="true"] {
  -moz-binding:
url("chrome://global/content/printPreviewBindings.xml#printpreviewtoolbar");
}

Comment 2

[Scott MacGregor]

Attachment: the fix (or at least the start of one)

Turns out we were never hooked up to use the toolkit print preview code. This
patch attempts to address that fact. I'm sure testing will turn up more issues
with the print preview toolbar in mail. The biggest problem I had was some
browser specific code in PrintUtils.js that I had to ifdef around. 

I also removed content/communicator.css which we don't need anymore.

Comment 3

[Scott MacGregor]

fixed on the branch and the trunk. Hopefully most of it works too :)

This new feature will need some testing help. 

Comment 4

[Robert Accettura [:raccettura]]

Just FYI, I don't think we need it on the Mac (since the Mac has it's own
print-preview function in the printer options).

Not sure if this is enabled on the Mac..  I'm at my PC laptop right now about to
head out.

Comment 5

[Scott MacGregor]

*** Bug 246497 has been marked as a duplicate of this bug. ***

Comment 6

[jay garcia]

WFM in today's build.