Bug 100053

Summary:

page-break-inside gives descendant table cell extra height

Product:

WebKit

Reporter:

ahochhaus

Component:

Printing

Assignee:

Nobody <webkit-unassigned>

Status:

UNCONFIRMED

Severity:

Normal

CC:

ahochhaus, eric, webkit.review.bot

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

All

Attachments:

Description	Flags
minimal test case html file demonstrating extra table cell height issue when printing	none
example PDF showing extra table cell height (saved from Safari 6 print dialog)	none
image marking extra table cell height in red from previous PDF example	none
RenderBlock patch which avoids boosting the height of table cells.	none
Fix style - avoid braces for single line if statement	none

ahochhaus

Reported 2012-10-22 16:43:10 PDT

Created attachment 170020 [details] minimal test case html file demonstrating extra table cell height issue when printing When using "page-break-inside: avoid" in conjunction with a table, the table cells can be given extra height when the table lands on a page break boundary. See the attached html example for a full minimal test case which demonstrates the problem in WebKit nightly on Windows (r131444), Safari 6 Mac OS 10.7 and Chrome 22 on Debian Linux.

Attachments
minimal test case html file demonstrating extra table cell height issue when printing (920 bytes, text/html) 2012-10-22 16:43 PDT, ahochhaus	no flags	Details
example PDF showing extra table cell height (saved from Safari 6 print dialog) (21.55 KB, application/pdf) 2012-10-22 16:44 PDT, ahochhaus	no flags	Details
image marking extra table cell height in red from previous PDF example (63.06 KB, image/png) 2012-10-22 16:45 PDT, ahochhaus	no flags	Details
RenderBlock patch which avoids boosting the height of table cells. (635 bytes, text/plain) 2012-10-23 13:18 PDT, ahochhaus	no flags	Details
Fix style - avoid braces for single line if statement (625 bytes, text/plain) 2012-10-23 13:24 PDT, ahochhaus	no flags	Details
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.

ahochhaus

Comment 1 2012-10-22 16:44:49 PDT

Created attachment 170022 [details] example PDF showing extra table cell height (saved from Safari 6 print dialog)

ahochhaus

Comment 2 2012-10-22 16:45:58 PDT

Created attachment 170025 [details] image marking extra table cell height in red from previous PDF example

ahochhaus

Comment 3 2012-10-23 13:18:15 PDT

Created attachment 170214 [details] RenderBlock patch which avoids boosting the height of table cells.

WebKit Review Bot

Comment 4 2012-10-23 13:21:34 PDT

Attachment 170214 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/rendering/RenderBlock.cpp']" exit_code: 1 Source/WebCore/rendering/RenderBlock.cpp:7214: One line control clauses should not use braces. [whitespace/braces] [4] Total errors found: 1 in 1 files If any of these errors are false positives, please file a bug against check-webkit-style.

ahochhaus

Comment 5 2012-10-23 13:24:50 PDT

Created attachment 170217 [details] Fix style - avoid braces for single line if statement

ahochhaus

Comment 6 2012-10-29 18:26:05 PDT

Comment on attachment 170214 [details] RenderBlock patch which avoids boosting the height of table cells. Does not work.

ahochhaus

Comment 7 2012-10-29 18:27:37 PDT

Comment on attachment 170217 [details] Fix style - avoid braces for single line if statement Does not work. I think the correct solution is to do add pagination struts to tables as Dave Hyatt suggests in https://bugs.webkit.org/show_bug.cgi?id=82617#c9.

Note You need to log in before you can comment on or make changes to this bug.