From 1b836af9612fe623e9660b15ec534295d078217d Mon Sep 17 00:00:00 2001 From: "B. Petersen" Date: Sat, 25 Jan 2020 23:17:18 +0100 Subject: [PATCH] do no clear controls on saving drafts, we save drafts in onPause() now and the activity may be resumed --- .../securesms/ConversationActivity.java | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/src/org/thoughtcrime/securesms/ConversationActivity.java b/src/org/thoughtcrime/securesms/ConversationActivity.java index d0a041a38..a00ab2fe5 100644 --- a/src/org/thoughtcrime/securesms/ConversationActivity.java +++ b/src/org/thoughtcrime/securesms/ConversationActivity.java @@ -953,7 +953,7 @@ public class ConversationActivity extends PassphraseRequiredActionBarActivity } } - //////// send message + //////// send message or save draft protected static final int ACTION_SEND_OUT = 1; protected static final int ACTION_SAVE_DRAFT = 2; @@ -971,10 +971,17 @@ public class ConversationActivity extends PassphraseRequiredActionBarActivity DcMsg msg = null; Integer recompress = 0; - composeText.setText(""); + // for a quick ui feedback, we clear the related controls immediately on sending messages. + // for drafts, however, we do not change the controls, the activity may be resumed. + if (action==ACTION_SEND_OUT) { + composeText.setText(""); + } if(slideDeck!=null) { - attachmentManager.clear(glideRequests, false); + + if (action==ACTION_SEND_OUT) { + attachmentManager.clear(glideRequests, false); + } try { List attachments = slideDeck.asAttachments();