Review Request: fixes thumbnails of KoPADocuments to be without empty spaces around (Flow, Stage)
Friedrich W. H. Kossebau
kossebau at kde.org
Fri Jun 29 22:25:58 BST 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105390/
-----------------------------------------------------------
Review request for Calligra, Yue Liu and Thorsten Zachmann.
Description
-------
Currently the size of the thumbnails is not adapted to the format of the page, instead the icon contains empty white space on the sides if the page is not square. That results in bad previews in the filesystem browsers.
The patch fixes that by calculating the size of the preview image to just match the rendered area.
It also reduces code duplication by moving the identic implementations of thumbImage(...) and generateThumbnail(...) to the base class.
And removes the unused and never-defined method createPageThumbnail() from KPrNotes.
Diffs
-----
libs/kopageapp/KoPAMasterPage.h 9852973
libs/kopageapp/KoPAMasterPage.cpp 42e9065
libs/kopageapp/KoPAPage.h 6a81b04
libs/kopageapp/KoPAPage.cpp cd9c5f4
libs/kopageapp/KoPAPageBase.h b9a5961
libs/kopageapp/KoPAPageBase.cpp cd25030
stage/part/KPrNotes.h 28ee984
Diff: http://git.reviewboard.kde.org/r/105390/diff/
Testing
-------
Cleared the thumbnail cache in ~/.thumbnails and enjoyed better thumbnails after the patch :) E.g. when browsing the templates.
Thanks,
Friedrich W. H. Kossebau
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20120629/da8eece0/attachment.htm>
More information about the calligra-devel
mailing list