[jira] [Commented] (FOP-1488) [PATCH] orphans/widows not respected in some cases

2015-05-07 Thread Andreas L. Delmelle (JIRA)

[ 
https://issues.apache.org/jira/browse/FOP-1488?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14532875#comment-14532875
 ] 

Andreas L. Delmelle commented on FOP-1488:
--

A very interesting question... 
If there is an image inserted, that would always be as an inline area, which is 
taken into account when calculating the height of the line area. 
Strictly speaking, I am inclined to say that from the spec point of view, it 
would indeed be non-compliant to treat images any differently than text or 
characters in this respect.

Having said that, the issue is actually a consequence of FOP being more 
compliant than before. It explains why I never simply committed the whole 
bunch, since it would be a rather significant change that quite some users may 
see as 'breaking' their existing cases. 
See also the patch with changes required to the layout tests... I think this 
definitely needs to be mentioned very, very clearly and prominently in the 
release documentation, to fend off a possible plethora of questions about this 
change in behaviour. 

Not sure if the workaround I used for some of the tests can be applied in your 
case, i.e. setting the orphans/widows values to 1 on the blocks in question 
would be sufficient to revert to the old behaviour. Both properties are 
inherited, so can be set on the flow and will be inherited by all descendant 
blocks, in case it needs to apply to a whole document.
Another possibility, but I have to test that one myself, could be to add empty 
blocks before and/or after each image, and make sure those blocks have a very 
small font-size or line-height. In that case, one image plus the preceding or 
following 'line' would satisfy the default value of 2 for both properties, 
even though the second line is invisible.

If neither of the above is feasible in those cases, I think the most 
appropriate way to accommodate that requirement would be through a config 
parameter or maybe an extension attribute.

 [PATCH] orphans/widows not respected in some cases
 --

 Key: FOP-1488
 URL: https://issues.apache.org/jira/browse/FOP-1488
 Project: Fop
  Issue Type: Bug
  Components: unqualified
Affects Versions: trunk
 Environment: Operating System: All
 Platform: All
Reporter: Andrew McFarland
 Attachments: FOP-1488-code.patch, FOP-1488-test.patch, b44328.patch, 
 b44328.patch, b44328.patch, b44328.patch, b44328.patch, b44328.patch, 
 b44328.patch, b44328_test.patch, block_orphans_widows.fo, 
 block_orphans_widows.fo, block_orphans_widows.fo, block_orphans_widows.fo, 
 block_orphans_widows.fo, widow.fo


 When I process the following fo, I get a PDF with a one-line widow at the 
 start
 of the second page, even though widows for that fo:block is set to 4.
 ?xml version=1.0 encoding=ISO-8859-1?
 fo:root xmlns:fo=http://www.w3.org/1999/XSL/Format;
 fo:layout-master-set
   fo:simple-page-master master-name=A4
 fo:region-body /
   /fo:simple-page-master
 /fo:layout-master-set
 fo:page-sequence master-reference=A4
   fo:flow flow-name=xsl-region-body
 fo:blockParagraph/fo:block
 fo:blockParagraph/fo:block
 fo:blockParagraph/fo:block
 fo:blockParagraph/fo:block
 fo:blockParagraph/fo:block
 fo:blockParagraph/fo:block
 fo:blockParagraph/fo:block
 fo:blockParagraph/fo:block
 fo:blockParagraph/fo:block
 fo:blockParagraph/fo:block
 fo:blockParagraph/fo:block
 fo:blockParagraph/fo:block
 fo:blockParagraph/fo:block
 fo:block widows=4 linefeed-treatment=preserve line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 /fo:block
   /fo:flow
 /fo:page-sequence
 /fo:root



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (FOP-1488) [PATCH] orphans/widows not respected in some cases

2015-05-06 Thread Matthias Reischenbacher (JIRA)

[ 
https://issues.apache.org/jira/browse/FOP-1488?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14530408#comment-14530408
 ] 

Matthias Reischenbacher commented on FOP-1488:
--

Thanks a lot Andreas for updating your patch. I updated my fop code base some 
weeks ago and had difficulties correctly applying your changes.
This patch has been very useful for my clients, the only minor thing which 
still needs to be addressed, is the line counting when images are used. If 
there are several images, one after each other, they are all hold together if a 
high orphans/widows value is used, because each image is counted as one line. 
If the images are very high, it can easily happen that the images overflow the 
page. Would it be correct (from a xsl-fo standard's point of view) to handle 
the line counting for images differently?

 [PATCH] orphans/widows not respected in some cases
 --

 Key: FOP-1488
 URL: https://issues.apache.org/jira/browse/FOP-1488
 Project: Fop
  Issue Type: Bug
  Components: unqualified
Affects Versions: trunk
 Environment: Operating System: All
 Platform: All
Reporter: Andrew McFarland
 Attachments: FOP-1488-code.patch, FOP-1488-test.patch, b44328.patch, 
 b44328.patch, b44328.patch, b44328.patch, b44328.patch, b44328.patch, 
 b44328.patch, b44328_test.patch, block_orphans_widows.fo, 
 block_orphans_widows.fo, block_orphans_widows.fo, block_orphans_widows.fo, 
 block_orphans_widows.fo, widow.fo


 When I process the following fo, I get a PDF with a one-line widow at the 
 start
 of the second page, even though widows for that fo:block is set to 4.
 ?xml version=1.0 encoding=ISO-8859-1?
 fo:root xmlns:fo=http://www.w3.org/1999/XSL/Format;
 fo:layout-master-set
   fo:simple-page-master master-name=A4
 fo:region-body /
   /fo:simple-page-master
 /fo:layout-master-set
 fo:page-sequence master-reference=A4
   fo:flow flow-name=xsl-region-body
 fo:blockParagraph/fo:block
 fo:blockParagraph/fo:block
 fo:blockParagraph/fo:block
 fo:blockParagraph/fo:block
 fo:blockParagraph/fo:block
 fo:blockParagraph/fo:block
 fo:blockParagraph/fo:block
 fo:blockParagraph/fo:block
 fo:blockParagraph/fo:block
 fo:blockParagraph/fo:block
 fo:blockParagraph/fo:block
 fo:blockParagraph/fo:block
 fo:blockParagraph/fo:block
 fo:block widows=4 linefeed-treatment=preserve line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 line
 /fo:block
   /fo:flow
 /fo:page-sequence
 /fo:root



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)