Re: The Table menu patch; another try

[Matthias Clasen <maclas gmx de>]

Am Die, 2003-08-19 um 00.29 schrieb Matthias Clasen:
> Am Sam, 2003-08-16 um 23.20 schrieb Kristian Rietveld:
> > Hey,
> > 
> > Attached is a new revision of the table menu patch, which hopefully
> > fixes most (if not all) issues/comments raised. I tested it with
> > Matthias' patch to testgtk and my own silly testprogram, and it seems to
> > work fine. I hope I didn't screw up as badly as the two previous times
> > ...
> > 
> > The only issue I can still think of is LTR/RTL. I didn't look at it yet
> > since the other code in gtkmenu.c doesn't seem to support it either.
> > 
> 
> Kristian, I tried it, and it works much better now. I'm still seeing a
> little problem with the highlighting of the "e" in my spanning example,
> see the attached screenshot. And torn-off menus are not redrawn at all.

I figured out the "e" problem now. Line 2100 of gtkmenu.c should read

child_allocation.x = l * base_width;

instead of 

child_allocation.x = l * child_allocation.width;



Regards, Matthias