Author: yurize
Date: Thu Jul 26 20:51:56 2012
New Revision: 1366196

URL: http://svn.apache.org/viewvc?rev=1366196&view=rev
Log:
Adds blip deletion confirmation dialog. By Vicente J. Ruiz Jurado.
https://reviews.apache.org/r/6152

Modified:
    
incubator/wave/trunk/src/org/waveprotocol/wave/client/wavepanel/impl/menu/MenuController.java

Modified: 
incubator/wave/trunk/src/org/waveprotocol/wave/client/wavepanel/impl/menu/MenuController.java
URL: 
http://svn.apache.org/viewvc/incubator/wave/trunk/src/org/waveprotocol/wave/client/wavepanel/impl/menu/MenuController.java?rev=1366196&r1=1366195&r2=1366196&view=diff
==============================================================================
--- 
incubator/wave/trunk/src/org/waveprotocol/wave/client/wavepanel/impl/menu/MenuController.java
 (original)
+++ 
incubator/wave/trunk/src/org/waveprotocol/wave/client/wavepanel/impl/menu/MenuController.java
 Thu Jul 26 20:51:56 2012
@@ -19,6 +19,7 @@ package org.waveprotocol.wave.client.wav
 import com.google.gwt.dom.client.Element;
 import com.google.gwt.dom.client.NativeEvent;
 import com.google.gwt.event.dom.client.ClickEvent;
+import com.google.gwt.user.client.Window;
 
 import org.waveprotocol.wave.client.wavepanel.WavePanel;
 import org.waveprotocol.wave.client.wavepanel.event.WaveClickHandler;
@@ -73,7 +74,10 @@ public final class MenuController implem
         actions.reply(item.getParent().getParent());
         break;
       case DELETE:
-        actions.delete(item.getParent().getParent());
+        // We delete the blip without confirmation if shift key is pressed
+        if (event.getNativeEvent().getShiftKey()
+            || Window.confirm("Please confirm the deletion of this message"))
+          actions.delete(item.getParent().getParent());
         break;
       case LINK:
         actions.popupLink(item.getParent().getParent());


Reply via email to