Review Request 121294: Improve wallpaper sourceSize handling

[Kai Uwe Broulik]

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

(Updated Dez. 4, 2014, 9:16 nachm.)


Review request for Plasma.


Changes
-------

Simplify
Thanks to Marco's patches the initial width and height are now set properly right away.

image.status === Image.Null is just a fancy way for testing whether the source is empty since URL comparison in QML can be tricky


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 (updated)
-----

  wallpapers/image/imagepackage/contents/ui/main.qml 1f11795 

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/20141204/a06e8f11/attachment-0001.html>