Bug 12275

Summary: Printing the page results in 2 printed pages and 40 empty ones
Product: WebKit Reporter: Dimitri Bouniol <dimitri008>
Component: PrintingAssignee: Dave Hyatt <hyatt>
Status: RESOLVED FIXED    
Severity: Normal CC: ap
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
URL: http://bouniol.homeip.net:8080/www.amrag.com/print-test.html
Attachments:
Description Flags
Don't add in negative margins when positioning page breaks, since they result in a backwards slide. eric: review+

Dimitri Bouniol
Reported 2007-01-14 20:34:59 PST
Using the CSS property 'page-break-after: always;' for the divs (class = page) in the file seem to cause it to create 40 blank pages in between the two printed ones. This does not happen in Firefox, but it does in non-nightly version of Safari.
Attachments
Don't add in negative margins when positioning page breaks, since they result in a backwards slide. (789 bytes, patch)
2007-01-15 01:48 PST, Dave Hyatt 		eric: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Alexey Proskuryakov
Comment 1 2007-01-14 21:17:08 PST
(In reply to comment #0) > This does not happen in Firefox, but it does in non-nightly > version of Safari. Do nightly builds work correctly for you?
Dimitri Bouniol
Comment 2 2007-01-14 23:54:01 PST
No, the bug is in the latest nightly. Just print the URL above and click preview - the first and last pages have something on then, but the 40 in the middle are blank.
Dave Hyatt
Comment 3 2007-01-15 01:48:38 PST
Created attachment 12450 [details] Don't add in negative margins when positioning page breaks, since they result in a backwards slide.
Eric Seidel (no email)
Comment 4 2007-01-15 02:08:03 PST
Comment on attachment 12450 [details] Don't add in negative margins when positioning page breaks, since they result in a backwards slide. Looks sane. We don't have to work about right margins or any other margins having this issue? We also need a way to let DRT test printing... bah.
Dave Hyatt
Comment 5 2007-01-15 02:28:09 PST
Fixed.
Dave Hyatt
Comment 6 2007-01-15 02:28:45 PST
Fixed in r18857.
Note You need to log in before you can comment on or make changes to this bug.