Re: [PATCH] Duplicated screensaver entry in control center



On Sun, 2005-10-16 at 14:47 +0200, Sebastien Bacher wrote:
> Le vendredi 14 octobre 2005 à 12:58 +0200, Rodrigo Moya a écrit :
> > Hi
> > 
> > Attached patch was submitted to bugzilla a month ago. It makes the
> > control center not show the xscreensaver capplet icon if
> > gnome-screensaver is installed (all other gnome modules run g-s if
> > installed, if not, xscreensaver).
> > 
> > I think this should also get into the 2.12 branch. Ok?
> 
> That seems to be an hack to me.

yes, but this is what all other related GNOME modules (panel, gdm,
settings daemon) do it. Brian, gdm maintainer, suggested to use a config
entry (not in GConf because of gdm) to set the screensaver to use.

That is the correct solution IMO, but it has the difficulty of where to
put that setting.

>  Are we going to do code special case for
> every single feature provided by different applications? What happens to
> the menu entries? The patch is for the canvas but the panel menu still
> has the duplicate ...
> 
right, forgot about that one, will send another patch in a little while.
-- 
Rodrigo Moya <rodrigo novell com>




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