New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 700466 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 2
Type: Bug-Regression



Sign in to add a comment

First top margin after forced break not applied

Reported by be...@macandmia.com, Mar 10 2017

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/57.0.2987.98 Safari/537.36

Steps to reproduce the problem:
1. Right click page and select print command
2. Generated preview for our specific page (packing lists from macandmia.com/dashboard shows that page 1 and 2 overlap each other
3. Default and Custom margins do not allow us to move the margin to correct the issue so the layout is only on 2 pages

What is the expected behavior?
The packing list screen will match the formatting of the print screen

What went wrong?
Generated preview for our specific page (packing lists from macandmia.com/dashboard shows that page 1 and 2 overlap each other

Did this work before? Yes Prior to 5pm CST 3/9/2017

Chrome version: 57.0.2987.98  Channel: stable
OS Version: 10.0
Flash Version:
 
print screen.docx
202 KB Download
packing list layout - correct.docx
157 KB Download

Comment 1 by be...@macandmia.com, Mar 10 2017

We also restored settings on chrome, used multiple users logins, restarted the computer and uninstalled and reinstalled the printer hardware.
Cc: thestig@chromium.org pchalla@chromium.org
Components: -UI Internals>Printing
Labels: M-57
pchalla@ can you please help us in reproducing this issue.
Labels: Needs-Feedback
It looks like the dashboard requires a login. Without being able to look at the dashboard, or a copy of it, it is next to impossible to figure out what's wrong. Can you do one of the following?

a) Make a test account, and email me the credentials, so I can view the dashboard.

or

b) Open the dashboard, press Ctrl + S to save the webpage, select "Webpage, Complete" rather than "Webpage, HTML only" and email me the saved files.

Comment 4 by a...@macandmia.com, Mar 10 2017

Also confirmed the behavior (slightly worse in terms of overlap) on Chrome Version 59.0.3037.0 (Official Build) canary (64-bit)
Components: Blink>Layout>MultiCol
Labels: -Needs-Feedback OS-Chrome OS-Linux OS-Mac
Owner: msten...@opera.com
Status: Assigned (was: Unconfirmed)
Bisect to https://chromium.googlesource.com/chromium/src/+log/ebec09cdbe2965a07c54715cf4ba3b50603efe39..506506eac78a8106c4e92866a59b8c724ddc8b88 so r435917 ?

Comment 6 by msten...@opera.com, Mar 13 2017

Labels: Needs-Feedback
Where's the test case?
Labels: ReleaseBlock-Stable

Comment 8 by gov...@chromium.org, Mar 13 2017

Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged into the release branch ASAP. Thank you!

Know that this issue shouldn't block the release?  Remove the ReleaseBlock-Stable label or move to M58.

mstensho: Please contact Alex in comment 4 for the test case.
Labels: -ReleaseBlock-Stable
I made a smaller test case that we may be able to attach to the bug. I'll let Alex decide if it looks ok.

I hope we can fix this soon, but it probably should not block releases.

Comment 11 by a...@macandmia.com, Mar 17 2017

Here is the test case - thanks to thestig for slimming it down!
packing_slip_example_production.html
1.5 KB View Download

Comment 12 by msten...@opera.com, Mar 17 2017

Cc: a...@macandmia.com

Comment 13 by msten...@opera.com, Mar 17 2017

Thanks for the test case.

I'm not sure if the bug summary is right. I don't see how this relates to "default printing settings".

In the test case I notice two peculiarities:

#1. The small square that intersects with the second .page-container is partially printed on the first and partially on the second page. The second .page-container follows a .page-break, which inserts a forced page break, meaning that the second .page-container will start at the exact top of page 2. The small square (.logo), however, has a negative top margin. What may happen in Blink when you are at the top of the second page and move backwards (e.g. with a negative top margin) is that you end up on the previous page. Firefox doesn't seem to behave like this, at least not with this test case. In Firefox the block with negative margin stays on the second page, and just gets clipped away if it ends up too high up on that page.

#2. The page gets zoomed out a lot (tested with A4 portrait). This also happens in Firefox, though. The reason appears to be that .page-container is 11.7in wide, which is wider than the page area for A4 portrait. So we have to zoom out to make it fit.

I'm guessing that what you want fixed is #1 - right? Or is it something else?
Labels: -Needs-Feedback
Try with landscape / US Letter. I think Alex is interested in fixing (1).

Comment 15 by msten...@opera.com, Mar 20 2017

Summary: Content after negative top margin printed on preceding page (was: Default printing settings changed and cannot be restored)
OK, I'll just assume that then.
tc.html
470 bytes View Download

Comment 16 by msten...@opera.com, Mar 20 2017

Summary: First top margin after forced break not applied (was: Content after negative top margin printed on preceding page)
Confirming that r435917 is responsible for this regression. Attaching a simpler test, which uses multicol instead (easier to test without having to enter print preview all the time).
tc-multicol.html
331 bytes View Download

Comment 17 by msten...@opera.com, Mar 20 2017

Work-around, based on the packing_slip_example_production.html test:

Change this:
.page-break {
  page-break-before: always;
}

To this:
.page-break {
  page-break-after: always;
}
Project Member

Comment 18 by sheriffbot@chromium.org, Oct 20 2017

Labels: Hotlist-Recharge-BouncingOwner
Owner: ----
Status: Untriaged (was: Assigned)
The assigned owner "mstensho@opera.com" is not able to receive e-mails, please re-triage.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -M-57
This needs to be retriaged by Blink>Layout folks.

Comment 20 by e...@chromium.org, Oct 30 2017

Components: -Internals>Printing
Status: Available (was: Untriaged)
Owner: mstensho@chromium.org
Status: Assigned (was: Available)
Cc: e...@chromium.org
Status: Fixed (was: Assigned)
The multicol test now works in LayoutNG (and the printing test will too, as soon as we implement printing). I don't think we'll ever get around to fixing it in the legacy engine, so we'll just have to wait for LayoutNG to ship.

Sign in to add a comment