D29401: [KRunner] Fix watched config file path
Konstantinos Smanis
noreply at phabricator.kde.org
Mon May 4 11:17:55 BST 2020
ksmanis created this revision.
ksmanis added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
ksmanis requested review of this revision.
REVISION SUMMARY
The "Clear History" button in the KCM is currently broken, since we are
not watching the actual config file, but a file called "General", i.e.,
the name of the KConfigGroup. Therefore, the QML runner is never
informed that the history was erased in the config file.
TEST PLAN
Clear History now works as expected on master and v5.18.4.1.
REPOSITORY
R120 Plasma Workspace
BRANCH
krunner-fix-watched-config-path (branched from master)
REVISION DETAIL
https://phabricator.kde.org/D29401
AFFECTED FILES
krunner/view.cpp
To: ksmanis, #plasma
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200504/93f2080d/attachment.html>
More information about the Plasma-devel
mailing list