Bug#600486: Bug#598132: gdm3: Should beep or permit to configure a beep

Josselin Mouette joss at debian.org
Tue Oct 19 14:17:45 UTC 2010


Le mardi 19 octobre 2010 à 14:24 +0200, Samuel Thibault a écrit :
> Samuel Thibault, le Tue 19 Oct 2010 14:07:54 +0200, a écrit :
> > Josselin Mouette, le Tue 19 Oct 2010 13:57:59 +0200, a écrit :
> > > Le mardi 19 octobre 2010 à 11:57 +0200, Samuel Thibault a écrit :
> > > > > Sorry but this is not going to happen. What use is there for such a
> > > > > configuration checkbox?
> > > > 
> > > > Err, well, for enabling/disabling it easily, just like other
> > > > accessibility features.
> > > 
> > > How is beeping in the *user session* an accessibility feature?
> > 
> > This is not about the user session, but about the Debian-gdm session,
> > used for gdm3.
> 
> Just to make sure it's clear: at the bottom of the gdm3 screen, you have
> an "accessibility" icon, which triggers an accessibility panel that my
> patch is modifying. This all happens in the Debian-gdm session.

Ah, my bad. I thought this code was in the simple-greeter itself.

In this case I don’t understand why you use 2 different GConf keys for
that effect. You should put as much as possible of the changes in g-s-d,
and just ship a different default for the greeter session - we can do so
in the configuration file.

I’d prefer if the beep was emitted by a new g-s-d plugin, but if done
this way we could do with using beep. In this case we shouldn’t forget
the correct X-GNOME-Autostart-Phase so that the beep only happens when
everything else is ready.

Cheers,
-- 
 .''`.
: :' :     “You would need to ask a lawyer if you don't know
`. `'       that a handshake of course makes a valid contract.”
  `-        --  J???rg Schilling






More information about the pkg-gnome-maintainers mailing list