[Differential] [Request, 4 lines] D2673: [AppletConfiguration] Don't call saveConfig() twice

broulik (Kai Uwe Broulik) noreply at phabricator.kde.org
Tue Sep 6 09:51:31 UTC 2016


broulik created this revision.
broulik added a reviewer: Plasma.
broulik set the repository for this revision to rPLASMADESKTOP Plasma Desktop.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  root.saveConfig() already calls saveConfig() on the current page, no need to explicitly do that in the apply button.

TEST PLAN
  No longer saves twice. Still saves once

REPOSITORY
  rPLASMADESKTOP Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D2673

AFFECTED FILES
  desktoppackage/contents/configuration/AppletConfiguration.qml

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: broulik, #plasma
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160906/faf2ba65/attachment.html>


More information about the Plasma-devel mailing list