On Sun, Oct 12, 2003 at 09:48:59AM +0000, Angus Leeming spake thusly:

> >     584             InsetOld * inset = pit->getInset(z);
> >     585             if (inset->breakLineBefore())  // bool to be
> >     impl
> >     586                 // suppress filling of line
> >     587                 f = 0;
> >     588         }
> > 
> > Completely untested, just given here to illustrate my thinking...
> 
> Why not call breakLineBefore() display() and break the line after too? 
> Then there would be no need to fudge the width. The width is needed 
> as-is when trying to render a pixmap image on the screen I think...

Hmmm, not so quickly. What is called here 'breakLineBefore()' actually
does not do that, but rather should be called 'noFillLineBefore()'
(...and with small changes, the above actually works for me!)

Breaking the line after would be an entirely different story again. As
Jean-Marc pointed out, isLineSeparator should not be used for this as
it only signals that a break *may* occur after this inset.

BTW where would you test for your display() and cause the post-inset
line break? Perhaps after this (text.C)?

    697     for (; i < last; ++i) {
    698         if (pit->isNewline(i)) {
    699             point = i;
    700             break;
    701         }

Perhaps worth a try.
 
> -- 
> Angus

- Martin 

Attachment: pgp00000.pgp
Description: PGP signature

Reply via email to