davidedmundson created this revision. davidedmundson added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY ContainmentConfigView has a special code path for wallpaper configs where, if you change wallpaper type, it will create a temporary config ConfigPropertyMap, then copy it over to the real config when the containment actually changes. ConfigPropertyMap does not call valueChanged() on insertion (See QQmlPropertyMap), and we only write the value into the actual KConfigSkeleton on valueChange. BUG: 375532 TEST PLAN Start plasma with image wallpaper Set to colour wallpaper and save colour Hit apply Close plasma Start plasma Before the correct colour wouldn't be loaded. Now it is. REPOSITORY R120 Plasma Workspace BRANCH master REVISION DETAIL https://phabricator.kde.org/D4285 AFFECTED FILES shell/containmentconfigview.cpp EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: davidedmundson, #plasma Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
