Review Request 125743: Keep aspect ratio in default KSplash background

Martin Klapetek martin.klapetek at gmail.com
Wed Oct 21 18:51:20 UTC 2015


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

(Updated Oct. 21, 2015, 8:51 p.m.)


Review request for Plasma.


Repository: plasma-workspace


Description (updated)
-------

I'm just not sure if it's better to do Fit or Crop. I went with Fit as that simply fits the whole artwork, but let me know if you think differently.

Ideally this should use a wallpaper package for the background, but I don't know how to use it from QML (if possible at all).

Issues with Fit:
* If the image is different aspect ratio than the screen, it would add black borders either above/below or left/right

Issues with Crop:
* if the image has bigger resolution than the screen, it will get cropped to be the screen resolution, so eg. the distro ships 1920x1200 wallpaper and on 1024x768 the crop would be quite big (the image would still be centered so cropped on the sides)


Diffs
-----

  lookandfeel/contents/splash/Splash.qml fdeb1ea 

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


Testing
-------


Thanks,

Martin Klapetek

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


More information about the Plasma-devel mailing list