Bug 361007 - Changing Audio volume in Lock Screen not possible anymore
Summary: Changing Audio volume in Lock Screen not possible anymore
Status: RESOLVED FIXED
Alias: None
Product: kscreenlocker
Classification: Plasma
Component: general (show other bugs)
Version: 5.5.95
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-03-26 10:49 UTC by Robby Engelmann
Modified: 2016-04-05 08:11 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.6.2
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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