551899 - Cannot set Orientation can Page size from Print Dialog/Page Setup tab.

Status: RESOLVED FIXED

(Core :: Widget: Gtk, defect)

Description

[Ritesh Khadgaray]

User-Agent:       Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.8) Gecko/20100216 Fedora/3.5.8-1.fc12 Firefox/3.5.8 GTB6 AutoPager/0.5.2.2 (http://www.teesoft.info/)
Build Identifier: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.8) Gecko/20100216 Fedora/3.5.8-1.fc12 Firefox/3.5.8 GTB6 AutoPager/0.5.2.2 (http://www.teesoft.info/)

Cannot set Orientation can Page size from Print Dialog/Page Setup tab. I have to use "Page Setup" dialog from File menu, which is rather cumbersome. 

Reproducible: Always

Steps to Reproduce:
1. Click on File->Print->Page Setup

Actual Results:  
Orientation and Page size is disabled

Expected Results:  
I should be able to change Orientation and Page size, from Print dialog

gtk_print_unix_dialog_set_embed_page_setup was added with 2.18.

Comment 1

[Ritesh Khadgaray]

Attachment: set gtk_print_unix_dialog_set_embed_page_setup to TRUE

Comment 2

[Martin Stránský [:stransky]]

Comment on attachment 432082 [details] [diff] [review]
set gtk_print_unix_dialog_set_embed_page_setup to TRUE

Can you please check this one?

Comment 3

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Comment on attachment 432082 [details] [diff] [review]
set gtk_print_unix_dialog_set_embed_page_setup to TRUE

looks ok, but I'm a bit worried that this will build a link-time dependency on GTK 2.18, which we might not want. I can never remember the GTK versioning story...

Comment 4

[Michael Ventnor]

Isn't there a GObject property you could set? Then you could just check it at runtime (gtk_major_version, gtk_minor_version). That's what I did when new GTK's supported Selection.

But is that all there needs to be done? There was a lot more work to get the original page setup integrated with Mozilla...

Comment 5

[Karl Tomlinson (back Feb 2 :karlt)]

Comment on attachment 432082 [details] [diff] [review]
set gtk_print_unix_dialog_set_embed_page_setup to TRUE

Thanks for the patch.  Usually we ensure that Gecko builds against GTK+-2.10 or newer, but, even when build against older libraries, still takes advantage of newer features where possible when run against newer libraries.

The easiest way to do this here would be to set the "embed-page-setup" property.
Still the property should only be set when run against versions of GTK that support the property.

There's already a suitable version check here, so it's probably easiest to simply add to the g_object_set list, with additions to the comments.

http://hg.mozilla.org/mozilla-central/annotate/15cb4be209b3/widget/src/gtk2/nsPrintDialogGTK.cpp#l299

Comment 6

[Martin Stránský [:stransky]]

Attachment: v2

Thanks. There is a version with the requested change.

Comment 7

[Martin Stránský [:stransky]]

Cool, thanks.

Comment 8

[Karl Tomlinson (back Feb 2 :karlt)]

http://hg.mozilla.org/mozilla-central/rev/1daf1dc743c2

Comment 9

[Christopher Aillon (sabbatical, not receiving bugmail)]

Comment on attachment 434195 [details] [diff] [review]
v2

Would like to get this in, we have a customer request for this against our packages (note: this needs the patch from bug 545502 to land first, I've already requested approval there).

Comment 10

[LegNeato]

Comment on attachment 434195 [details] [diff] [review]
v2

a=LegNeato for 1.9.2.8. Please land this on mozilla-1.9.2 default.

Comment 11

[Marco Bonardo [:mak]]

http://hg.mozilla.org/releases/mozilla-1.9.2/rev/b374c411830c