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