[Digikam-devel] [digikam] digikam/main: Added comment about deleting albums. It is not possible to delete an album

Andi Clemens andi.clemens at googlemail.com
Sun Sep 25 09:05:57 BST 2011

Git commit 3e381acdbb56e736b986a6b63810596cf5064287 by Andi Clemens.
Committed on 25/09/2011 at 10:05.
Pushed by aclemens into branch 'master'.

Added comment about deleting albums. It is not possible to delete an album

CCMAIL:digikam-devel at kde.org

M  +4    -0    digikam/main/digikamapp.cpp


diff --git a/digikam/main/digikamapp.cpp b/digikam/main/digikamapp.cpp
index 0a6033d..8230862 100644
--- a/digikam/main/digikamapp.cpp
+++ b/digikam/main/digikamapp.cpp
@@ -968,6 +968,10 @@ void DigikamApp::setupActions()
     // -----------------------------------------------------------
     // Pop up dialog to ask user whether to permanently delete
+    // FIXME: This action is never used?? How can someone delete a album directly, without moving it to the trash first?
+    //        This is especially important when deleting from a different partiton or from a net source.
+    //        Also note that we use the wrong icon for the default album delete action, which should have a thrashcan icon instead
+    //        of a red cross, it confuses users.
     d->imageDeletePermanentlyAction = new KAction(KIcon("edit-delete"), i18n("Delete Permanently"), this);
     connect(d->imageDeletePermanentlyAction, SIGNAL(triggered()), d->view, SLOT(slotImageDeletePermanently()));

More information about the Digikam-devel mailing list