After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 644059 - Screen locking vs. Screen blanking
Screen locking vs. Screen blanking
Status: RESOLVED FIXED
Product: gnome-control-center
Classification: Core
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: Control-Center Maintainers
Control-Center Maintainers
[gnome3-important]
Depends on:
Blocks:
 
 
Reported: 2011-03-07 00:00 UTC by Owen Taylor
Modified: 2011-03-17 05:48 UTC
See Also:
GNOME target: 3.0
GNOME version: ---



Description Owen Taylor 2011-03-07 00:00:39 UTC
Schema defaults:

 org.gnome.desktop.session idle-delay
   10 [minutes]
 org.gnome.settings-daemon.plugins.power sleep-display-[ac/battery]
   1800 [seconds]

So, that would make that we would blank the screen after 10 minutes when we go idle, then turn off the screen after 30 minutes. This makes no sense to me as defaults, it means that for 20 minutes we are sitting there with the date, the lock emblem and the name displayed on a lit-but-blank screen.

But the control center issue is that when you start the control center with the above settings you see:

  Brightness: [...]

    Turn off after 30 minutes

  Lock screen after: [Screen turns off]

It appears there is no code that actually sets the "Lock screen after" combo from GConf on startup, it just always end up with [Screen turns off].

I've only verified the behavior with 2.91.90, but the code and schemas look unchanged in git master.
Comment 1 Bastien Nocera 2011-03-07 18:02:54 UTC
(In reply to comment #0)
<snip>
> It appears there is no code that actually sets the "Lock screen after" combo
> from GConf on startup, it just always end up with [Screen turns off].
> 
> I've only verified the behavior with 2.91.90, but the code and schemas look
> unchanged in git master.

Reverting c805978234e2830a0fa898cbbf90c7fa6f5f3974 fixes the problem of the combo box never being set.

What was that all about Jon?
Comment 2 Bastien Nocera 2011-03-07 18:03:30 UTC
Owen, if the defaults need changing, file a bug against gnome-settings-daemon.
Comment 3 Owen Taylor 2011-03-07 18:04:47 UTC
(In reply to comment #2)
> Owen, if the defaults need changing, file a bug against gnome-settings-daemon.

I have no idea which of the two defaults, one of which is in g-s-d, one of which is in gsettings-desktop-schemas, needs changing.
Comment 4 Matthias Clasen 2011-03-11 17:43:21 UTC
<mclasen> mccann: two things. first, your rework of the timeout juggling for idle/lock/whatever timeouts does not quite work
<mccann> oh :(
<mclasen> I've fixed up some problems, but one issue remains
<mclasen> the problem is that we loose information by storing the lock delay as relative
<mclasen> 0 can mean two things there
<mclasen> either it is really 'whenever the screen turns off' ie a special case
<mclasen> or it is 'the lock delay happened to be smaller than the dpms timeout
<mclasen> so I'll make idle delay = lock delay and save a relative delay of 0
<mclasen> so if you set turn off to 10 and lock to 5
<mclasen> and restart the panel
<mclasen> it comes up with turn off == 10 and lock 'when the screen turns off'
Comment 5 William Jon McCann 2011-03-17 04:17:28 UTC
Ok, I've fixed up the defaults a bit and changed the units used to be consistent.  For testing purposes it is good to do:

gsettings reset org.gnome.desktop.screensaver lock-delay
gsettings reset org.gnome.desktop.session idle-delay
gsettings reset org.gnome.settings-daemon.plugins.power sleep-display-ac
gsettings reset org.gnome.settings-daemon.plugins.power sleep-display-battery
gsettings get org.gnome.settings-daemon.plugins.power sleep-display-battery

I'll into the rest next.
Comment 6 William Jon McCann 2011-03-17 05:48:58 UTC
Ok, I've changed it back to using relative lock times.  This means that you can't lock the screen before the backlight turns off.  But that's probably fine.  It is a little unclear that the lock times are relative but at least the "after screen turns off" item in the menu helps a bit.  I think it is the best we can do for now.  Going to mark this fixed.  Please open a new bug if any new issues arise.