Issue metadata
Sign in to add a comment
|
Table headers overwrite body text on printed output
Reported by
maddalo...@gmail.com,
Oct 4 2016
|
||||||||||||||||||||||||
Issue description
Chrome Version : Version 53.0.2785.143 (64-bit)
URLs (if applicable) :N/A
Other browsers tested:
Add OK or FAIL, along with the version, after other browsers where you
have tested this issue:
Safari:OK
Firefox:OK
IE:OK
What steps will reproduce the problem?
(1) Open the attached StandardReports_BadHeader.htm in the latest version of Chrome
(2) Do File -> Print
(3) See in the print preview how the header line is replicated on all pages and on pages and later the header is overwriting body text.
What is the expected result? I would prefer the original Chrome behavior of having the header line not printed on subsequent pages. Alternatively I would like the header line justfied at the top of the page and not over the body text.
What happens instead? The header row gets re-printed at the top of each page overwriting body text.
Please provide any additional information below. Attach a screenshot if
possible.
See the PDF for an example. This occurs in printing and preview printing of HTML pages. In our reports a <CAPTION><div...></Caption> precedes the <thead> on page one where they show up correctly. Until recently, the thead row was not printed on subsequent pages. Now the thead is printed on subsequent pages but appears to be offset from the top of the page similar to the first page where it was under the caption. This pushes the thead row down into the body rows. Is there a way to keep the thead row from showing up on subsequent pages? If not, your most expeditious attention to fixing this would be greatly appreciated since it is breaking reporting for our customers in our application on Chrome. This occurs on Mac and Windows.
MS Edge shows the header on every page correctly formatted.
Mac/FireFox shows the header on every page correctly formatted
MS IE 11 does not show the header on every page.
,
Oct 5 2016
Able to reproduce this issue on Windows 10, Mac 10.12 and Ubuntu 14.04 on latest chrome Stable version 53.0.2785.143. Issue is broken in M53. Below are the bisect details for the same: Bisect Info: =========== Good Build : 53.0.2758.0 , Revision Range (397861) Bad Build : 53.0.2759.0 , Revision Range (397936) Change Log: ----------- https://chromium.googlesource.com/chromium/src/+log/25966f2863c58fb04faa48986150c363d30be6fa..7737b321ccfb843469dd625bb01dadda17a3a66f From the above CL suspecting the below change Review URL:https://codereview.chromium.org/2021703002 robhogan @- Could please look into this issue, if it's released to your change? if not could you please help us to reassign this issue to the right owner. Thanks.!
,
Oct 5 2016
,
Oct 12 2016
,
Oct 14 2016
Issue 655098 has been merged into this issue.
,
Oct 19 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/42151f9c0132a60a7be3b1793295a1966f532ca6 commit 42151f9c0132a60a7be3b1793295a1966f532ca6 Author: robhogan <robhogan@gmail.com> Date: Wed Oct 19 18:08:31 2016 Repeating header groups should align properly when captions are present BUG= 652792 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Review-Url: https://chromiumcodereview.appspot.com/2422953003 Cr-Commit-Position: refs/heads/master@{#426243} [add] https://crrev.com/42151f9c0132a60a7be3b1793295a1966f532ca6/third_party/WebKit/LayoutTests/fragmentation/single-line-cells-repeating-thead-with-caption-expected.html [add] https://crrev.com/42151f9c0132a60a7be3b1793295a1966f532ca6/third_party/WebKit/LayoutTests/fragmentation/single-line-cells-repeating-thead-with-caption.html [add] https://crrev.com/42151f9c0132a60a7be3b1793295a1966f532ca6/third_party/WebKit/LayoutTests/fragmentation/single-line-cells-repeating-thead-with-two-captions-expected.html [add] https://crrev.com/42151f9c0132a60a7be3b1793295a1966f532ca6/third_party/WebKit/LayoutTests/fragmentation/single-line-cells-repeating-thead-with-two-captions.html [modify] https://crrev.com/42151f9c0132a60a7be3b1793295a1966f532ca6/third_party/WebKit/Source/core/paint/TableSectionPainter.cpp
,
Oct 26 2016
Hi I have a couple questions. Is this a distinct issue from Issue 642814 ? If so should it perhaps be un-merged? Is there any estimate available for when this issue with the caption causing the overwrite might be addressed? Thanks!
,
Oct 26 2016
It is currently unmerged, as the Status is "Assigned" and not "Duplicate". The Merged line is a weird artifact left over from when it was marked as a duplicate.
,
Oct 27 2016
,
Oct 27 2016
@maddalozzo - this is fixed in the latest canary builds and will be fixed in the M55 release.
,
Oct 27 2016
Your change meets the bar and is auto-approved for M55 (branch: 2883)
,
Oct 27 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8ec55a26615743c726e3c56bccc1eea659f33c53 commit 8ec55a26615743c726e3c56bccc1eea659f33c53 Author: Robert Hogan <robhogan@gmail.com> Date: Thu Oct 27 08:45:31 2016 Repeating header groups should align properly when captions are present BUG= 652792 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Review-Url: https://chromiumcodereview.appspot.com/2422953003 Cr-Commit-Position: refs/heads/master@{#426243} (cherry picked from commit 42151f9c0132a60a7be3b1793295a1966f532ca6) Review URL: https://codereview.chromium.org/2453023003 . Cr-Commit-Position: refs/branch-heads/2883@{#331} Cr-Branched-From: 614d31daee2f61b0180df403a8ad43f20b9f6dd7-refs/heads/master@{#423768} [add] https://crrev.com/8ec55a26615743c726e3c56bccc1eea659f33c53/third_party/WebKit/LayoutTests/fragmentation/single-line-cells-repeating-thead-with-caption-expected.html [add] https://crrev.com/8ec55a26615743c726e3c56bccc1eea659f33c53/third_party/WebKit/LayoutTests/fragmentation/single-line-cells-repeating-thead-with-caption.html [add] https://crrev.com/8ec55a26615743c726e3c56bccc1eea659f33c53/third_party/WebKit/LayoutTests/fragmentation/single-line-cells-repeating-thead-with-two-captions-expected.html [add] https://crrev.com/8ec55a26615743c726e3c56bccc1eea659f33c53/third_party/WebKit/LayoutTests/fragmentation/single-line-cells-repeating-thead-with-two-captions.html [modify] https://crrev.com/8ec55a26615743c726e3c56bccc1eea659f33c53/third_party/WebKit/Source/core/paint/TableSectionPainter.cpp
,
Oct 27 2016
@robhogan Works for me on the canary build tested on MacOS El Capitan 10.11.6. Thanks!
,
Oct 27 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8ec55a26615743c726e3c56bccc1eea659f33c53 commit 8ec55a26615743c726e3c56bccc1eea659f33c53 Author: Robert Hogan <robhogan@gmail.com> Date: Thu Oct 27 08:45:31 2016 Repeating header groups should align properly when captions are present BUG= 652792 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Review-Url: https://chromiumcodereview.appspot.com/2422953003 Cr-Commit-Position: refs/heads/master@{#426243} (cherry picked from commit 42151f9c0132a60a7be3b1793295a1966f532ca6) Review URL: https://codereview.chromium.org/2453023003 . Cr-Commit-Position: refs/branch-heads/2883@{#331} Cr-Branched-From: 614d31daee2f61b0180df403a8ad43f20b9f6dd7-refs/heads/master@{#423768} [add] https://crrev.com/8ec55a26615743c726e3c56bccc1eea659f33c53/third_party/WebKit/LayoutTests/fragmentation/single-line-cells-repeating-thead-with-caption-expected.html [add] https://crrev.com/8ec55a26615743c726e3c56bccc1eea659f33c53/third_party/WebKit/LayoutTests/fragmentation/single-line-cells-repeating-thead-with-caption.html [add] https://crrev.com/8ec55a26615743c726e3c56bccc1eea659f33c53/third_party/WebKit/LayoutTests/fragmentation/single-line-cells-repeating-thead-with-two-captions-expected.html [add] https://crrev.com/8ec55a26615743c726e3c56bccc1eea659f33c53/third_party/WebKit/LayoutTests/fragmentation/single-line-cells-repeating-thead-with-two-captions.html [modify] https://crrev.com/8ec55a26615743c726e3c56bccc1eea659f33c53/third_party/WebKit/Source/core/paint/TableSectionPainter.cpp
,
Nov 2 2016
,
Nov 4 2016
[Automated comment] removing mislabelled merge-merged-2840 |
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by maddalo...@gmail.com
, Oct 4 2016