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

Issue 692875 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Use other robhogan account instead.
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug

Blocked on:
issue 656232



Sign in to add a comment

Chrome no longer repeats header lines on printed pages

Reported by bgallo...@medianetsol.com, Feb 16 2017

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.87 Safari/537.36

Example URL:

Steps to reproduce the problem:
1. Open the attached reportheader.html in the latest version of Chrome Canary (58.0.3013.0)
2. Select Print Command
3. In the print preview the header line only shows on the first page and is no longer replicated on subsequent pages.

What is the expected behavior?
In the attached html document the header line should be repeated and appear at the top of each printed page.

What went wrong?
The header line only appears once.

Does it occur on multiple sites: Yes

Is it a problem with a plugin? No 

Did this work before? Yes   Version 55 worked correctly, version 56 was a regression where header lines are overlapping content, version 57 (on Canary) does not display header lines

Does this work in other browsers? Yes

Chrome version: 56.0.2924.87  Channel: stable
OS Version: 10.0
Flash Version: Shockwave Flash 24.0 r0

We would prefer that Chrome treats the header line as it previously has and how it currently functions in Microsoft Edge, Internet Explorer, Firefox and Safari where the header line is repeated at the top of all subsequent pages when printing.

This functionality is important to us as when printing reports in our application we use the header line inside the <thead> tag to display important student information on all pages of our report.

 

Comment 1 by hdodda@chromium.org, Feb 16 2017

Cc: hdodda@chromium.org
Labels: Needs-Feedback
@ bgalloway-- Thanks for the report .. Attachment is missing .. COuld you please attach the same again , that would help us in traiging the issue .

Thanks!
Yes, here is the attachment.  Thank you!
reportheader.html
6.7 KB View Download
Labels: -Needs-Feedback Needs-Triage-M56
Components: UI>Browser>PrintPreview
Labels: -Needs-Triage-M56 M-58 OS-Linux OS-Mac
Status: Untriaged (was: Unconfirmed)
Able to reproduce this issue on Mac 10.12.2, Win-10 and Ubuntu 14.04 using chrome reported version #56.0.2924.87 and latest canary #58.0.3014.0.

This is a non-regression issue as it is observed from M30 old builds. 

Hence, marking it as untriaged to get more inputs from dev team.

Thanks...!!
Components: -Blink Blink>Layout>Table
Cc: robhogan@chromium.org
Labels: Needs-Bisect
Owner: krajshree@chromium.org
krajshree: the reporter said it's a regression from 55. Will you please ignore M30 and start a bisect with 55?

robhogan: was this regression suspected and do we already have a bug open for it?
Blockedon: 656232
This is because the first tbody row doesn't fit on a single page. We don't repeat the header in such cases.

The spec has moved on and we will reinstate the behaviour where we repeate the headers/footers even if an entire row doesn't fit on the page:

See 

https://lists.w3.org/Archives/Public/www-style/2016Dec/0070.html
and
https://drafts.csswg.org/css-tables-3/#repeated-headers

My plan is to address this and the other updates to the spec in https://codereview.chromium.org/2584143003. So we can leave this open until that's resolved.

See also https://bugs.chromium.org/p/chromium/issues/detail?id=675453 where we stopped repeating the header in cases where the row straddles multiple pages pending spec clarification.

Comment 9 by e...@chromium.org, Feb 21 2017

Status: ExternalDependency (was: Untriaged)
Cc: -robhogan@chromium.org krajshree@chromium.org
Owner: robhogan@chromium.org

Comment 11 by ajha@chromium.org, Feb 23 2017

Labels: -Needs-Bisect
Removing the Needs-Bisect label as per C#7. Feel free to add it back if this still requires any bisect.
Status: Fixed (was: ExternalDependency)
Fixed by https://codereview.chromium.org/2882043002

Sign in to add a comment