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