Re: svn commit: r367395 - /xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/XMLWhiteSpaceHandler.java

2006-01-31 Thread Luca Furini
Manuel Mall wrote: [cut] Hi Manuel! I was just wondering how we could proceed with the breaking / white space handling stuff we started discussing some time ago (if you have time and will to do it, obviously). Regards Luca

Re: svn commit: r367395 - /xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/XMLWhiteSpaceHandler.java

2006-01-13 Thread Andreas L Delmelle
On Jan 13, 2006, at 15:47, Manuel Mall wrote: 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 follo

Re: svn commit: r367395 - /xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/XMLWhiteSpaceHandler.java

2006-01-13 Thread Manuel Mall
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: > > > > > > without your patch I cannot really replicate this. When I run > > >

Re: svn commit: r367395 - /xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/XMLWhiteSpaceHandler.java

2006-01-12 Thread Manuel Mall
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: > > > > without your patch I cannot really replicate this. When I run > > through the debugger now I can see the space being given fro

Re: svn commit: r367395 - /xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/XMLWhiteSpaceHandler.java

2006-01-12 Thread Andreas L Delmelle
On Jan 12, 2006, at 01:14, Manuel Mall wrote: On Thu, 12 Jan 2006 05:34 am, Andreas L Delmelle wrote: 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

Re: svn commit: r367395 - /xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/XMLWhiteSpaceHandler.java

2006-01-12 Thread Chris Bowditch
Andreas L Delmelle wrote: On Jan 11, 2006, at 17:56, Andreas L Delmelle wrote: OK. So apparently, this has nothing to do with refinement white-space- handling, IIC. If I remove the border-* and padding-* properties, the trailing space area for the latter disappears...? Of course

Re: svn commit: r367395 - /xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/XMLWhiteSpaceHandler.java

2006-01-11 Thread Manuel Mall
On Thu, 12 Jan 2006 05:34 am, Andreas L Delmelle wrote: > On Jan 11, 2006, at 17:56, Andreas L Delmelle wrote: > > On Jan 10, 2006, at 14:02, Manuel Mall wrote: > >> I got the lastest svn version of fop and then commented out the > >> removeElementsForTrailingSpaces() method in LineLayoutManager as

Re: svn commit: r367395 - /xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/XMLWhiteSpaceHandler.java

2006-01-11 Thread Andreas L Delmelle
On Jan 11, 2006, at 17:56, Andreas L Delmelle wrote: On Jan 10, 2006, at 14:02, Manuel Mall wrote: I got the lastest svn version of fop and then commented out the removeElementsForTrailingSpaces() method in LineLayoutManager as theoretically your patch should make this unnecessary. However, we

Re: svn commit: r367395 - /xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/XMLWhiteSpaceHandler.java

2006-01-11 Thread Andreas L Delmelle
On Jan 10, 2006, at 14:02, Manuel Mall wrote: I got the lastest svn version of fop and then commented out the removeElementsForTrailingSpaces() method in LineLayoutManager as theoretically your patch should make this unnecessary. However, we get a erroneous trailing space in the block_white-spac

Re: svn commit: r367395 - /xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/XMLWhiteSpaceHandler.java

2006-01-10 Thread Manuel Mall
On Tue, 10 Jan 2006 05:24 am, Andreas L Delmelle wrote: > On Jan 9, 2006, at 22:20, [EMAIL PROTECTED] wrote: > > Author: adelmelle > > Date: Mon Jan 9 13:20:07 2006 > > New Revision: 367395 > > > > URL: http://svn.apache.org/viewcvs?rev=367395&view=rev > > OK folks, > > This should solve the 'trai

Re: svn commit: r367395 - /xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/XMLWhiteSpaceHandler.java

2006-01-09 Thread Andreas L Delmelle
On Jan 9, 2006, at 22:20, [EMAIL PROTECTED] wrote: Author: adelmelle Date: Mon Jan 9 13:20:07 2006 New Revision: 367395 URL: http://svn.apache.org/viewcvs?rev=367395&view=rev OK folks, This should solve the 'trailing white-space for trailing (nested) inlines' issue recently discussed ad n