147653 - "Page setup..." menu option for printing does not fire in MailNews

Status: VERIFIED FIXED

(MailNews Core :: Printing, defect)

Description

[Brice Ruth]

Selecting "Page setup..." from the File menu does not fire the event in
Mail/News while viewing an email. Switching to a browser window and selecting
"Page setup..." again brings up the proper window and the setting entered
(portrait, landscape, etc.) is carried over to the print settings for Mail/News.
You just can't open the Page setup dialog from there.

I'm running build 2002052305 (1.0.0 RC3+) on Mac OS X v.10.1.4

Comment 1

[Greg K.]

Confirmed using FizzillaCFM/2002052305 (RC3).

Comment 2

[dcone (gone)]

more of a UI mailnew printing issue

Comment 3

[Benjamin Mucci]

Same under Mac8.6

Comment 4

[Greg K.]

Setting OS to Mac System 8.6 per comment 3.

Comment 5

[Greg K.]

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

Comment 6

[Reinout van Schouwen]

Possibly related to bug 151669.

Comment 7

[Andrew Hagen]

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

Comment 8

[Andrew Hagen]

All/all on basis of duplicate bug 151669.

Comment 9

[esther]

Using build 20030218 trunk on winxp, linux and maco osx this is only a maco osx
bug.  Changing OS to mac osx.

Comment 10

[sairuh (rarely reading bugmail)]

conrad / simon, any idea why File - Page Setup from mailnews isn't working at all?

also, this doesn't work either from an editor window or the addressbk window.

as mentioned above, the native OS X page setup dlg does work fine when selecting
file - Page Setup from a browser window instead.

Comment 11

[Samir Gehani]

Mail triage team: nsbeta1+/adt1

Comment 12

[Andrew Hagen]

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

Comment 13

[shliang]

Attachment: patch

platformCommunicatorOverlay uses command id Browser:PrintSetup for Page Setup
which only the browser has

Comment 14

[(not reading, please use [e-mail] instead)]

Comment on attachment 117145 [details] [diff] [review]
patch

talked it over with shuehan.

platformCommunicatorOverlay.xul actually has two usages of this command.

even though it's "Browser:*", maybe the right fix is to add a <command ...> for
it to the right places in mailnews?

(note, there is a usage in mail compose, does it work?)

Comment 15

[shliang]

Attachment: better patch

Comment 16

[(not reading, please use [e-mail] instead)]

Comment on attachment 117160 [details] [diff] [review]
better patch

r/sr=sspitzer

there's some cruft in utilityOverlay.xul (lines 51-61) can you remove it when
you checkin?

Comment 17

[shliang]

resolving

Comment 18

[jag (Peter Annema)]

Comment on attachment 117160 [details] [diff] [review]
better patch

r=jag

Comment 19

[esther]

using trunk build 20030410 on mac osx 10.2.4 this is fixed.  Verified.  Now can
we get the Print preview 181552 bug fixed.  Thanks.