On Jul 18, 2007, at 20:06, [EMAIL PROTECTED] wrote:

Hi Manuel & others

Author: manuel
Date: Wed Jul 18 11:06:09 2007
New Revision: 557347

URL: http://svn.apache.org/viewvc?view=rev&rev=557347
Log:
Added support for keep-togther.within-line="always"

Cool to see this implemented so quickly!
On the other hand, I do have a few (non-blocking) comments.

There is no real support for inline-level elements yet, as your adaptation of the testcase demonstrates: in one case the fo:inline is removed, in the other it is substituted with an fo:wrapper. IMO, in the near future, keeps on other inline-level FOs should be reactivated as well. I was thinking of reactivating the related properties on fo.flow.InlineLevel...

I had also been thinking in the direction of letting the TextLM deal with this, but further considerations made me think this was suboptimal. Although this approach works perfectly for a value of "always", it could(?) become rather messy when looking at integer-keeps.

My last ideas on the subject went in the direction of letting the inline-level LM accumulate the regular sequence, and, for a value of "always", wrap it in a single box. If the keep-constraint is not absolute, the TextLM's parent could still do something similar for the individual elements, or something like: surround the entire sequence by two penalties, and insert penalties in the sequence but with significantly higher/lower values.

WDYT?


Cheers

Andreas

Reply via email to