<table><tr><td style="">broulik created this revision.<br />broulik added a reviewer: Plasma.<br />broulik set the repository for this revision to R120 Plasma Workspace.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.
</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>REVISION SUMMARY</strong><div><p>Directly expose "targetSize" as property. The wallpaper is already initialized with the correct size for some time now but the fact that we had two properties width and height in there had it still do unneccessary work when it emitted "sizeChanged(2560, 0)" and then the final "sizeChanged(2560, 1440)" afterwards.</p>

<p>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. I changed it to use the current Screen there - I think that's good enough.</p>

<p>BUG: 374689<br />
FIXED-IN: 5.9.0</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>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.</p></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>AFFECTED FILES</strong><div><div>wallpapers/image/image.cpp<br />
wallpapers/image/image.h<br />
wallpapers/image/imagepackage/contents/config/main.xml<br />
wallpapers/image/imagepackage/contents/ui/config.qml<br />
wallpapers/image/imagepackage/contents/ui/main.qml</div></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>plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, andreaska, sebas<br /></div>