diff options
author | Shawn Pearce <sop@google.com> | 2014-04-24 12:01:57 -0700 |
---|---|---|
committer | David Pursehouse <david.pursehouse@sonymobile.com> | 2014-04-25 09:40:45 +0900 |
commit | 0e26575790a7502a83fec0efde7e084fba6063ac (patch) | |
tree | 0cf232690a8518675bb5d0b88384ff8598c945b3 | |
parent | 9fa4975b821dc92d35219037d02629795fd3e3c8 (diff) |
ChangeScreen2: Only reset the commit message text on cancel
Allow the user to begin editing the commit message, dismiss the
box by clicking outside of it (e.g. to copy part of a file name
from the Files table), and then re-open the current draft text
without resetting the box.
Only reset the box when the user explicitly clicks Cancel. Track
this by using the empty string in the text editor to mean there
is no valid message and the current original should be used.
Change-Id: Icd99b92c50ff92e0b1dc31a78fc57612f57a143b
-rw-r--r-- | gerrit-gwtui/src/main/java/com/google/gerrit/client/change/EditMessageBox.java | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/EditMessageBox.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/EditMessageBox.java index 2d444f0241..a6671977d5 100644 --- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/EditMessageBox.java +++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/EditMessageBox.java @@ -55,6 +55,7 @@ class EditMessageBox extends Composite { this.revision = revision; this.originalMessage = msg.trim(); initWidget(uiBinder.createAndBindUi(this)); + message.setText(""); new TextBoxChangeListener(message) { public void onTextChanged(String newText) { save.setEnabled(!newText.trim() @@ -65,8 +66,10 @@ class EditMessageBox extends Composite { @Override protected void onLoad() { - message.setText(originalMessage); - save.setEnabled(false); + if (message.getText().isEmpty()) { + message.setText(originalMessage); + save.setEnabled(false); + } Scheduler.get().scheduleDeferred(new ScheduledCommand() { @Override public void execute() { @@ -89,6 +92,7 @@ class EditMessageBox extends Composite { @UiHandler("cancel") void onCancel(ClickEvent e) { + message.setText(""); hide(); } |