Bug 222354 - kphotoalbum can't remain pref
Summary: kphotoalbum can't remain pref
Status: RESOLVED FIXED
Alias: None
Product: kphotoalbum
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Debian unstable Linux
: NOR normal
Target Milestone: ---
Assignee: KPhotoAlbum Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-01-12 11:30 UTC by Fathi Boudra
Modified: 2010-06-14 18:32 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Reenable Auto-Show Thumbnails if number of pictures is below threshold. (3.67 KB, patch)
2010-06-05 21:00 UTC, Andreas Neustifter
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Fathi Boudra 2010-01-12 11:30:32 UTC
Version:           4.1.1 (using KDE 4.3.4)
OS:                Linux
Installed from:    Debian testing/unstable Packages

This a forwaded Debian bug report:
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=563821

With this last version of kphotoalbum (4.1.1), if I set 10 for the "limit"
to view pict after search, it's always down to 0.

The size of view pict (height and larger) are not remember by
kphotoalbum.

Thank for your works.
Comment 1 Miika Turkia 2010-01-12 21:37:38 UTC
The first issue can be found from: settings -> Configure KPhotoAlbum -> Thumbnail View -> Auto display limit

The other about settings in Viewer configuration are not stored on kphotoalbum 4.1.1. However, this is fixed already (bug #220373) and should be included in next release.
Comment 2 Fathi Boudra 2010-01-13 10:16:46 UTC
do you know the commit revision of the fix ?
Comment 3 Miika Turkia 2010-01-13 16:57:33 UTC
Revision 1070499 on stable branch.
Comment 4 Andreas Neustifter 2010-06-05 21:00:56 UTC
Created attachment 47716 [details]
Reenable Auto-Show Thumbnails if number of pictures is below threshold.

This patch now fixes the second part of this Bug (not storing and using the Auto-show feature).
Comment 5 Jesper K. Pedersen 2010-06-14 18:32:00 UTC
SVN commit 1137915 by blackie:

BUG:222354
I have a nasty feeling I remove this feature on purpose, but I can't find
the details, so lets just get this back in.

Thanks to Andreas Neustifter for the patch.


 M  +1 -1      Browser/BrowserWidget.cpp  
 M  +7 -0      Browser/CategoryPage.cpp  
 M  +1 -1      MainWindow/FeatureDialog.cpp  
 M  +1 -1      Settings/SettingsData.cpp  
 M  +1 -1      Settings/SettingsData.h  
 M  +4 -4      Viewer/VideoDisplay.cpp  
 M  +1 -1      Viewer/VideoDisplay.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1137915