136785 - Print Preview uses the Zoom and shouldn't

Status: VERIFIED FIXED

(Core :: Printing: Output, defect)

Description

[Max Alekseyev]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (OS/2; U; Warp 4.5; en-US; rv:0.9.9+) Gecko/20020409
BuildID:    2002040916

Mozilla always print 100% scaled text (no matter what Print Preview shows).

Reproducible: Always
Steps to Reproduce:
1. Go to given URL
2. Set View -> Text Zoom -> 50%
3. Go to File->Print Preview and see that all text takes 8 pages (in my case).
4. Push "Print" button and get 28 pages on the printer of 100% scaled text


Actual Results:  Text is printed unscaled. No correlation with what Print
Preview shows.

Comment 1

[Mike Kaply [:mkaply]]

Happens on Windows as well.

Either print preview shouldn't scale, or we should print scaled.

Comment 2

[Max Alekseyev]

Changing scale in Print Preview takes no effect either.
So at the moment there is no way to print scaled text. ;(

Comment 3

[rods (gone)]

Looks like Print Preview takes into account Text Zooming and should or print
need to take into account text zooming

Comment 4

[rods (gone)]

Zoom/Text Zoom is for visually "zooming" the page in galley mode but is not
intended to to do anyhting for printing. Scaling in the the page setup is
intended for that.

The bug here is that PP obeys the TextZoom and should not.

Comment 5

[Kevin McCluskey (gone)]

nsbeta1+

Comment 6

[rods (gone)]

Attachment: patch

Cache the Zoom and TextZoom for the main DC while in Print Preview and then set
it back when leaving Print Preview

Comment 7

[dcone (gone)]

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

r=dcone

Comment 8

[Marc Attinasi]

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

sr=attinasi

Comment 9

[rods (gone)]

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

sett as sr per marc's sr comment

Comment 10

[rods (gone)]

fixed

Comment 11

[Kevin McCluskey (gone)]

adt: Fixing this bug is important in maintaining the user's confidence that
print preview is showing what the actual printed page will look like.

Comment 12

[sujay]

verified on 4/16 trunk build.

Max, this should be working for you now..download latest build
to see fix....if not fixed, then REOPEN.

Comment 13

[Michael Buckland]

sujay, is there any reason why you think this might be reopened?

Comment 14

[Max Alekseyev]

Nice fix but not for the problem I tried to state. Mostly I worried about
ability to print scaled text. I didn't have it before, neither do now.

What I had before:
Print Preview showed scaled text but the was printed unscaled;

What I have now:
Print Preview doesn't scale text at all, and the text is still printed unscale.

Should I reopen this bug, or create a new burreport?

Comment 15

[Max Alekseyev]

Sorry for misprints in previous posting.

Nice fix but not for the problem I tried to state. Mostly I worried about
ability to print scaled text. I didn't have it before, neither do now.

What I had before:
Print Preview showed scaled text, but the text was printed unscaled;

What I have now:
Print Preview doesn't scale text at all, and the text is still printed unscaled.

Should I reopen this bug, or create a new bugreport?

Comment 16

[sujay]

filed bug 137984 for new issue.

Comment 17

[rods (gone)]

Zoom doesn't affect scaling zooming is only for the viewing of web pages not
printing. If you are trying to scal from Print Preview, first open the UI and
turn off "Shrink To Fit" the UI toolbar in PP should be disabled when Shrink TO
Fit is o. That bug has already been filed.

Comment 18

[scottputterman]

adding adt1.0.0+.  Please check into the branch as soon as possible and add
fixed1.0.0

Comment 19

[rods (gone)]

fixed on branch

Comment 20

[sujay]

verified in 4/23 branch build.