[Bug 229795] New: Position in album lost when switching album forth and back

classic Classic list List threaded Threaded
30 messages Options
12
Reply | Threaded
Open this post in threaded view
|

[Bug 229795] New: Position in album lost when switching album forth and back

Bugzilla from joern.schimmelpfeng@web.de
https://bugs.kde.org/show_bug.cgi?id=229795

           Summary: Position in album lost when switching album forth and
                    back
           Product: digikam
           Version: unspecified
          Platform: Ubuntu Packages
        OS/Version: unspecified
            Status: UNCONFIRMED
          Severity: normal
          Priority: NOR
         Component: general
        AssignedTo: [hidden email]
        ReportedBy: [hidden email]


Version:            (using KDE 4.4.1)
Installed from:    Ubuntu Packages

Problem description:

If you are switching from one album forth and back again, the last position in
the album is not preserved. This makes it very difficult to review a large
number of pictures and compare them with a different album.

After coming back you need to start searching your pictures from top down until
found the previous picture. With more than 100 pictures this is a painful task.

The same problem happens if you are starting a similarity search. When your
search is completed you cannot just go back to the original picture. Here again
you need to walk top down until found.

Expected behavior:

When I press the "Go Back" button I expect to jump back exactly to the location
in my album where I started from. This should happen regardless if I where in a
different album or in the search tab.

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 229795] Position in album lost when switching album forth and back

Gilles Caulier-4
https://bugs.kde.org/show_bug.cgi?id=229795


Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
          Component|general                     |Albums GUI
            Version|unspecified                 |1.1.0
     Ever Confirmed|0                           |1




--- Comment #1 from Gilles Caulier <caulier gilles gmail com>  2010-03-07 18:21:02 ---
I confirm, and i agree with the wish

Gilles Caulier

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 229795] Position in album lost when switching album forth and back

Marcel Wiesweg
In reply to this post by Bugzilla from joern.schimmelpfeng@web.de
https://bugs.kde.org/show_bug.cgi?id=229795


Marcel Wiesweg <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]




--- Comment #2 from Marcel Wiesweg <marcel wiesweg gmx de>  2010-03-17 19:08:37 ---
Please see bug 208506: How can these two be reconciled? We must decide for
either behavior for the case that the currently selected item is also contained
in the album switched to.

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 229795] Position in album lost when switching album forth and back

Bugzilla from joern.schimmelpfeng@web.de
In reply to this post by Bugzilla from joern.schimmelpfeng@web.de
https://bugs.kde.org/show_bug.cgi?id=229795





--- Comment #3 from Jörn Schimmelpfeng <joern schimmelpfeng web de>  2010-03-17 21:25:51 ---
Well, why do you think that both requirements are contradictionary? In this
request you are moving back. In the other your switching to a new view.

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 229795] Position in album lost when switching album forth and back

Marcel Wiesweg
In reply to this post by Bugzilla from joern.schimmelpfeng@web.de
https://bugs.kde.org/show_bug.cgi?id=229795





--- Comment #4 from Marcel Wiesweg <marcel wiesweg gmx de>  2010-03-17 21:38:35 ---
When image a and b are contained in two views, let's say they are in the same
album and have a common tag. Select a in the album, switch to the tag. Select b
in tag view. Switch back to the album. Shall a be selected (this wish), or b
(208506)?

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 229795] Position in album lost when switching album forth and back

Johannes Wienke-3
In reply to this post by Bugzilla from joern.schimmelpfeng@web.de
https://bugs.kde.org/show_bug.cgi?id=229795





--- Comment #5 from Johannes Wienke <languitar semipol de>  2010-03-17 21:41:01 ---
Add a user option?

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 229795] Position in album lost when switching album forth and back

Johannes Wienke-3
In reply to this post by Bugzilla from joern.schimmelpfeng@web.de
https://bugs.kde.org/show_bug.cgi?id=229795





