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

Issue 669690 link

Starred by 13 users

Issue metadata

Status: Fixed
Owner:
Use other robhogan account instead.
Closed: Dec 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

second and subsequent pages overlap thead+tbody when printing

Reported by paulvi...@gmail.com, Nov 29 2016

Issue description

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

Example URL:
https://bugs.chromium.org/p/chromium/issues/attachmentText?aid=249827

Steps to reproduce the problem:
1. visit the "overlap.html" attachment from  bug #24826 
2. hit control-P to enter a print dialogue
3. scroll print preview to second page; observe top of page

What is the expected behavior?
in MSIE, Edge, and FF, the thead is displayed on second and subsequent pages, and the tbody is displayed below the thead.

What went wrong?
the tbody is displayed on top of the thead, as if it was position:absolute rather than position:relative. note that adding position:relative (from stackoverflow search) is not a workaround, nor is min-height:0.

Does it occur on multiple sites: Yes

Is it a problem with a plugin? No 

Did this work before? No 

Does this work in other browsers? Yes

Chrome version: 54.0.2840.99  Channel: stable
OS Version: 10.0
Flash Version: Shockwave Flash 23.0 r0

it's bizarre to find something working in Edge that's broken in Chrome. this tilts my preferred world view. please make open-source at least as good as proprietary in this use case.
 
overlap.html
1.6 KB View Download
Components: UI>Browser>PrintPreview
Labels: -Pri-2 M-54 Needs-Triage-M54 Pri-1

Comment 2 by paulvi...@gmail.com, Nov 29 2016

let me add that PrintPreview is not required; you can also Save as PDF, or Use System Print Dialogue and then Microsoft PDF Writer, and see the same defects.
Cc: msten...@opera.com
Components: -Blink Blink>Layout>Table
Labels: -Type-Bug OS-Mac Type-Bug-Regression
Status: Untriaged (was: Unconfirmed)
Confirmed on Windows 7, using Chrome 54.

Comment 4 by msten...@opera.com, Nov 30 2016

Cc: robhogan@chromium.org
Cc: -robhogan@chromium.org
Labels: -M-54 hasbisect-per-revision ReleaseBlock-Beta M-57 OS-Linux
Owner: robhogan@chromium.org
Status: Assigned (was: Untriaged)
Able to reproduce the issue on Windows 10,Mac 10.11.6 and Ubuntu 14.04 using stable 54.0.2840.98/99/100 and canary 57.0.2937.0 (used the attached overlap.html file to reproduce).

Bisect info:
==============
Good:53.0.2758.0 -- header is not printed in preview
Bad :53.0.2759.0 -- header is printed in preview and overlapped

CHANGELOG URL:
  https://chromium.googlesource.com/chromium/src/+log/9142199dcf3d38db9c5012aeb6604aa1f921f94d..81c0fc6d4e08e4e2bb4eb8a42747f21b13303616

Suspect: Review-Url: https://codereview.chromium.org/2021703002

robhogan@: Could you please take a lok into it.
Added releaseblock-beta against M57 as its a recent regression and would be good if it can be fixed prior to M57 beta push.

Comment 6 by cjava...@gmail.com, Dec 1 2016

> it's bizarre to find ...

Let's not give the Google developers such grief. lol

But yes, it would be quite helpful if you could implement this for us!

Comment 7 by ajha@chromium.org, Dec 5 2016

Cc: wangxianzhu@chromium.org
robhogan@: Could you please take a look at this.

Thank you!
Fentle remainder !!

@robhogan -- Could you please look into the issue and update it accordingly.
Thank you.

Comment 9 by e...@chromium.org, Dec 13 2016

Labels: -ReleaseBlock-Beta OS-Chrome
Repeating headers across pages is a relatively new feature in chrome, one that has been requested for years. This is an edge case where it overlaps the content of a cell if it is tall enough. While we should fix that I don't think it is worth blocking the repeated headers feature on it as that fixes more problems than it causes.

If this is causing problems on any popular real world websites we'll reconsider of course. In that case please re-add the ReleaseBlock label.

Robhogan/Morten: We should still try to fix this for the next release as it legibility and layout suffers. Leaving as P1.

Comment 10 by robho...@gmail.com, Dec 13 2016

CL has landed so bugdroid should be along soon.
Status: Fixed (was: Assigned)
Fixed by https://codereview.chromium.org/2545243002

Sign in to add a comment