Review Request 121123: Render Words thumbnails optionally like for print (fixes file thumbnails)
Friedrich W. H. Kossebau
kossebau at kde.org
Fri Nov 14 18:48:13 GMT 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121123/
-----------------------------------------------------------
Review request for Calligra, Camilla Boemann, Dan Leinir Turthra Jensen, and Thorsten Zachmann.
Repository: calligra
Description
-------
Currently thumbnails for ODT files etc. are generated using the configured view settings (incl. showing non-existing table borders, annotations, textshape outlines, inlineobjects highlight, etc). IMHO pure thumbnails used for previews outside of the editor app should just render pages like for print, and only inside editor apps (e.g. when used for page overview widgets) the thumbnails should be resembling the current display of the page in the editor.
Attached patch enables this, by adding a parameter to `KWPage::thumbnail(...)` which allows to control how the rendering of thumbnails is done.
This fixes the preview thumbnails as generated for ODT files, both for embedding into the file but also when generating previews in the filemanager with the thumbnailer.
Not sure about `CSThumbProviderWords::createThumbnails(...)` from cstester though. Thorsten, which rendering should be done there? As for print, or as for whatever is configured for the editor view?
Diffs
-----
extras/okularodtgenerator/OkularOdtGenerator.cpp d1b98a5
words/part/KWDocument.cpp e287829
words/part/KWPage.h 0821ccd
words/part/KWPage.cpp 5dc6a37
Diff: https://git.reviewboard.kde.org/r/121123/diff/
Testing
-------
Thanks,
Friedrich W. H. Kossebau
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20141114/95740fe4/attachment.htm>
More information about the calligra-devel
mailing list