Qt Engine PDF - Table row overlaps repeating table header when row's height is greater than one page
Reported by
ncharr...@deversus.com,
Jul 19 2017
|
||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/59.0.3071.115 Safari/537.36 Steps to reproduce the problem: 1. Unzip table.zip 2. Open table.html in any modern version of Chrome. 3. File -> Print 4. Destination -> Save as PDF 5. Table row's content overlaps <thead> on second page. What is the expected behavior? Table row should not overlap table header. What went wrong? Repeating table header overlaps table row content Did this work before? No Does this work in other browsers? Yes Chrome version: 59.0.3071.115 Channel: stable OS Version: OS X 10.12.5 Flash Version: N/A This appears to be a problem with Qt Engine and how it handles PDF generation. Please let me know if there is another place I should report this bug. The key to reproducing the bug is having a single table row that spans more than one page. Both the CSS and HTML are valid, and I also am confident the use case is valid. Please let me know if I can be of any assistance. Thanks, Nathan Charrois nathancharrois@gmail.com
,
Jul 20 2017
Hi brajkumar, thanks for your response. I did make sure to test in Firefox before making this post and I'm not able to reproduce generating a PDF with the same steps (I've attached the Firefox pdf below, generated from 54 latest). I've also attached the one generated by Safari, although it appears to have different issue where the PDF header does not repeat at all. Please let me know if there's anything else I can help with.
,
Jul 20 2017
Thank you for providing more feedback. Adding requester "brajkumar@chromium.org" to the cc list and removing "Needs-Feedback" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 21 2017
Tested the issue #59 .0.3071.115 on Mac 10.12.5 and was unable to reproduce the issue as per the steps mentioned comment #0. Able to reproduce the issue #59 .0.3071.115 on Ubuntu 14.04. Find the screenshot. @ncharrois: Could you please find the attached screen cast and confirm Is that issue with Mac os or any? Thanks!!
,
Jul 21 2017
Hi @sandeepkumars, your screencast does not complete step #4. Presumably you are trying to demonstrate that this is an issue with the operating system and not Chrome? I am skeptical of this as other browsers don't have this issue (Firefox, Safari).
,
Jul 21 2017
As mention in my original bug report, the bug occurs at the time of PDF generation, not print preview.
,
Jul 21 2017
Thank you for providing more feedback. Adding requester "sandeepkumars@chromium.org" to the cc list and removing "Needs-Feedback" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 24 2017
Can some one from PDF team please look into this issue. Thanks!!
,
Jul 24 2017
halcanary@ is this something from the Skia side?
,
Jul 24 2017
I'm not seeing this on Windows, Mac, or Linux with 59.0.3071.115. Here's a sample table.pdf from "Save As PDF" on Linux. Did I do something wrong while trying to reproduce this bug? Also, can you try Chrome Canary and see the bug still happens there for you?
,
Jul 26 2017
,
Aug 2 2017
This was reproducing for me on M59 Chrome for Linux, but is no longer reproducing for me on 60.0.3112.78. I was not able to reproduce on Mac with 59.0.3071.115. I suspect that this was fixed by a change in M59. Unless someone can reproduce this for M60 I recommend closing this bug.
,
Aug 3 2017
rharrison: If you can repro with M59 locally, can you try: tools/bisect-builds.py -a linux64 -g 464641 -b 474897 --use-local-cache --verify-range in a Chromium checkout and see if you can figure out what fixed it?
,
Aug 8 2017
Hi guys, I can confirm this is fixed in Chrome 60 on OSX using the original steps to reproduce. Will confirm with Windows asap. Would also like to know what fixed it as I'm trying to ultimately have this fix backported to a library we're using that involves Qt Engine. Thanks a lot guys
,
Aug 10 2017
,
Oct 18 2017
This issue hasn't being fixed. Checkout this screenshot https://cl.ly/411B3C0h0p2C of last page "table.pdf" file. Using Version 61.0.3163.100 (Official Build) (64-bit) on OSX.
,
Oct 18 2017
Can confirm with Comment 17. Using the original steps to reproduce looks like the bug has regressed since 60. High Sierra 10.13 Chrome Version 61.0.3163.100 (Official Build) (64-bit) Also I think this issue was mistakenly marked as 'Wont fix'.
,
Oct 18 2017
Well, in comment 15, you said it doesn't happen anymore, so at that point, what more can we do at that point to fix the bug?
,
Oct 18 2017
If the bug was fixed in Chrome 60 the issue should have been marked as "Fixed". And seeing as multiple people have confirmed the bug, marking it "Wont fix" doesn't really make sense, does it?
,
Oct 18 2017
This is a common point of confusion. When we didn't actually take any action to fix the bug, then we didn't fix it, did we? Thus the resoltion is WontFix. If someone actually did something, then we mark it fixed. As I said in comment 10, I can't reproduce this, so I can't even confirm that this was broken in M59, fixed in M60, and broken again in M61. For all we know, the bug may be flaky in some way, and you got luck with M60. Thus the bug is currently open again, and we'll try to figure it out.
,
Nov 7 2017
This is a bug with PDF generation so removing the Internals>Plugin>PDF label. Assigning to halcanary@ to take a look.
,
Apr 3 2018
I am not able to reproduce at this time with Chrome 67.0.3381.0 (Official Build) dev (64-bit) on Linux.
,
Apr 24 2018
,
Apr 26 2018
Tested on latest Chrome #66.0.3359.117 and Canary #68.0.3408.0 and was not able to reproduce the issue. Attached the PDF file generated. ncharrois@ -- Could you please verify on latest builds and let us know your observations. Thanks!
,
Apr 26 2018
Thank you for providing more feedback. Adding the requester to the cc list. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Apr 26 2018
pnangunoori@ Yes, I can confirm using the original steps to reproduce that this has been fixed. Thanks for your efforts and everybody who participated in helping reproduce the bug.
,
Apr 27 2018
Closing this issue as the issue seems to be fixed. Thanks! |
||||||||||||||||
►
Sign in to add a comment |
||||||||||||||||
Comment 1 by brajkumar@chromium.org
, Jul 20 2017Labels: Needs-Feedback