<table><tr><td style="">davidedmundson added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D21225">View Revision</a></tr></table><br /><div><div><p>I'm not completely blocking this, but I have two concerns with this patch:</p>

<hr class="remarkup-hr" />

<ol class="remarkup-list">
<li class="remarkup-list-item">wallpaper is somewhat different to the other configs, I'm not super convinced we want to reset it without prompts</li>
</ol>



<hr class="remarkup-hr" />

<ol class="remarkup-list" start="2">
<li class="remarkup-list-item">the resetting has potential to mess with 3rd parties setting the wallpaper.</li>
</ol>

<p>From a customer project we have:<br />
usr/share/plasma/look-and-feel/net.bluesystems.[redacted]/contents/plasmoidsetupscripts/org.kde.plasma.folder.js:applet.writeConfig("Image", "file:///usr/share/wallpapers/[redated]os.jpg")</p>

<p>Currently:<br />
if you click on the lnf with reset layout you go to the lnf set one. That will result in it switching to the distro set one if available otherwise fall back to lnf<br />
if  you click on the lnf without reset layout you don't change the wallpaper at all</p>

<p>After this patch:<br />
if you click on the lnf with reset layout everything is as before. That's all good.</p>

<p>if you click on the lnf without reset layout we will:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">if defaults exist and wallpaper is set, change it</li>
<li class="remarkup-list-item">if defaults exist and wallpaper is not set, do nothing</li>
<li class="remarkup-list-item">if defaults does not exist, load the wallpaper from the superclass lnf</li>
</ul>

<p>Which might seem a bit random.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D21225">https://phabricator.kde.org/D21225</a></div></div><br /><div><strong>To: </strong>mart, Plasma<br /><strong>Cc: </strong>davidedmundson, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>