--- Comment #6 from Johannes Wienke <languitar semipol de>  2010-03-17 21:45:06 ---
Think a bit more about this I would also say these two reports are
contradictory. This report is valid for switching from one album to the other
of the same type, where as the other bug report is about switching between
different album types.

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 229795] Position in album lost when switching album forth and back

Bugzilla from krivakin@gmail.com
In reply to this post by Bugzilla from joern.schimmelpfeng@web.de
https://bugs.kde.org/show_bug.cgi?id=229795





--- Comment #7 from konstantin <krivakin gmail com>  2010-04-07 00:14:09 ---
Created an attachment (id=42550)
 --> (http://bugs.kde.org/attachment.cgi?id=42550)
patch for bug229795

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 229795] Position in album lost when switching album forth and back

Bugzilla from krivakin@gmail.com
In reply to this post by Bugzilla from joern.schimmelpfeng@web.de
https://bugs.kde.org/show_bug.cgi?id=229795


konstantin <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]




--- Comment #8 from konstantin <krivakin gmail com>  2010-04-07 00:14:49 ---
This patch implements the conservation position (physical and tags) when
switching between albums. In my opinion what has been written in the first
record.

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 229795] Position in album lost when switching album forth and back [patch]

Gilles Caulier-4
In reply to this post by Bugzilla from joern.schimmelpfeng@web.de
https://bugs.kde.org/show_bug.cgi?id=229795


Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Position in album lost when |Position in album lost when
                   |switching album forth and   |switching album forth and
                   |back                        |back [patch]




--- Comment #9 from Gilles Caulier <caulier gilles gmail com>  2010-04-07 08:38:50 ---
Marcel, Johaness,

What do you think about this patch ? It's not too intrusive.

Gilles Caulier

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 229795] Position in album lost when switching album forth and back [patch]

Gilles Caulier-4
In reply to this post by Bugzilla from joern.schimmelpfeng@web.de
https://bugs.kde.org/show_bug.cgi?id=229795


Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]
            Version|1.1.0                       |1.3.0




--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 229795] Position in album lost when switching album forth and back [patch]

Bugzilla from krivakin@gmail.com
In reply to this post by Bugzilla from joern.schimmelpfeng@web.de
https://bugs.kde.org/show_bug.cgi?id=229795





--- Comment #10 from konstantin <krivakin gmail com>  2010-04-07 16:40:53 ---
Do you think should be done as an option, which can be turned off in settings?

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 229795] Position in album lost when switching album forth and back [patch]

Gilles Caulier-4
In reply to this post by Bugzilla from joern.schimmelpfeng@web.de
https://bugs.kde.org/show_bug.cgi?id=229795





--- Comment #11 from Gilles Caulier <caulier gilles gmail com>  2010-04-07 17:00:47 ---
Konstantin,

I vote for no option in setup. Sound like this behavior is natural for me.

In fact, i waiting this feature for a long time.

Note : There is another bugzilla entry, relevant of icon view, where users want
to customize items order like they want. But this is another story, if you is
interested... I think that database need to be adapted to store album filtering
and ordering properties.

https://bugs.kde.org/show_bug.cgi?id=91562

Gilles Caulier

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 229795] Position in album lost when switching album forth and back [patch]

Bugzilla from krivakin@gmail.com
In reply to this post by Bugzilla from joern.schimmelpfeng@web.de
https://bugs.kde.org/show_bug.cgi?id=229795





--- Comment #12 from konstantin <krivakin gmail com>  2010-04-07 17:45:30 ---
What do you think about this patch? It stands as a change, improve, leave as
is, or he does not need?

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 229795] Position in album lost when switching album forth and back [patch]

Gilles Caulier-4
In reply to this post by Bugzilla from joern.schimmelpfeng@web.de
https://bugs.kde.org/show_bug.cgi?id=229795





--- Comment #13 from Gilles Caulier <caulier gilles gmail com>  2010-04-07 18:28:58 ---
Just compiled and tested :

