Differential D29411

[KRunner] Watch config with KConfigWatcher
ClosedPublic
Actions

Authored by broulik on May 4 2020, 2:21 PM.

Details

Reviewers
ngraham
ksmanis
Group Reviewers
Plasma
Commits
R120:6db69ae09524: [KRunner] Watch config with KConfigWatcher
Summary

Rather than using a KDirWatch that points to the wrong place.

Test Plan

5.18 please

  • Cleared history in KCM with D29410, confi was reloaded and history gone right away

Alternative for D29401

Diff Detail

Repository
R120 Plasma Workspace
Lint
Lint Skipped
Unit
Unit Tests Skipped
broulik created this revision.May 4 2020, 2:21 PM
Restricted Application added a project: Plasma. · View Herald TranscriptMay 4 2020, 2:21 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.May 4 2020, 2:21 PM
broulik edited the test plan for this revision. (Show Details)
ngraham accepted this revision.May 4 2020, 2:42 PM

Yeah, much nicer! Thanks.

This revision is now accepted and ready to land.May 4 2020, 2:42 PM
Closed by commit R120:6db69ae09524: [KRunner] Watch config with KConfigWatcher (authored by broulik). · Explain WhyMay 4 2020, 2:48 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mkrunner/view.h (2 lines)
Mkrunner/view.cpp (21 lines)

Diff 81896

View Options

krunner/view.h

Loading...
View Options

krunner/view.cpp

Loading...