load testcase in print preview see that the second page is empty one should be able to read last line. This is probably a dupe but I can't find it. I will attach soon.
It'll take me a bit to get to this -- my net connection is spotty.... :(
Comment on attachment 167924 [details] [diff] [review] revised patch >Index: nsTableFrame.cpp >+ aDesiredSize.height = borderPadding.bottom + GetCellSpacingY() + >+ lastChildReflowed->GetRect().YMost(); Why is this still adding in the cellspacing? I don't think it should... r+sr=bzbarsky with that fixed or a comment explaining why it's correct.
the row groups are inset at the top by a border padding and a cellspacing this is reflected in the YMost. But a table has the borderpadding bottom and another cellspacing below the table and this, as the cellspacing above the row group not included (see CSS 2.1 "Next is the layer containing the row groups. Each row group extends from the top left corner of its topmost cell in the first column to the bottom right corner of its bottommost cell in the last column."). So I believe the patch is correct.
Oh, if we're putting cellspacing below the last lowgroup then the patch is in fact correct...
fix checked in
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.