[KPhotoAlbum] [Bug 145309] make "show this image" load all images in current view
Robert L Krawitz
rlk at alum.mit.edu
Thu May 24 03:20:13 BST 2007
Date: 12 May 2007 16:51:31 -0000
From: kpabugs at willden.org
Perfect addon! For a long time, I've thought it was annoying but I
ended up getting used to it and almost forgot.
I suppose you made only the double-click provide the new behaviour
on purpose. I'm wondering if we couldn't just decide that if
there's exactly one photo selected always allow to do what your
patch does.
In fact, for the example use case: sometimes I select a photo, then
press escape and only use the keyboard arrows to move the selection
and hit enter (so only one photo is currently selected).
I can't any reason why when just one photo is to be displayed, then
the behaviour your patch allow would be activated. I guess we
should ask the list (though this post should be sent to the list,
thanks to Shawn, if I'm not wrong).
One problem I've seen with this: if I'm on a cold system, with all of
my images in the thumbnail view (~20000 images), when I double click
on one image it takes a few minutes until the image pops up. On a
warm system, it's fast, suggesting that it's stat'ing every image (I
haven't looked at the code). This bottleneck needs to be eliminated.
Two other problems:
1) Viewing a RAW image (a .crw, to be precise) does not rotate the
image if it was taken in vertical orientation (it previously did,
as of about a month ago).
2) The selection in the thumbnail view isn't highlighted -- there's
nothing indicating which images are selected or not. The
background's a uniform black, and the images don't get the
"dithered" look.
--
Robert Krawitz <rlk at alum.mit.edu>
Tall Clubs International -- http://www.tall.org/ or 1-888-IM-TALL-2
Member of the League for Programming Freedom -- mail lpf at uunet.uu.net
Project lead for Gutenprint -- http://gimp-print.sourceforge.net
"Linux doesn't dictate how I work, I dictate how Linux works."
--Eric Crampton
More information about the Kphotoalbum
mailing list