Fop
  1. Fop
  2. FOP-1459

keep-together breaks table on 2 pages. without keep-togetther table on 1 page.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Resolution: Fixed
    • Affects Version/s: 0.94
    • Fix Version/s: None
    • Component/s: page-master/layout
    • Labels:
      None
    • Environment:
      Operating System: Windows 2000
      Platform: PC
    • External issue ID:
      43934

      Description

      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

      1. BUG-table_with-KT.fo
        1 kB
        Guillaume Levrero
      2. BUG-table_without-KT.fo
        1 kB
        Guillaume Levrero

        Activity

        Hide
        Guillaume Levrero added a comment -

        Attachment BUG-table_without-KT.fo has been added with description: Table without keep-together

        Show
        Guillaume Levrero added a comment - Attachment BUG-table_without-KT.fo has been added with description: Table without keep-together
        Hide
        Guillaume Levrero added a comment -

        Attachment BUG-table_with-KT.fo has been added with description: Table with keep-together

        Show
        Guillaume Levrero added a comment - Attachment BUG-table_with-KT.fo has been added with description: Table with keep-together
        Hide
        Vincent Hennebert added a comment -

        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.

        Show
        Vincent Hennebert added a comment - 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.
        Hide
        Vincent Hennebert added a comment -

        Fixed in rev. 614566 of the Trunk.

        Show
        Vincent Hennebert added a comment - Fixed in rev. 614566 of the Trunk.
        Hide
        Glenn Adams added a comment -

        batch transition pre-FOP1.0 resolved+fixed bugs to closed+fixed

        Show
        Glenn Adams added a comment - batch transition pre-FOP1.0 resolved+fixed bugs to closed+fixed

          People

          • Assignee:
            fop-dev
            Reporter:
            Guillaume Levrero
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development