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

Reply via email to