Bug 35534 - fo document output is placed in page margins
Summary: fo document output is placed in page margins
Status: CLOSED FIXED
Alias: None
Product: Fop - Now in Jira
Classification: Unclassified
Component: page-master/layout (show other bugs)
Version: trunk
Hardware: All All
: P2 major
Target Milestone: ---
Assignee: fop-dev
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-06-29 02:36 UTC by Nils Meier
Modified: 2012-04-01 06:49 UTC (History)
0 users



Attachments
input (95.87 KB, text/xml)
2005-06-29 02:37 UTC, Nils Meier
Details
output with svn code in branch trunk (14.14 KB, application/pdf)
2005-06-29 02:37 UTC, Nils Meier
Details
output with maint branch (as expected) (17.93 KB, application/pdf)
2005-06-29 02:37 UTC, Nils Meier
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Nils Meier 2005-06-29 02:36:32 UTC
See attachment 

fo.fo - the input
trunk.pdf - the output generated with svn trunk code
maint.pdf - the output generated with 0.20.5 (how it should look like)
Comment 1 Nils Meier 2005-06-29 02:37:15 UTC
Created attachment 15550 [details]
input
Comment 2 Nils Meier 2005-06-29 02:37:36 UTC
Created attachment 15551 [details]
output with svn code in branch trunk
Comment 3 Nils Meier 2005-06-29 02:37:55 UTC
Created attachment 15552 [details]
output with maint branch (as expected)
Comment 4 Jeremias Maerki 2005-06-29 09:12:27 UTC
Initial analysis: there seems to be a descrepancy between the reported element 
list and what the addAreas() stage is writing out to the page. I suspect that 
space-before is not handled properly in list-items. I'll investigate further.

BTW, Nils' example shows again that we have another point to attend to: Our 
current nominal text box is much bigger than it was for 0.20.5. I've seen this 
before and I think we're approaching the time when this needs to be fixed.
Comment 5 Jeremias Maerki 2005-06-29 09:41:23 UTC
It is as I suspected. ListItemLM currently doesn't generate the glues for 
space-before and space-after. I've written a test case:

https://svn.apache.org/repos/asf/xmlgraphics/fop/trunk/test/layoutengine/testca
ses/list4.xml

I'll try to look at this shortly.
Comment 6 Nils Meier 2005-07-04 05:30:12 UTC
Maybe I should also mention the following showing up on System.err

03.07.2005 23:22:12 org.apache.fop.fo.flow.Table bind::
The collapsing border model on an fo:table is currently not supported by FOP::
03.07.2005 23:22:12 org.apache.fop.fo.flow.Table bind::
Comment 7 Jeremias Maerki 2005-07-08 08:59:56 UTC
Nils,
the warning message was to be expected. The default of the "border-collapse" 
property is "collapse" and the implementation of the collapsing border model 
is incomplete. The warning is simply there to remind you not to expect nicely 
drawn borders in this mode. border-collapse="separate" works fine, though, but 
it obviously needs to be specified explicitely.
Comment 8 Nils Meier 2005-07-19 19:02:30 UTC
> the warning message was to be expected. The default of the "border-collapse" 
> property is "collapse" and the implementation of the collapsing border model 
> is incomplete. The warning is simply there to remind you not to expect nicely 
> drawn borders in this mode. border-collapse="separate" works fine, though, but 
> it obviously needs to be specified explicitely.

ok, thanks for the explanation. Problem is that docbook-xsl creates
tons of fo:table without applying a central property-set (there's one
but it's not used everywhere) where I could override border-collapse.

If the border-collapse=collapse won't be supported in the near future
maybe we could add a switch that allows to flip the default explicitly
/globally to 'separate'?

Thanks
Comment 9 Jeremias Maerki 2005-07-26 09:55:29 UTC
The situation is now improved here. The missing glues are now produced but if
there are shrink and stretch possibilities the spaces are currently not
adjusted, i.e. differing .minimum/.optimum/.maximum values on space-before and
space-after may produce undesirable results.

Changes:
http://svn.apache.org/viewcvs?rev=225133&view=rev
http://svn.apache.org/viewcvs?rev=225135&view=rev
http://svn.apache.org/viewcvs?rev=225147&view=rev
http://svn.apache.org/viewcvs?rev=225249&view=rev
http://svn.apache.org/viewcvs?rev=225250&view=rev
Comment 10 Pascal Sancho 2007-11-20 02:15:43 UTC
Fixed in FOP 0.94 and probably earlier versions
Comment 11 Glenn Adams 2012-04-01 06:49:33 UTC
batch transition pre-FOP1.0 resolved+fixed bugs to closed+fixed