Summary: | leave tags tree (right sidebar) as set when closing digikam | ||
---|---|---|---|
Product: | [Applications] digikam | Reporter: | Daniel Bauer <linux> |
Component: | Albums-TreeView | Assignee: | Digikam Developers <digikam-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | monoceros84 |
Priority: | NOR | ||
Version: | 0.9.3 | ||
Target Milestone: | --- | ||
Platform: | openSUSE | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | 0.10.0 |
Description
Daniel Bauer
2007-07-24 16:23:19 UTC
Two additional buttons "open complete tree" and "close complete tree" would also help. And/ or a setting to display the tree opened/ closed/ in the last view when starting digiKam. Andi, i remember that you have implemented something like this with album tree view. Right ? Gilles Caulier No not really, this was for collapsing the tree when using quickfilter search on the folder list (but the patch might be used for this). I wanted to fix this issue here some time ago, but it wasn't that easy. The same issue was reported in another bugreport, I will search for it and mark this one here a a duplicate. Andi I found the bug, but it is not a duplicate of it: bug 161988 Now I can remember what was wrong with those folder views, they contain CheckListItems which are not capable to deliver some values that a FolderItem does (I don't right now what it was), and so they are not saved correctly on exit. When starting digiKam, those views are always fully expanded. I will take a look at this again... Andi SVN commit 919579 by aclemens: Fix saveViewState() for folderviews with CheckListItems, like TAlbumListView and TagFilterView. With current implementation, we are not able to have one common loadViewState() and saveViewState() method, so we need to re-implement them in TAlbumListView and TagFilterView. This is not nice because we have duplicate code now, but since we are going to switch to ModelView concept next release, it would be a waste of time to create a new common interface or abstract class now. We can not call virtual methods in constructors / destructors, so every view inherited from FolderView needs to call saveViewState() in its own destructor instead. BUG:148166 M +2 -0 digikam/albumfolderview.cpp M +1 -1 digikam/folderview.cpp M +72 -1 digikam/tagfilterview.cpp M +5 -1 digikam/tagfilterview.h M +3 -1 digikam/tagfolderview.cpp M +6 -2 libs/imageproperties/imagedescedittab.cpp M +71 -1 libs/imageproperties/talbumlistview.cpp M +7 -1 libs/imageproperties/talbumlistview.h M +3 -1 utilities/fuzzysearch/fuzzysearchfolderview.cpp M +3 -1 utilities/gpssearch/gpssearchfolderview.cpp M +4 -2 utilities/searchwindow/searchfolderview.cpp M +3 -1 utilities/timeline/timelinefolderview.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=919579 SVN commit 919759 by aclemens: backport commit #919579 from KDE4 trunk CCBUG:148166 M +2 -0 digikam/albumfolderview.cpp M +2 -2 digikam/folderview.cpp M +8 -6 digikam/searchfolderview.cpp M +73 -1 digikam/tagfilterview.cpp M +7 -3 digikam/tagfilterview.h M +3 -1 digikam/tagfolderview.cpp M +5 -3 digikam/timelinefolderview.cpp M +4 -3 digikam/timelinefolderview.h M +6 -2 libs/imageproperties/imagedescedittab.cpp M +1 -0 libs/imageproperties/imagedescedittab.h M +73 -2 libs/imageproperties/talbumlistview.cpp M +7 -1 libs/imageproperties/talbumlistview.h WebSVN link: http://websvn.kde.org/?view=rev&revision=919759 |