DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT <http://issues.apache.org/bugzilla/show_bug.cgi?id=29025>. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE.
http://issues.apache.org/bugzilla/show_bug.cgi?id=29025 Document/LayoutStrategy consolidation Summary: Document/LayoutStrategy consolidation Product: Fop Version: 1.0dev Platform: Other OS/Version: Other Status: NEW Severity: Normal Priority: Other Component: page-master/layout AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] Team, The following patch incorporates the (rather simple) LayoutStrategy code into the apps.Document class, and does away with LayoutStrategy and LayoutManagerLS.java. Implementation of multiple layout strategies, should we ever be doing this, is still possible, and with roughly the same complexity, however by reimplementing and/or subclassing apps.Document instead. This simplifies the LayoutManager code and makes the tracing of Document creation a bit easier to follow, hopefully beneficial for other developers besides me. The motivation for this change is that currently I'm trying to implement space-before correctly in fo:static-content (as a digression, doesn't work on pages past the first, because I reset its value to zero for fo:flow, need to find a way to differentiate the layout between the two types) and some reduction of the number of dimensions of the source code makes it easier for me to understand the process. Anyway, if no complaints I'll commit this in a couple of days. Thanks, Glen