Re: Nautilus sidebar width and more



On Tue, 2007-03-13 at 19:24 +0100, Stefano Teso wrote:
> Hi,
> 
> > > [1] - Looks pretty ok, except I don't like the (FALSE ==
> > > open_in_new_window) idiom. Just do if (!foo) or if (foo) if its a
> > > boolean value. Also, it would be nicer if the code for activation was
> > > shared between the menu activation and row_activated_callback instead of
> > > duplicated.
> > 
> > Okay, I'll take care of those issues soon.
> 
> Done. Please take a look at
> 
> http://bugzilla.gnome.org/show_bug.cgi?id=415206

Commited. Thanks.

=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
 Alexander Larsson                                            Red Hat, Inc 
                   alexl redhat com    alla lysator liu se 
He's a time-tossed flyboy assassin with no name. She's a wealthy 
African-American lawyer with her own daytime radio talk show. They fight 
crime! 




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