On Mon, 2006-06-19 at 11:21 +0200, Jean-Marc Lasgouttes wrote: > >>>>> "Martin" == Martin Vermeer <[EMAIL PROTECTED]> writes: > > >> Done. I propose this for trunk and 1.4 (Jean-Marc?), as it fixes a > >> rendering bug. > > Martin> Committed for trunk. Jean-Marc? > > OK.
No. I found a better way to do this. See attached. In my newfound tradition of making everything simpler :-) If you paint the whole tabular background, you don't have to figure out what cell you're in... Please give it some exercise before the go-ahead for 1.4. I would also like to replace my 1.5 patch with this. - Martin
Index: insettabular.C =================================================================== --- insettabular.C (revision 14143) +++ insettabular.C (working copy) @@ -332,6 +332,15 @@ setPosCache(pi, x, y); LCursor & cur = pi.base.bv->cursor(); + + x += scx_ + ADD_TO_TABULAR_WIDTH; + + // Paint background of current tabular + int const w = tabular.getWidthOfTabular(); + int const h = tabular.getHeightOfTabular(); + int yy = y - tabular.getAscentOfRow(0); + pi.pain.fillRectangle(x, yy, w, h, backgroundColor()); + if (!cur.selection()) return; if (!ptr_cmp(&cur.inset(), this)) @@ -339,7 +348,6 @@ //resetPos(cur); - x += scx_ + ADD_TO_TABULAR_WIDTH; if (tablemode(cur)) { row_type rs, re;
signature.asc
Description: This is a digitally signed message part