- Scrolling position is perfectly saved and restored.
- Selection is not restored. Only the first image of selection is restored, in
case of more than one item is selected.

Gilles Caulier

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 229795] Position in album lost when switching album forth and back [patch]

Bugzilla from krivakin@gmail.com
In reply to this post by Bugzilla from joern.schimmelpfeng@web.de
https://bugs.kde.org/show_bug.cgi?id=229795





--- Comment #14 from konstantin <krivakin gmail com>  2010-04-07 18:43:30 ---
All right. I'll do whatever remained selection.

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 229795] Position in album lost when switching album forth and back [patch]

Marcel Wiesweg
In reply to this post by Bugzilla from joern.schimmelpfeng@web.de
https://bugs.kde.org/show_bug.cgi?id=229795





--- Comment #15 from Marcel Wiesweg <marcel wiesweg gmx de>  2010-04-07 19:37:10 ---
You can use DigikamView's signalImageSelected() to get information about all
currently selected images. There is no good way currently to apply this
information though - remember that images are loaded in chunks.
For me it's fine as it is.

Some minor remarks: Cleanup the map entry in deleteAlbum? In one or other
occasion you can use const ImageInfo& to pass by value. Replace tabs by 4
spaces.

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 229795] Position in album lost when switching album forth and back [patch]

Bugzilla from krivakin@gmail.com
In reply to this post by Bugzilla from joern.schimmelpfeng@web.de
https://bugs.kde.org/show_bug.cgi?id=229795


konstantin <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #42550|0                           |1
        is obsolete|                            |




--- Comment #16 from konstantin <krivakin gmail com>  2010-04-10 22:44:08 ---
Created an attachment (id=42661)
 --> (http://bugs.kde.org/attachment.cgi?id=42661)
Save all the selected images

Added to the patch:
- Save all the selected images
- Delete information about an album from the history map when deleting album
- Just a couple of minor fixes

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 229795] Position in album lost when switching album forth and back [patch]

Gilles Caulier-4
In reply to this post by Bugzilla from joern.schimmelpfeng@web.de
https://bugs.kde.org/show_bug.cgi?id=229795





--- Comment #17 from Gilles Caulier <caulier gilles gmail com>  2010-04-11 11:09:44 ---
Tested. The patch sound working fine.

Some remark about coding style :

always use m_ prefix for class members (sometime you use it, sometime no). A
lots of my time in digiKam project, i review code to polish implementation. The
goal is to have same coding style everywhere to have an homogenous
implementation. It's important.

about pointer declaration, use "MyClass*    ptr" is better than "MyClass    
*ptr" and align all of then, line by line.

When it's possible, use d private class container to speed up compilation, and
clean up include file in header implementation.

Marcel, check out again before patch inclusion in svn

Thanks

Gilles

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 229795] Position in album lost when switching album forth and back [patch]

Bugzilla from krivakin@gmail.com
In reply to this post by Bugzilla from joern.schimmelpfeng@web.de
https://bugs.kde.org/show_bug.cgi?id=229795





--- Comment #18 from konstantin <krivakin gmail com>  2010-04-11 11:35:52 ---
(From update of attachment 42661)
diff -u ./dk.original/graphics/digikam/digikam/albumhistory.cpp
./dk.patch/graphics/digikam/digikam/albumhistory.cpp
--- ./dk.original/graphics/digikam/digikam/albumhistory.cpp    2010-04-07
01:53:31.744391200 +0400
+++ ./dk.patch/graphics/digikam/digikam/albumhistory.cpp    2010-04-11
13:31:53.936729064 +0400
@@ -32,6 +32,8 @@
 // Local includes

 #include "album.h"
+#include "imageinfo.h"
+#include "albummanager.h"

 namespace Digikam
 {
@@ -65,11 +67,36 @@
     QWidget* widget;
 };

