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;

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to