Uploaded image for project: 'FOP'
  1. FOP
  2. FOP-1459

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

    Details

    • Type: Bug
    • Status: Closed
    • Resolution: Fixed
    • Affects Version/s: 0.94
    • Fix Version/s: None
    • Component/s: layout/unqualified
    • 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
        gadams Glenn Adams added a comment -

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

        Show
        gadams Glenn Adams added a comment - batch transition pre-FOP1.0 resolved+fixed bugs to closed+fixed
        Hide
        vhennebert Vincent Hennebert added a comment -

        Fixed in rev. 614566 of the Trunk.

        Show
        vhennebert Vincent Hennebert added a comment - Fixed in rev. 614566 of the Trunk.
        Hide
        vhennebert 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
        vhennebert 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
        guillaume.levrero@free.fr Guillaume Levrero added a comment -

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

        Show
        guillaume.levrero@free.fr Guillaume Levrero added a comment - Attachment BUG-table_with-KT.fo has been added with description: Table with keep-together
        Hide
        guillaume.levrero@free.fr Guillaume Levrero added a comment -

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

        Show
        guillaume.levrero@free.fr Guillaume Levrero added a comment - Attachment BUG-table_without-KT.fo has been added with description: Table without keep-together

          People

          • Assignee:
            fop-dev@xmlgraphics.apache.org fop-dev
            Reporter:
            guillaume.levrero@free.fr Guillaume Levrero
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development