Bug 361007

Summary: Changing Audio volume in Lock Screen not possible anymore
Product: [Plasma] kscreenlocker Reporter: Robby Engelmann <robby.engelmann>
Component: generalAssignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: bhush94, mgraesslin, olaf.the.lost.viking, robby.engelmann
Priority: NOR    
Version: 5.5.95   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In: 5.6.2
Sentry Crash Report:

Description Robby Engelmann 2016-03-26 10:49:33 UTC
Using Audio control volume special keys it is not possible anymore to change audio volume when screen is locked.
This occurs since I upgraded to plasma 5.6.

Reproducible: Always
Comment 1 OlafLostViking 2016-03-28 18:45:59 UTC
This is still valid for the final version of 5.6.0 (report was for 5.5.95).
Comment 2 Robby Engelmann 2016-03-28 20:57:12 UTC
I know, but latest version selectable is 5.5.95
Comment 3 Martin Flöser 2016-03-29 06:35:53 UTC
Do you also use Qt 5.6?
Comment 4 Robby Engelmann 2016-03-29 06:57:35 UTC
It is Qt 5.5.1 under openSUSE Tumbleweed.
Comment 5 OlafLostViking 2016-03-29 09:47:49 UTC
Qt 5.6.0, Archlinux AMD64
Comment 6 Martin Flöser 2016-04-04 12:40:30 UTC
possible patch: https://phabricator.kde.org/D1314
Comment 7 Martin Flöser 2016-04-04 16:49:36 UTC
Git commit e9984e21b798a0894e49dc74489a8fc7c2c2347d by Martin Gräßlin.
Committed on 04/04/2016 at 16:49.
Pushed by graesslin into branch 'master'.

Workaround problems with Qt::QueuedConnection

Summary:
For unknown reasons our signals with Qt::QueuedConnection are not
delivered with Qt 5.6 if the context object is this (KSldApp instance).

If we use a different context object (e.g. the sender) it works.

The lack of signals not working triggered quite a few bugs, like
* grace time not working
* global shortcuts not working
Related: bug 361008

Reviewers: #plasma

Subscribers: plasma-devel

Projects: #plasma

Differential Revision: https://phabricator.kde.org/D1314

M  +0    -3    autotests/ksldtest.cpp
M  +6    -2    ksldapp.cpp

http://commits.kde.org/kscreenlocker/e9984e21b798a0894e49dc74489a8fc7c2c2347d
Comment 8 Robby Engelmann 2016-04-04 16:54:31 UTC
Besides Qt5.6....I was submitting the bug report for Qt 5.5.1, thus the issue occurred not since qt5.6 but starting with plasma 5.6!
same for the other bug with the password delay!
Comment 9 Martin Flöser 2016-04-04 17:01:48 UTC
(In reply to Robby Engelmann from comment #8)
> Besides Qt5.6....I was submitting the bug report for Qt 5.5.1, thus the
> issue occurred not since qt5.6 but starting with plasma 5.6!
> same for the other bug with the password delay!

Thanks, that's good information and matches with the results of build.kde.org where it also failed with Qt 5.5.1. Though that leaves me even more puzzled.
Comment 10 Martin Flöser 2016-04-05 08:11:36 UTC
Git commit f0d2746615eb823770d14af1ca857918e3e7c399 by Martin Gräßlin.
Committed on 05/04/2016 at 08:09.
Pushed by graesslin into branch 'Plasma/5.6'.

Workaround problems with Qt::QueuedConnection

Summary:
For unknown reasons our signals with Qt::QueuedConnection are not
delivered with Qt 5.5.1 or later if the context object is this
(KSldApp instance).

If we use a different context object (e.g. the sender) it works.

The lack of signals not working triggered quite a few bugs, like
* grace time not working
* global shortcuts not working
Related: bug 361008
FIXED-IN: 5.6.2

Reviewers: #plasma

Subscribers: plasma-devel

Projects: #plasma

Differential Revision: https://phabricator.kde.org/D1314

M  +6    -2    ksldapp.cpp

http://commits.kde.org/kscreenlocker/f0d2746615eb823770d14af1ca857918e3e7c399