+class HistoryPosition
+{
+public:
+
+    HistoryPosition()
+    {
+
+    };
+
+    HistoryPosition(ImageInfo c, KUrl::List s)
+    {
+        current  = c;
+        select = s;
+    };
+
+    bool operator==(const HistoryPosition& item)
+    {
+        return (current == item.current) && (select == item.select);
+    }
+
+    ImageInfo current;
+    KUrl::List select;
+};
+
 AlbumHistory::AlbumHistory()
 {
     m_backwardStack = new AlbumStack;
     m_forwardStack  = new AlbumStack;
     m_moving        = false;
+    m_blockSelection  = false;
 }

 AlbumHistory::~AlbumHistory()
@@ -344,4 +371,75 @@
     return (m_backwardStack->count() <= 1) ? true : false;
 }

+void AlbumHistory::slotAlbumSelected()
+{
+    Album* currentAlbum = AlbumManager::instance()->currentAlbum();
+    if (m_historyPos.contains(currentAlbum))
+    {
+        if (currentAlbum->type() == Album::PHYSICAL || currentAlbum->type() ==
Album::TAG)
+        {
+            m_blockSelection = true;
+            emit signalSetCurrent(m_historyPos[currentAlbum].current.id());
+        }
+    }
+}
+
+void AlbumHistory::slotAlbumCurrentChanged()
+{
+    Album* currentAlbum = AlbumManager::instance()->currentAlbum();
+    if (m_historyPos.contains(currentAlbum))
+    {
+        if (currentAlbum->type() == Album::PHYSICAL || currentAlbum->type() ==
Album::TAG)
+        {
+            if (m_historyPos[currentAlbum].select.size())
+            {
+                emit signalSetSelectedUrls(m_historyPos[currentAlbum].select);
+            }
+        }
+    }
+    m_blockSelection = false;
+}
+
+void AlbumHistory::slotCurrentChange(const ImageInfo& info)
+{
+    Album* currentAlbum = AlbumManager::instance()->currentAlbum();
+    m_historyPos[currentAlbum].current = info;
+}
+
+void AlbumHistory::slotImageSelected(const ImageInfoList& selectedImage)
+{
+    if (m_blockSelection)
+        return;
+    Album *currentAlbum = AlbumManager::instance()->currentAlbum();
+    if (m_historyPos.contains(currentAlbum))
+        m_historyPos[currentAlbum].select.clear();
+    for (int i = 0; i < selectedImage.count(); i++)
+    {
+        if
(!m_historyPos[currentAlbum].select.contains(selectedImage[i].fileUrl()))
+        {
+            m_historyPos[currentAlbum].select.insert(0,
selectedImage[i].fileUrl());
+        }
+    }
+}
+
+void AlbumHistory::slotClearSelectPAlbum(const ImageInfo& imageInfo)
+{
+    Album* album =
dynamic_cast<Album*>(AlbumManager::instance()->findPAlbum(imageInfo.albumId()));
+    if (m_historyPos.contains(album))
+        m_historyPos[album].select.clear();
+}
+
+void AlbumHistory::slotClearSelectTAlbum(int id)
+{
+    Album* album =
dynamic_cast<Album*>(AlbumManager::instance()->findTAlbum(id));
+    if (m_historyPos.contains(album))
+        m_historyPos[album].select.clear();
+}
+
+void AlbumHistory::slotAlbumDeleted(Album* album)
+{
+    if (m_historyPos.contains(album))
+        m_historyPos.remove(album);
+}
+
 }  // namespace Digikam
diff -u ./dk.original/graphics/digikam/digikam/albumhistory.h
./dk.patch/graphics/digikam/digikam/albumhistory.h
--- ./dk.original/graphics/digikam/digikam/albumhistory.h    2010-04-07
01:53:31.796392789 +0400
+++ ./dk.patch/graphics/digikam/digikam/albumhistory.h    2010-04-11
13:25:49.872729576 +0400
@@ -30,14 +30,21 @@
 // Qt includes

 #include <QList>
+#include <QMap>
 #include <QObject>
 #include <QStringList>

