Review Request: Enlarge image in folder preview when there's only one image

[Martin Engelmann]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/6332/
-----------------------------------------------------------

Review request for kdelibs.


Summary
-------

As stated in the bug report, the image on the folder preview should be enlarged if there is only one image.

To achieve this there are two images created for the folder: "img" as before and "oneTileImg". "oneTileImg" starts with the same image as "img", but after the first thumbnail is generated "oneTileImg" won't be touched. The number of successfully created thumbnails is counted in "validThumbnails". If only one thumbnail could be generated, "oneTileImg" is returned.

The code between lines 566 and 589 in the original thumbnail.cpp was partly extracted into a new function called "drawSubThumbnail". This is done to simplify the creation of the second preview.

The extra effort to create the "oneTileImg" comparable to creating a fifth preview image for the folder.


This addresses bug 240861.
    https://bugs.kde.org/show_bug.cgi?id=240861


Diffs
-----

  /trunk/KDE/kdebase/runtime/kioslave/thumbnail/thumbnail.h 1214477 
  /trunk/KDE/kdebase/runtime/kioslave/thumbnail/thumbnail.cpp 1214477 

Diff: http://svn.reviewboard.kde.org/r/6332/diff


Testing
-------

Tested using both Dolphin and Konqueror from trunk.

The thumbnail generation was tested by moving images and files without thumbnail plug-in to a new folder. Then one file after another was deleted until only files without thumbnail plug-in were present.

Also I opened randomly some folders with sub-folders containing large amounts of images (e.g. oxygen icon set with > 550 action icons) to the performance.


Screenshots
-----------

Folder thumbnails for oxygen icon set in Konqueror
  http://svn.reviewboard.kde.org/r/6332/s/604/


Thanks,

Martin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20110115/20ed89c4/attachment.htm>