In T14860#263323, @alex wrote:

Not sure if KCModuleLoader::isDefaults needs a replacement, it is just loading the plugin & calling KCModuleData::isDefaults().

Also it only seems to be used in systemsettings, consequently calling the stuff there manually seems fine IMHO.

In D28662#664211, @heikobecker wrote:

This breaks the lookandfeel-kcm test. I saw this with 5.18.5 but it also fails on build.kde.org for a while, e.g.:

14:11:19  FAIL!  : KcmTest::testKCMSave() Compared values are not the same
14:11:19     Actual   (cg.readEntry("widgetStyle", QString())): ""
14:11:19     Expected (QString("testValue"))                  : "testValue"
14:11:19     Loc: [/home/jenkins/workspace/Plasma/plasma-desktop/stable-kf5-qt5 SUSEQt5.14/kcms/lookandfeel/autotests/kcmtest.cpp(185)]

https://build.kde.org/job/Plasma/job/plasma-desktop/job/stable-kf5-qt5%20SUSEQt5.14/23/console

restore blank lines

consider Kai's comments

In D29285#660158, @broulik wrote:

Can this go into 5.18?

some code style related comments

clear the nitpicks

Consider Kai's comments

Update code comment accordingly

use qAsConst, indent

maange reset to default application notification

Work on fonts and style

Fix action Present Windows - Current desktop that was saved as All desktop and vice versa

Fix comments

These new properties can be managed by KConfig.
It will automatically handle the read/write process and isSaveNeeded/isDefault status.

Fix two little things

port KCM cursor theme

fix import

The indicators are not well placed.

fix defaults

fix defaults

proper naming

fix error

rename include guard

From previous revision, I moved code related to the Monitor part in the ui in a separate class KWinScreenEdge, that will be used by KCM KWinScreenEdges to remove duplicated code and ease the port to KConfigXT.

rework to allow code reuse in KCM KWinScreenEdges