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