<table><tr><td style="">broulik updated the summary for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-pekrr3uhytsoafc/" rel="noreferrer">(Show Details)</a><br />broulik updated the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-4zvl52k3m7eqcyy/" rel="noreferrer">(Show Details)</a>
</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/D4001" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>EDIT DETAILS</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>Also, I didn't really see a point in storing the wallpaper size in its config, it's overwritten by the wallpaper anyway and only used in the wallpaper config dialog for the aspect ratio. <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">I changed it to use the current Screen there - I think that's good enough</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">We can just ask the view directly then</span>.<div style="padding: 8px 0;">...</div></div></div></div><br /><div><strong>EDIT DETAILS</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">Verified that setSingleImage is now only called once on startup. Image still crisp. Aspect ratio of images in config dialog is still correct - didn't test with multiscreen though, I suppose the aspect ratio will now be the of the one where the dialog currently is, not of the actual containment you're configuring. We could probably fix that by asking the containment but storing the size in config is wrong either way imho.</span></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D4001" rel="noreferrer">https://phabricator.kde.org/D4001</a></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>broulik, Plasma<br /><strong>Cc: </strong>davidedmundson, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, andreaska, sebas<br /></div>