On Fri, 13 Jan 2006 08:01 am, Manuel Mall wrote:
> On Fri, 13 Jan 2006 12:49 am, Andreas L Delmelle wrote:
> > On Jan 12, 2006, at 01:14, Manuel Mall wrote:
> > > On Thu, 12 Jan 2006 05:34 am, Andreas L Delmelle wrote:
> > > <snip />
> > > without your patch I cannot really replicate this. When I run
> > > through the debugger now I can see the space being given from the
> > > FO to the TextLayoutManager (look at the textarray in the TextLM
> > > constructor) therefore to me its still a refinement issue. But if
> > > your latest modifications fixes that then I would need that patch
> > > to further investigate.
> >
> > Try the diff in attach (modifications to XMLWhiteSpaceHandler and
> > LineLayoutManager)
>
> Applied the diff but it didn't change my observation in the debugger.
> The extra space you noticed is coming from the FO. Therefore IMO the
> patch is not quite doing what you would like it to do. I haven't
> looked at your code yet.
>
Andreas,
The following change in XMLWhiteSpaceHandler
&& (/*nonWhiteSpacePresent ||*/ nextChild != null))
seems to fix it. That is I just commented out the test for
nonWhiteSpacePresent. I do not fully understand the implications of
this change. I'll leave it up to you to further explore.
> Manuel
Manuel