RFC: Adding Cancel to the Copy/Move/Drop popup

[David Faure]

I remember this being discussed in the past, but I don't remember the
result of the discussion... Should this popup have a Cancel menuitem ?
(IMHO it can't hurt - and BTW Windows has it too, in the RMB-DnD-popup).

----------  Forwarded Message  ----------

Subject: little patch ...
Date: Sun, 07 Apr 2002 14:47:41 +0200
From: Peter Nimmervoll <vanHell at gmx.at>
To: David Faure <david at mandrakesoft.com>

Hi,

I´ve a little patch for libkonq, which add  "cancel" to the 
 file-drop-popup-menu.

Peter

-- 
ICQ #123021437
YIM xvanhell
This message is GPL
The UNIX Guru's View of Sex:
gawk; talk; nice; date; wine; grep; touch; unzip; strip; touch; gasp; \
finger; gasp; mount; fsck; more; yes; gasp; umount; make clean; \
make mrproper; sleep



-------------------------------------------------------


-- 
David FAURE, david at mandrakesoft.com, faure at kde.org
http://people.mandrakesoft.com/~david/, http://www.konqueror.org/
KDE, Making The Future of Computing Available Today

-------------- next part --------------
Index: konq_operations.cc
===================================================================
RCS file: /cvs/kdebase/libkonq/konq_operations.cc,v
retrieving revision 1.92
diff -u -3 -p -r1.92 konq_operations.cc
--- konq_operations.cc	2002/03/29 01:49:23	1.92
+++ konq_operations.cc	2002/04/04 16:12:38
@@ -565,7 +565,9 @@ void KonqOperations::doFileCopy()
             popup.insertItem( i18n( "&Move Here" ), 2 );
         popup.insertItem(SmallIconSet("www"), i18n( "&Link Here" ), 3 );
         if (bSetWallpaper)
-            popup.insertItem(SmallIconSet("background"), i18n( "Set as &Wallpaper"), 4 );
+            popup.insertItem(SmallIconSet("background"), i18n( "Set as &Wallpaper" ), 4 );
+        popup.insertSeparator();
+        popup.insertItem(SmallIconSet("cancel"), i18n( "C&ancel" ), 5);

         int result = popup.exec( m_info->mousePos );
         switch (result) {
@@ -579,6 +581,7 @@ void KonqOperations::doFileCopy()
             delete this;
             return;
         }
+        case 5 :
         default : delete this; return;
         }
     }