diff options
author | Zalan Blenessy <zalanb@axis.com> | 2015-01-15 10:59:24 +0100 |
---|---|---|
committer | David Pursehouse <david.pursehouse@sonymobile.com> | 2015-01-15 10:57:07 +0000 |
commit | 92b976c2f4e0447780dae4ffb31ec960d4c70630 (patch) | |
tree | 5c9fd81241d4afa11f8876cf4de1ae91bffea44c | |
parent | 7794b9e5e83590189ad5b151bb68decf6a8bc1e1 (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.java | 12 |
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(); } } |