Printing page breaks not correct with floated container
Reported by
andrewdi...@mso.net,
Mar 13 2017
|
|||||||
Issue descriptionUserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.87 Safari/537.36 Steps to reproduce the problem: 1) Create an HTML page with the following code: <html> <body> <div style="float:left;"> <div style="page-break-after:always;">T1</div> <div style="page-break-after:always;">T2</div> <div style="page-break-after:always;">T3</div> <div style="page-break-after:always;">T4</div> </div> </body> </html> 2) Open the HTML page in Chrome 3) Select to print the page and you will see in the print preview that the page breaks have been ignored. 4) If you remove the "float: left" from the container DIV, it will break it over 4 pages as expected. Please note, the code shown in step 1 works as expected, e.g. breaking over 4 pages, in Firefox and Safari. What is the expected behavior? For it to respect the page breaks. What went wrong? It didn't respect the page breaks. Did this work before? N/A Chrome version: 56.0.2924.87 Channel: n/a OS Version: OS X 10.12.3 Flash Version: There was previously a ticket for this issue #118627 , which was closed as "Won't Fix", with a comment saying it was now working.
,
Mar 14 2017
I don't think we support fragmenting floats. This is likely something that we won't fix until LayoutNG as it would be very hard in our current architecture.
,
Mar 14 2017
,
Mar 14 2017
,
Mar 17 2017
Is there a timeframe for the implementation of LayoutNG?
,
Apr 24 2017
We do support fragmenting floats, but we ignore any forced breaks inside them - i.e. only unforced breaks are supported. I'm not sure if there's a good reason for prohibiting forced breaks inside floats. We already do support parallel flows anyway.
,
Apr 24 2017
I find no reason to prohibit forced breaks inside floats, so I'll just fix it.
,
Apr 25 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/bbeb856815e147d66b8510a270a8c7faa444895d commit bbeb856815e147d66b8510a270a8c7faa444895d Author: mstensho <mstensho@opera.com> Date: Tue Apr 25 01:08:45 2017 Allow forced breaks inside floats. BUG= 700919 Review-Url: https://codereview.chromium.org/2840443003 Cr-Commit-Position: refs/heads/master@{#466848} [add] https://crrev.com/bbeb856815e147d66b8510a270a8c7faa444895d/third_party/WebKit/LayoutTests/fragmentation/forced-break-inside-float.html [modify] https://crrev.com/bbeb856815e147d66b8510a270a8c7faa444895d/third_party/WebKit/Source/core/layout/LayoutBox.cpp
,
Apr 25 2017
,
Jun 30 2017
|
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by thestig@chromium.org
, Mar 13 2017Status: Untriaged (was: Unconfirmed)