Bug 342596

Summary: "Never" option in screenlocker KCM can't be discovered intuitively
Product: [Plasma] plasmashell Reporter: Elias Probst <mail>
Component: generalAssignee: David Edmundson <kde>
Status: RESOLVED FIXED    
Severity: normal CC: bshah, kbumicon, rdieter, thomas.pfeiffer
Priority: NOR Flags: bshah: Usability+
Version First Reported In: master   
Target Milestone: 1.0   
Platform: Other   
OS: Linux   
Latest Commit: http://commits.kde.org/plasma-workspace/65045cb50a68114799711f2b891eac1b0370287e Version Fixed In:
Sentry Crash Report:
Attachments: attachment-7282-0.html

Description Elias Probst 2015-01-07 18:49:52 UTC 
The "Never" option in screenlocker KCM can't be discovered intuitively.

One needs to scroll down to 0 in the "Lock screen after" field to get the option to never lock the screen - I just discovered this by accident but could have never guessed how to get this option otherwise.

IMHO there should be a "[x] Lock screen automatically" checkbox instead above the "Lock screen after" and "Require password after locking" options.

When the checkbox is disabled, those options should be greyed out.
Comment 1 Thomas Pfeiffer 2015-01-08 13:40:58 UTC 
You are right, and I must admit that I missed that during my original usability review.
I had voted for using a combobox instead of the number spinner as per Sebas' suggestion, but that wasn't implemented.
Now tzhat you mention it I also notice the logical inconsistency: A zero value for "Lock screen after" would logically mean "immediately" instead of "never".
I do agree that using a checkbox would be the best way to solve this.
Comment 2 David Edmundson 2015-03-06 12:11:11 UTC 
Git commit 65045cb50a68114799711f2b891eac1b0370287e by David Edmundson.
Committed on 04/03/2015 at 20:56.
Pushed by davidedmundson into branch 'master'.

Add separate checkbox for automatically lock screen

This improves usability over setting the timeout to 0.
Defaults behaviour is unchanged

REVIEW: 122598

M  +7    -2    ksmserver/screenlocker/kcfg/kscreenlockersettings.kcfg
M  +69   -56   ksmserver/screenlocker/kcm/kcm.ui
M  +2    -1    ksmserver/screenlocker/ksldapp.cpp

http://commits.kde.org/plasma-workspace/65045cb50a68114799711f2b891eac1b0370287e
Comment 3 Wolfgang Bauer 2015-03-29 09:46:20 UTC 
*** Bug 345633 has been marked as a duplicate of this bug. ***
Comment 4 Ki Bum Lee 2015-03-29 21:06:24 UTC 
Created attachment 91811 [details]
attachment-7282-0.html

It worked. Danke shoen!

Ki Lee

2015-03-29 5:46 GMT-04:00 Wolfgang Bauer <wbauer@tmo.at>:

> https://bugs.kde.org/show_bug.cgi?id=342596
>
> Wolfgang Bauer <wbauer@tmo.at> changed:
>
>            What    |Removed                     |Added
>
> ----------------------------------------------------------------------------
>                  CC|                            |kbumicon@gmail.com
>
> --- Comment #3 from Wolfgang Bauer <wbauer@tmo.at> ---
> *** Bug 345633 has been marked as a duplicate of this bug. ***
>
> --
> You are receiving this mail because:
> You are on the CC list for the bug.
>