[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