Print and Print preview ignore the margin setting

VERIFIED FIXED in mozilla0.9.8

Status

()

Core
Printing: Output
P1
normal
VERIFIED FIXED
16 years ago
16 years ago

People

(Reporter: Jean-Marc Desperrier, Assigned: rods (gone))

Tracking

Trunk
mozilla0.9.8
x86
Windows NT
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

1.80 KB, patch
dcone (gone)
: review+
Marc Attinasi
: superreview+
Details | Diff | Splinter Review
(Reporter)

Description

16 years ago
Tested on 2002011703

Steps : 
- change the values for margin in the Page Setup Dialog
- print the page or do a Print Preview
- whatever value you set in the margin option, you get exactly the same result

I had the same behaviour with print preview under Linux with 2002010308.
(Assignee)

Updated

16 years ago
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Priority: -- → P1
Target Milestone: --- → mozilla0.9.8
(Assignee)

Comment 1

16 years ago
Created attachment 65480 [details] [diff] [review]
patch

I really can't explain where and how this got broken. Because I know I tested
this. Oh, well. The key here is you HAVE to be able to get the margins out of
the PrintSettings before you do the reflow.

Comment 2

16 years ago
Comment on attachment 65480 [details] [diff] [review]
patch

r=dcone
Attachment #65480 - Flags: review+

Comment 3

16 years ago
Comment on attachment 65480 [details] [diff] [review]
patch

sr=attinasi
Attachment #65480 - Flags: superreview+

Comment 4

16 years ago
sr=hyatt
a=blizzard on behalf of drivers for 0.9.8
Keywords: mozilla0.9.8+
(Assignee)

Comment 6

16 years ago
fixed
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED

Comment 7

16 years ago
Jean-Marc, please verify....and mark verified-fixed...thanks.
(Reporter)

Comment 8

16 years ago
verified-fixed with 2002020103.

They are other problems :
- "Page Scaling" does not seem to be saved. 
I see the same problem as there was with margins.
Modify the value, click OK, reopen the dialog, the value is back to 100.

- Print preview problem with headers display. 
When using small marging, the display is quite wrong, especially the headers do
not quite appear the way they will in print.
I will open a bug for that, but it does not seem very severe.

Comment 9

16 years ago
please file nbew bugs for those other problems.

Comment 10

16 years ago
so if a new bug is gonna be filed, can we close out this one and mark it
VERIFIED-FIXED ?
(Reporter)

Comment 11

16 years ago
I filed two other bugs :
- bug 123342 : changes to "Page Scaling" value in page setup dialog ignored 
- bug 123345 : changes to "Header" values in page setup dialog impossible

About the display of header with small margin, I will fill a bug when I have 
time for a complete description, and it's not so important, the printing itself 
is right.
Can be closed

Comment 12

16 years ago
Jean-Marc, so this bug can be closed...Please mark it verified-fixed..thanks!
(Reporter)

Comment 13

16 years ago
Verified.

Sorry "http://www.mozilla.org/bugs/" says it's QA that marks bugs as Verified , 
and also I wasn't sure there was really a difference between RESOLVED-FIXED, and 
VERIFIED-FIXED.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.