Re: gtkmenu.c - attachment issues ...
- From: "Matthias Clasen" <maclas gmx de>
- To: Alexander Larsson <alexl redhat com>
- Cc: michael ximian com, gtk-devel-list gnome org
- Subject: Re: gtkmenu.c - attachment issues ...
- Date: Thu, 22 Jan 2004 13:52:34 +0100 (MET)
Ouch, appears there is a simple thinko in my version of Michaels
gtk_menu_do_remove(). I'm setting priv->columns to zero before the loop in
order to recalculate it, yet expect to check priv->columns == 1 in order to
recognize regular menus in the loop. You could try
gboolean single_column = priv->columns == 1 ? TRUE : FALSE;
before the resetting of priv->columns, and replace the priv->columns == 1
and see if that fixes it for you.
> On Thu, 2004-01-22 at 12:09, Matthias Clasen wrote:
> > > On Wed, 2004-01-14 at 18:17, Michael Meeks wrote:
> > > > Hi guys,
> > > >
> > > > So - the libbonoboui code relies pretty heavily on the
> > > > gtk_container_get_children list (with some caveats) being in-sync
> > > > the visual list of widgets.
> > > >
> > > > There is a particularly pernicious bug in nautilus whereby your
> > > > and particularly / separators get horribly jumbled up. After some
> > > > extremely time-consuming debugging this was found to be down to the
> > > > gtkmenu.c 'attachment' child properties which were getting pretty
> > > > mangled; eg. walking the gtk_container_get_child list in order I was
> > > > getting ~20 items with attach properties of ~30 or so, and a totally
> > > > jumbled list.
> > > >
> > > > The attached patch shifts items down to fill voids created on
> removal -
> > > > effectively the converse of the insertion process; this fixes all
> > > > nautilus problems I could see, and would appear to make some sense.
> > > > also eliminated one _container_get_child_properties in the
> > > > insert at least to make it slightly more efficient.
> > > >
> > > > Comments or may I commit ?
> > >
> > > Did anyone comment on this? Is it in bugzilla?
> > > This bug causes some bad regressions in nautilus.
> > >
> > Should be fixed in CVS now. Are you still seeing problems ?
> Yeah, i'm still seing the separator reordering on view switch in
> Michael, shouldn't this be fixed now? With latest gtk+ and libbonoboui i
> still get it.
> Alexander Larsson Red Hat, Inc
> alexl redhat com alla lysator liu se
> He's an ungodly Jewish farmboy whom everyone believes is mad. She's a
> sharp-shooting blonde safe cracker in the witness protection program. They
> fight crime!
> gtk-devel-list mailing list
> gtk-devel-list gnome org
+++ GMX - die erste Adresse für Mail, Message, More +++
Bis 31.1.: TopMail + Digicam für nur 29 EUR http://www.gmx.net/topmail
] [Thread Prev