[Differential] [Request, 5 lines] D2325: Don't unconditionally emit buttonPressed on profile load
broulik (Kai Uwe Broulik)
noreply at phabricator.kde.org
Mon Aug 1 14:42:08 UTC 2016
broulik created this revision.
broulik added reviewers: Plasma, afiestas.
broulik set the repository for this revision to rPOWERDEVIL Powerdevil.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
A button press (technically the "lid closed" event is also a button press) should always be some consicious action triggered by the user. By emitting this signal unconditionally on profile load (ie. whenever changing an activity or (un)plugging AC) the session might always lock or suspend when this happens.
If and only if a profile would really like to take some action or set some state depending on the lid state on profile load - which none do, the handle button press action should only react on explicit button presses anyway - it can still ask the backend for isLidClosed in its onProfileLoad method.
BUG: 366125
FIXED-IN: 5.7.4
TEST PLAN
Closed the lid, still suspends. I can't find any evidence about the reason this was put in there in the first place.
REPOSITORY
rPOWERDEVIL Powerdevil
REVISION DETAIL
https://phabricator.kde.org/D2325
AFFECTED FILES
daemon/powerdevilcore.cpp
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: broulik, #plasma, afiestas
Cc: plasma-devel, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160801/b29b50e0/attachment.html>
More information about the Plasma-devel
mailing list