Review Request: make color wallpaper thumbnail aspect ratio similar to primary screen aspect ratio

[Aaron J. Seigo]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/106754/#review20051
-----------------------------------------------------------

Ship it!


looks functional.


plasma/generic/wallpapers/color/backgroundlistmodel.cpp
<http://git.reviewboard.kde.org/r/106754/#comment15879>

    is m_size was a QSize rather than a QSizeF, this line would be simpler: QPixmap scaledPixmap(m_size)


- Aaron J. Seigo


On Oct. 8, 2012, 12:17 a.m., Reza Shah wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106754/
> -----------------------------------------------------------
> 
> (Updated Oct. 8, 2012, 12:17 a.m.)
> 
> 
> Review request for Plasma and Marco Martin.
> 
> 
> Description
> -------
> 
> I tried to make the behavior of the thumbnail to follow image wallpaper.
> The thumbnail will have aspect ratio similar to active screen aspect ratio if possible.
> 
> 
> Diffs
> -----
> 
>   plasma/generic/wallpapers/color/CMakeLists.txt 5ad829d 
>   plasma/generic/wallpapers/color/backgroundlistmodel.h 8bec6b4 
>   plasma/generic/wallpapers/color/backgroundlistmodel.cpp 6fa4682 
>   plasma/generic/wallpapers/color/color.cpp 05897ba 
> 
> Diff: http://git.reviewboard.kde.org/r/106754/diff/
> 
> 
> Testing
> -------
> 
> Tested using single monitor only but with different resolution.
> I hope somebody with multi monitor setup can test this patch.
> 
> 
> Thanks,
> 
> Reza Shah
> 
>

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