[KPhotoAlbum] [Bug 145309] make "show this image" load all images in current view

Robert L Krawitz rlk at alum.mit.edu
Fri May 25 00:44:30 BST 2007


   Date: Fri, 25 May 2007 00:42:27 +0200
   From: =?ISO-8859-1?Q?Jan_Kundr=E1t?= <jkt at gentoo.org>

   Jan Kundrát wrote:
   > Yup, I'm aware of that, but I haven't tested it yet so I don't propose 
   > the change. I'm not sure how would Viewer react if it's told to show 
   > image 123 and images 100 - 110 aren't readable, for example.
   > 
   > And if we change this, we should do the same for Ctrl+A (select all images).

   Please see the attached patch that addresses those two issues and
   tries to "skip" those non-existant images when viewing. This
   auto-skipping could be a bit confusing because commands like "seek
   100 images back" don't take these "holes" into account...

   I've also changed one iterator to a constant one. It compiles and
   works fine, so I guess it's a safe change.

It seems to solve the problem.

Note that in any event KPA has to be prepared to deal with nonexistent
images -- an image may be deleted between when KPA scans and when
someone tries to look at it.



More information about the Kphotoalbum mailing list