D6443: [CurrentContainmentActionsModel] Add configurationChanged signal
Kai Uwe Broulik
noreply at phabricator.kde.org
Fri Jun 30 09:38:22 UTC 2017
broulik created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
This allows the config UI to know when to enable the Apply button.
Currently we only do that when the shortcut or actions themselves change but not their configuration.
TEST PLAN
Added a `Connections` to ConfigurationContainmentActions that triggers the config dialog's `configurationChanged()` and had the Apply button correctly enabled when changing a containment action's settings
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D6443
AFFECTED FILES
shell/currentcontainmentactionsmodel.cpp
shell/currentcontainmentactionsmodel.h
To: broulik, #plasma
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170630/b365327d/attachment.html>
More information about the Plasma-devel
mailing list