[KPhotoAlbum] Little patch and some ideas I like to implement

[Henner Zeller]

Hi,
On 11/19/06, Jesper K. Pedersen <blackie at blackie.dk> wrote:
> Right, with the current video support in KDE I dont have a way to ask for the
> size of the videos. I'll make sure size is never shown.

Well, the size of videos often is known, so in that case we _do_ want
to show the size.

> | For me, it always displays 256x192 either for my 640x480 videos (my
> | new cam) or my 320x200 (old cam), and sometimes -1x-1, and always
> | "0.0MP" in the parenthesis.

.. however, the 'Megapixels' are not very worth displaying for 320x200
pixels. And for the unknown  -1x-1 sizes, the image size shouldn't be
displayed at all.

How about the following patch: image size is not displayed, if the
size is unknown (however, still for videos, that actually have a
size). Megapixels are only shown if the value is more than 0.1 (the
display resolution).

cheers,
  -henner
-------------- next part --------------
A non-text attachment was scrubbed...
Name: megapix-only-if-useful.patch.gz
Type: application/x-gzip
Size: 484 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kphotoalbum/attachments/20061119/d7564c8f/attachment.bin>