+//KDE includes
+#include <KUrl>
+
 namespace Digikam
 {

 class Album;
 class HistoryItem;
+class HistoryPosition;
+class ImageInfo;
+class ImageInfoList;

 /**
  * Manages the history of the last visited albums.
@@ -67,6 +74,21 @@
     bool            isForwardEmpty() const;
     bool            isBackwardEmpty() const;

+Q_SIGNALS:
+
+    void            signalSetCurrent(qlonglong imageId);
+    void            signalSetSelectedUrls(const KUrl::List&);
+
+public Q_SLOTS:
+
+    void            slotAlbumCurrentChanged();
+    void            slotAlbumDeleted(Album* album);
+    void            slotAlbumSelected();
+    void            slotClearSelectPAlbum(const ImageInfo& imageInfo);
+    void            slotClearSelectTAlbum(int id);
+    void            slotCurrentChange(const ImageInfo& info);
+    void            slotImageSelected(const ImageInfoList& selectedImage);
+
 private:

     HistoryItem*    getCurrentAlbum() const;
@@ -76,10 +98,12 @@

     typedef QList<HistoryItem*> AlbumStack;

-    bool             m_moving;
+    bool            m_moving;
+    bool            m_blockSelection;

     AlbumStack      *m_backwardStack;
     AlbumStack      *m_forwardStack;
+    QMap<Album*, HistoryPosition> m_historyPos;
 };

 }  // namespace Digikam
diff -u ./dk.original/graphics/digikam/digikam/digikamview.cpp
./dk.patch/graphics/digikam/digikam/digikamview.cpp
--- ./dk.original/graphics/digikam/digikam/digikamview.cpp    2010-04-07
01:53:31.580397973 +0400
+++ ./dk.patch/graphics/digikam/digikam/digikamview.cpp    2010-04-11
00:26:00.100730406 +0400
@@ -495,6 +495,34 @@

     connect(AlbumSettings::instance(), SIGNAL(setupChanged()),
             this, SLOT(slotSidebarTabTitleStyleChanged()));
+
+    // -- Album History -----------------
+    connect(this, SIGNAL(signalAlbumSelected(bool)),
+            d->albumHistory, SLOT(slotAlbumSelected()));
+
+    connect(this, SIGNAL(signalImageSelected(const ImageInfoList&, bool, bool,
const ImageInfoList&)),
+            d->albumHistory, SLOT(slotImageSelected(const ImageInfoList&)));
+
+    connect(d->iconView, SIGNAL(currentChanged(const ImageInfo&)),
+            d->albumHistory, SLOT(slotCurrentChange(const ImageInfo&)));
+
+    connect(d->iconView, SIGNAL(gotoAlbumAndImageRequested(const ImageInfo&)),
+            d->albumHistory, SLOT(slotClearSelectPAlbum(const ImageInfo&)));
+
+    connect(d->iconView, SIGNAL(gotoTagAndImageRequested(int)),
+            d->albumHistory, SLOT(slotClearSelectTAlbum(int)));
+
+    connect(d->iconView->imageModel(),
SIGNAL(imageInfosAdded(QList<ImageInfo>)),
+            d->albumHistory, SLOT(slotAlbumCurrentChanged()));
+
+    connect(d->albumHistory, SIGNAL(signalSetCurrent(qlonglong)),
+            d->iconView, SLOT(setCurrentWhenAvailable(qlonglong)));
+
+    connect(d->albumHistory, SIGNAL(signalSetSelectedUrls(KUrl::List)),
+            d->iconView, SLOT(setSelectedUrls(KUrl::List)));
+            
+    connect(d->albumManager, SIGNAL(signalAlbumDeleted(Album*)),
+            d->albumHistory, SLOT(slotAlbumDeleted(Album*)));
 }

 void DigikamView::connectIconViewFilter(AlbumIconViewFilter *filter)
Общие подкаталоги: ./dk.original/graphics/digikam/digikam/.svn и
./dk.patch/graphics/digikam/digikam/.svn

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
12