Review Request 118886: Add paintedWidth and paintedHeight properties to WindowThumbnail

David Edmundson david at davidedmundson.co.uk
Mon Jun 23 17:44:37 UTC 2014


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

Ship it!


Ship It!

- David Edmundson


On June 23, 2014, 5:39 p.m., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118886/
> -----------------------------------------------------------
> 
> (Updated June 23, 2014, 5:39 p.m.)
> 
> 
> Review request for Plasma and Martin Gräßlin.
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> -------
> 
> This adds paintedWidth and paintedHeight properties to PlasmaCore.WindowThumbnail which tells as how large the thumbnail, which is scaled keeping aspect ratio, actually is, similar to what QML Image does. This is will eventually allow the taskmanager to size its tooltips more appropriately.
> 
> (Is it better to store m_paintedWidth and m_paintedHeight separately, or is the QSize thing I used ok?)
> 
> 
> Diffs
> -----
> 
>   src/declarativeimports/core/windowthumbnail.h 14fc44a 
>   src/declarativeimports/core/windowthumbnail.cpp b10f030 
> 
> Diff: https://git.reviewboard.kde.org/r/118886/diff/
> 
> 
> Testing
> -------
> 
> Works, reports the actual size
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

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


More information about the Plasma-devel mailing list