Review Request 124311: Improve wallpaper rendering

Kai Uwe Broulik kde at privat.broulik.de
Fri Jul 10 20:03:25 UTC 2015


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

(Updated July 10, 2015, 8:03 p.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma.


Changes
-------

Submitted with commit 29e39fe58aed5ef3b9155fbf96227eea3c7dddb2 by Kai Uwe Broulik to branch master.


Bugs: 338506
    https://bugs.kde.org/show_bug.cgi?id=338506


Repository: plasma-workspace


Description
-------

Update the targetSize when it changes to ensure when resolution changes, we update the image. Also, since the view initially has a 1024x768 size it would just load that image and never use the full resolution leaving you with an ugly blurry image (used to work at some point though iirc)

Also don't bother loading anything until everything has settled and the size is the final one. This potentially improves startup performance by saving a bit of IO.


Diffs
-----

  wallpapers/image/image.h 5d297b5 
  wallpapers/image/image.cpp f5920bf 

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


Testing
-------

At least in Scaled mode I can no longer see a difference between the image opened in Gwenview and on the Desktop. For some reason it still stretches it when I use Scale and Crop although it loads the 2560x1440 image. Changed screen resolution and it loaded a new image.

That test image in the bug report looks fine to me now. Ok, no, that image doesn't go through that target size thing probably so did not change.


Thanks,

Kai Uwe Broulik

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


More information about the Plasma-devel mailing list