use setText instead of append()

Daniel Gultsch created 2 years ago

Change summary

src/main/java/eu/siacs/conversations/ui/ConversationFragment.java | 6 
src/main/java/eu/siacs/conversations/utils/MessageUtils.java      | 2 
2 files changed, 5 insertions(+), 3 deletions(-)

Detailed changes

src/main/java/eu/siacs/conversations/ui/ConversationFragment.java

@@ -2792,12 +2792,14 @@ public class ConversationFragment extends XmppFragment
         this.binding.textSendButton.setContentDescription(
                 activity.getString(R.string.send_message_to_x, conversation.getName()));
         this.binding.textinput.setKeyboardListener(null);
-        this.binding.textinput.setText("");
         final boolean participating =
                 conversation.getMode() == Conversational.MODE_SINGLE
                         || conversation.getMucOptions().participating();
         if (participating) {
-            this.binding.textinput.append(this.conversation.getNextMessage());
+            this.binding.textinput.setText(this.conversation.getNextMessage());
+            this.binding.textinput.setSelection(this.binding.textinput.length());
+        } else {
+            this.binding.textinput.setText(MessageUtils.EMPTY_STRING);
         }
         this.binding.textinput.setKeyboardListener(this);
         messageListAdapter.updatePreferences();

src/main/java/eu/siacs/conversations/utils/MessageUtils.java

@@ -45,7 +45,7 @@ public class MessageUtils {
 
     private static final Pattern LTR_RTL = Pattern.compile("(\\u200E[^\\u200F]*\\u200F){3,}");
 
-    private static final String EMPTY_STRING = "";
+    public static final String EMPTY_STRING = "";
 
     public static String prepareQuote(Message message) {
         final StringBuilder builder = new StringBuilder();