Re: gtkmenu.c - attachment issues ...



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 with
> > > the visual list of widgets.
> > > 
> > > 	There is a particularly pernicious bug in nautilus whereby your items
> > > and particularly / separators get horribly jumbled up. After some
> > > extremely time-consuming debugging this was found to be down to the new
> > > 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 the
> > > nautilus problems I could see, and would appear to make some sense. I
> > > also eliminated one _container_get_child_properties in the equivalent
> > > 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
Nautilus. 

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! 




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]