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

Issue 604824 link

Starred by 2 users

Issue metadata

Status: Duplicate
Merged: issue 567317
Owner:
Not on Chrome anymore
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Compat



Sign in to add a comment

Extra left padding while printing the page content

Reported by msha...@gmail.com, Apr 19 2016

Issue description

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

Example URL:
N/A (protected page)

Steps to reproduce the problem:
1. Open the attached page (saved as HTML complete page in Chrome)
2. Ctrl+P to print preview

What is the expected behavior?
The page content should be aligned to left.

What went wrong?
There is a big left padding over the section which hidden for printing

Does it occur on multiple sites: N/A

Is it a problem with a plugin? No 

Did this work before? Yes ver 48 or lower

Does this work in other browsers? Yes 

Chrome version: 50.0.2661.75  Channel: stable
OS Version: 6.3
Flash Version: Shockwave Flash 21.0 r0

IE11/FF45/Edge - no issues
Opera36 has the same bug
 
chrome print bug.zip
218 KB Download
Cc: rnimmagadda@chromium.org
Components: UI>Browser>PrintPreview
Labels: M-50 OS-Linux OS-Mac
Owner: nainar@chromium.org
Status: Assigned (was: Unconfirmed)
====================================

Good Build:

47.0.2520.0    Base Position: 350949


Bad Build:

47.0.2526.0    Base Position: 352221

=====================================

Able to repro this issue on Windows 7, MAC (10.11.4) & Ubuntu Trusty (14.04) for the Google Chrome Stable Version - 50.0.2661.75

This is a regression issue broken in M47, below mentioned is the bisect info:

CHANGELOG URL: https://chromium.googlesource.com/chromium/src/+log/a81593e7f162428585832ac8f6e71f75592b53e7..54343df7b15f054d878db7862a1eb5cc2f7a738c

Suspecting Commit: 39a296120585ce464ccffaeb8ed2067ee1ba56d9 			

Review URL: https://codereview.chromium.org/1375833003

@nainar: Could you please look into the issue, and if it has nothing to do with your changes and if possible please do assign it to the concerned owner.

Thank you.

Comment 2 by nainar@chromium.org, Apr 27 2016

Hi! I am attaching the PDF produced from the print preview. It seems ok to me, could you PTAL? And let me know if I am missing out on something? Thanks!
604824.pdf
81.8 KB Download

Comment 3 by nainar@chromium.org, Apr 27 2016

To add I was testing on Chrome 50
@nainar: Yes, the PDF you have shared looks good and no issue in that. But the actual PDF attached by the reporter is still problematic.

Thank you.

Comment 5 by nainar@chromium.org, Apr 27 2016

To confirm the contents of the zip file were an html and a folder. Could you please attach the PDF in question here? Thank you!
Labels: Needs-Feedback
@msharko: Could you please confirm that we should be using the PDF file "Welcome - HPIBET Admin (Complete)" from the attached .ZIP file?

Just wanted to confirm whether we are on same page.

Thank you.
Welcome - HPIBET Admin (Complete).html
69.2 KB View Download
Hi, i'm having the same problem since Chrome 47.

Can you see if it's related to this bug that was reported fix but this for margins: https://bugs.chromium.org/p/chromium/issues/detail?id=412281


The problem may be with transitions and padding on containers.

Looking at OPs hpibet file:


On desktop, #wrapper has a padding-left of 250px which is set to 0 in @media print which is ignored. @media print ignores his transition none reset.

If you remove his transition properties on #wrapper and print preview, it renders properly.




Comment 8 Deleted

Useragent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/50.0.2661.94 Safari/537.36

I've attached a sample html file.

You'll notice how the @media screen 1024px affects the print preview. But also the sample shows chrome ignoring the resets on @media print as well.

Version 52.0.2729.0 canary has the bug as well if you used the sample html attached. However, on it's jsfiddle version, the problem isn't there.

https://jsfiddle.net/2uar5vh3/
chrometest.html
2.9 KB View Download
Please ignore the canary comment about jsfiddle showing fixed. It was because the jsfiddle preview being less than 1024px.
I can confirm that I am able to repro the test provided in #9. Will look into it.
Mergedinto: 567317
Status: Duplicate (was: Assigned)
First of all, sorry it has taken this long to get back on this bug. 

I talked to @esprehn about this at BlinkOn - and seems like the issue lies in: https://cs.chromium.org/chromium/src/components/printing/renderer/print_web_view_helper.cc?rcl=0&l=708 where we are resizing way before we should which is triggering transitions - this is unfamiliar code to me. I am taking a look at it. Theoretically will have to move resize code to later and probably move it to Blink land.

Sign in to add a comment