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