summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorZalan Blenessy <zalanb@axis.com>2015-01-15 10:59:24 +0100
committerDavid Pursehouse <david.pursehouse@sonymobile.com>2015-01-15 10:57:07 +0000
commit92b976c2f4e0447780dae4ffb31ec960d4c70630 (patch)
tree5c9fd81241d4afa11f8876cf4de1ae91bffea44c
parent7794b9e5e83590189ad5b151bb68decf6a8bc1e1 (diff)
Re-enable the Cherry Pick button when cancelled
If the Cherry Pick dialog is cancelled, the Cherry Pick button cannot be clicked again. Change-Id: I5c2fb48ae4b01cfa122797b84a226cfb4ef6efe5 (cherry picked from commit 06810e758c7ea0eabf7ff43d785c65e2c90d1373)
-rw-r--r--gerrit-gwtui/src/main/java/com/google/gerrit/client/change/CherryPickAction.java12
1 files changed, 11 insertions, 1 deletions
diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/CherryPickAction.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/CherryPickAction.java
index 10d72979ad..6b442990dd 100644
--- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/CherryPickAction.java
+++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/CherryPickAction.java
@@ -23,10 +23,14 @@ import com.google.gerrit.client.ui.CherryPickDialog;
import com.google.gerrit.common.PageLinks;
import com.google.gerrit.reviewdb.client.Change;
import com.google.gerrit.reviewdb.client.Project;
+import com.google.gwt.event.logical.shared.CloseEvent;
import com.google.gwt.user.client.ui.Button;
+import com.google.gwt.user.client.ui.FocusWidget;
+import com.google.gwt.user.client.ui.PopupPanel;
+
class CherryPickAction {
- static void call(Button b, final ChangeInfo info, final String revision,
+ static void call(final Button b, final ChangeInfo info, final String revision,
String project, final String commitMessage) {
// TODO Replace CherryPickDialog with a nicer looking display.
b.setEnabled(false);
@@ -62,6 +66,12 @@ class CherryPickAction {
}
});
}
+
+ @Override
+ public void onClose(CloseEvent<PopupPanel> event) {
+ super.onClose(event);
+ b.setEnabled(true);
+ }
}.center();
}
}