Review Request 121294: Improve wallpaper sourceSize handling

Kai Uwe Broulik kde at privat.broulik.de
Sun Nov 30 14:48:14 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121294/
-----------------------------------------------------------

Review request for Plasma.


Repository: plasma-workspace


Description
-------

Do not bind the image's sourceSize to the root item's size since that will reload the image whenever the size changes. This is especially noticeable in plasmoidviewer where resizing the window causes the background to go black momentarily.

This makes it wait 1s before actually changing the source size, preventing continuous reloading when resizing the window but should also improve resolution changes. Also, it waits for the size to become non-zero before loading the image. Setting a sourceSize of (0,0) is equivalent to not setting it at all, causing it to load the full image, just to reload it in the proper size afterwards.

There's still an issue that the view starts up with a size of 1024x768, loads the wallpaper, and then resizes itself to the screen geometry, causing a delayed reload of the image which was previously not visible but together with me waiting for the size to become available you can see that on plasmashell startup, when using eg. the tiled fill mode.

Unrelated:
 - Move the swap image stuff into a separate function
 - Use my animate binding trick for fillMode change too, now these are lag-free also


Diffs
-----

  wallpapers/image/imagepackage/contents/ui/main.qml 17683c0 

Diff: https://git.reviewboard.kde.org/r/121294/diff/


Testing
-------

Plasmoidviewer is much prettier to use now, no flickering, Plasmashell startup is a bit more nasty imho (especially when using a wallpaper background color) but should be concealed by ksplash. Switching between wallpapers and fillmodes is smooth, it fades from black when changing a wallpaper and fillmode at the same time but afaics that has been the case previously, too.


Thanks,

Kai Uwe Broulik

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20141130/f0b71fda/attachment.html>


More information about the Plasma-devel mailing list