Bug 91043

Summary: When a table row height grows because of pagination, not all cells’ heights are adjusted
Product: WebKit Reporter: mitz
Component: TablesAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: eric, webkit.review.bot
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 91048    
Bug Blocks:    
Attachments:
Description Flags
Grow the row and all cells occurring in it as needed only after laying out all cells on the row sam: review+

mitz
Reported 2012-07-11 19:19:23 PDT
<rdar://problem/11794104> The fix for in bug 83595 in <http://trac.webkit.org/r113738> made table rows grow as necessary to fit cells that grow as a result of pagination. But it had two bad side effects: earlier cells on the row would not grow by the same amount, and later cells on the row would factor the existing growth into their intrinsic padding.
Attachments
Grow the row and all cells occurring in it as needed only after laying out all cells on the row (6.62 KB, patch)
2012-07-11 19:25 PDT, mitz 		sam: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 2012-07-11 19:25:37 PDT
Created attachment 151846 [details] Grow the row and all cells occurring in it as needed only after laying out all cells on the row
Sam Weinig
Comment 2 2012-07-11 19:34:54 PDT
Comment on attachment 151846 [details] Grow the row and all cells occurring in it as needed only after laying out all cells on the row View in context: https://bugs.webkit.org/attachment.cgi?id=151846&action=review > LayoutTests/fast/multicol/table-row-height-increase-expected.html:1 > +<meta name=viewport content="width=device-width"> Seems unnecessary. > LayoutTests/fast/multicol/table-row-height-increase.html:1 > +<meta name=viewport content="width=device-width"> Same here.
mitz
Comment 3 2012-07-11 19:39:13 PDT
Fixed in <http://trac.webkit.org/r122408>.
Note You need to log in before you can comment on or make changes to this bug.