Bug 88121 - tag sometimes does not accept drop of photo(s), or reports wrong tag name to drop to
Summary: tag sometimes does not accept drop of photo(s), or reports wrong tag name to ...
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Tags-Engine (show other bugs)
Version: 0.7.0
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-08-26 13:13 UTC by Wilbert Berendsen
Modified: 2022-01-22 14:02 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In: 7.6.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Wilbert Berendsen 2004-08-26 13:13:32 UTC
Version:           0.7.0-cvs (using KDE 3.3.0, compiled sources)
Compiler:          gcc version 3.3.4 20040623 (Gentoo Linux 3.3.4-r1, ssp-3.3.2-2, pie-8.7.6)
OS:                Linux (i686) release 2.6.8-gentoo

Today I updated CVS and noticed the dragable tags, great!

But since this version there also is a little regression: Sometimes, when I drag photo(s) to a tag to add that tag to that photo(s), the tag view does not accept the drop. Especially when the target tag is somewhere in the lower half of the tag view.

When I keep dragging and move the mouse somewhere higher in the tagview, the tagview suddenly starts to accept drops. But when I then drop the photo(s) onto the desired tags, a menu pops up with the wrong tagname! So then selecting the option "Add this tag to the dropped photos" would add the wrong tag to them.
Comment 1 Wilbert Berendsen 2004-08-26 13:15:01 UTC
I could add that sometimes folding and unfolding some tags help to work around the problem.
Comment 2 Joern Ahrens 2004-08-26 18:00:12 UTC
CVS commit by jahrens: 

fixed... Can you test it Wilbert?

CCMAIL: 88121-done@bugs.kde.org


  M +2 -1      albumfolderview.cpp   1.28


--- kdeextragear-3/digikam/digikam/albumfolderview.cpp  #1.27:1.28
@@ -1177,5 +1177,6 @@ void AlbumFolderView::contentsDragMoveEv
     // Do not allow an album to be dropped on it's child
     AlbumFolderItem *folderItem = dynamic_cast<AlbumFolderItem*>(dragItem_);
-    if (!folderItem || folderItem->album()->isAncestorOf(newDropTarget->album()))
+    if ((!TagItemsDrag::canDecode(event) && !AlbumItemsDrag::canDecode(event)) &&
+        (!folderItem || folderItem->album()->isAncestorOf(newDropTarget->album())))
     {
         event->ignore();


Comment 3 Wilbert Berendsen 2004-08-27 11:38:01 UTC
It seems to be fixed, thanks!