Have the wallpaper combo managed by KConfigDialogManager
ClosedPublic

Authored by ervin on Nov 15 2019, 10:34 AM.

Details

Summary

We use an intermediate property which maps to an index since it's
exactly what KConfigDialogManager expects with a QComboBox.

Also relayouted a bit the ctor and the file to make it easier to
understand (several aspect were weaved together).

Diff Detail

Repository
R133 KScreenLocker
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ervin created this revision.Nov 15 2019, 10:34 AM
Restricted Application added a project: Plasma. · View Herald TranscriptNov 15 2019, 10:34 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
ervin requested review of this revision.Nov 15 2019, 10:34 AM
ervin updated this revision to Diff 70430.Nov 27 2019, 3:00 PM

On some condition the assert would be triggered

mart accepted this revision.Nov 28 2019, 9:29 AM
This revision is now accepted and ready to land.Nov 28 2019, 9:29 AM
This revision was automatically updated to reflect the committed changes.