Bug 43934 - keep-together breaks table on 2 pages. without keep-togetther table on 1 page.
Summary: keep-together breaks table on 2 pages. without keep-togetther table on 1 page.
Status: CLOSED FIXED
Alias: None
Product: Fop - Now in Jira
Classification: Unclassified
Component: page-master/layout (show other bugs)
Version: 0.94
Hardware: PC Windows 2000
: P2 normal
Target Milestone: ---
Assignee: fop-dev
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-11-22 01:35 UTC by Guillaume Levrero
Modified: 2012-04-01 07:01 UTC (History)
0 users


Attachments
Table without keep-together (1.33 KB, text/plain)
2007-11-22 01:36 UTC, Guillaume Levrero 		Details
Table with keep-together (1.32 KB, text/plain)
2007-11-22 01:36 UTC, Guillaume Levrero 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Guillaume Levrero 2007-11-22 01:35:00 UTC 
Rendering a simple table (1 column 2 rows) with different table-cell padding-top.

Without keep-together on the table-row, the flow is displayed on one page
With keep-together on the table-row, the table is broken over two pages.

Guillaume
Comment 1 Guillaume Levrero 2007-11-22 01:36:17 UTC 
Created attachment 21174 [details]
Table without keep-together
Comment 2 Guillaume Levrero 2007-11-22 01:36:41 UTC 
Created attachment 21175 [details]
Table with keep-together
Comment 3 Vincent Hennebert 2007-11-22 03:40:49 UTC 
Hi Guillaume,

Thanks for the bug report and the simple examples.

Details: in ActiveCell.java, when keep-together is set on table-row, a single
box is created for the whole cell (makeBoxForWholeRow), taking the height of the
row as its length. But getNextStep returns the size of the box plus the
paddings, which are already taken into account in the box. That leads to wrong
element generation. In the given examples, the generated penalty has a non-zero
value which makes the corresponding break feasible and breaks the table.
Comment 4 Vincent Hennebert 2008-01-23 07:23:54 UTC 
Fixed in rev. 614566 of the Trunk.
Comment 5 Glenn Adams 2012-04-01 07:01:34 UTC 
batch transition pre-FOP1.0 resolved+fixed bugs to closed+fixed