From d137cc8c855b98880d7771d62dd796d7e29aabdb Mon Sep 17 00:00:00 2001
From: M66B <M66B@users.noreply.github.com>
Date: Fri, 21 Dec 2018 18:45:38 +0100
Subject: [PATCH] Fixed message text positioning, small improvement

---
 .../java/eu/faircode/email/AdapterMessage.java  |  4 ++--
 .../java/eu/faircode/email/FragmentCompose.java | 17 ++++++++++++-----
 2 files changed, 14 insertions(+), 7 deletions(-)

diff --git a/app/src/main/java/eu/faircode/email/AdapterMessage.java b/app/src/main/java/eu/faircode/email/AdapterMessage.java
index fdb3118ec8..05cdfa49e2 100644
--- a/app/src/main/java/eu/faircode/email/AdapterMessage.java
+++ b/app/src/main/java/eu/faircode/email/AdapterMessage.java
@@ -248,8 +248,6 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
             grpHeaders = itemView.findViewById(R.id.grpHeaders);
             grpAttachments = itemView.findViewById(R.id.grpAttachments);
             grpExpanded = itemView.findViewById(R.id.grpExpanded);
-
-            tvBody.setMovementMethod(new UrlHandler());
         }
 
         private void wire() {
@@ -493,6 +491,7 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
                 ta.recycle();
 
                 tvBody.setText(null);
+                tvBody.setMovementMethod(null);
                 pbBody.setVisibility(View.VISIBLE);
 
                 if (message.content) {
@@ -849,6 +848,7 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
                 ibQuotes.setVisibility(has_quotes && show_expanded && !show_quotes ? View.VISIBLE : View.GONE);
                 ibImages.setVisibility(has_images && show_expanded && !show_images ? View.VISIBLE : View.GONE);
                 tvBody.setText(body);
+                tvBody.setMovementMethod(new UrlHandler());
                 pbBody.setVisibility(View.GONE);
             }
 
diff --git a/app/src/main/java/eu/faircode/email/FragmentCompose.java b/app/src/main/java/eu/faircode/email/FragmentCompose.java
index ac4125f9af..46a7c75fdb 100644
--- a/app/src/main/java/eu/faircode/email/FragmentCompose.java
+++ b/app/src/main/java/eu/faircode/email/FragmentCompose.java
@@ -33,7 +33,6 @@ import android.graphics.drawable.Drawable;
 import android.net.Uri;
 import android.os.Bundle;
 import android.os.Handler;
-import android.os.Looper;
 import android.preference.PreferenceManager;
 import android.provider.ContactsContract;
 import android.provider.OpenableColumns;
@@ -1687,6 +1686,13 @@ public class FragmentCompose extends FragmentEx {
                         EntityMessage replying = db.message().getMessage(draft.replying);
                         EntityOperation.queue(db, replying, EntityOperation.ANSWERED, true);
                     }
+
+                    Handler handler = new Handler(context.getMainLooper());
+                    handler.post(new Runnable() {
+                        public void run() {
+                            Toast.makeText(context, R.string.title_queued, Toast.LENGTH_LONG).show();
+                        }
+                    });
                 }
 
                 db.setTransactionSuccessful();
@@ -1712,7 +1718,7 @@ public class FragmentCompose extends FragmentEx {
 
             if (action == R.id.action_delete) {
                 autosave = false;
-                getFragmentManager().popBackStack();
+                finish();
 
             } else if (action == R.id.action_save) {
                 // Do nothing
@@ -1722,8 +1728,7 @@ public class FragmentCompose extends FragmentEx {
 
             } else if (action == R.id.action_send) {
                 autosave = false;
-                getFragmentManager().popBackStack();
-                Toast.makeText(getContext(), R.string.title_queued, Toast.LENGTH_LONG).show();
+                finish();
             }
         }
 
@@ -1733,7 +1738,9 @@ public class FragmentCompose extends FragmentEx {
             Helper.setViewsEnabled(view, true);
             getActivity().invalidateOptionsMenu();
 
-            if (ex instanceof IllegalArgumentException)
+            if (ex instanceof MessageRemovedException)
+                finish();
+            else if (ex instanceof IllegalArgumentException)
                 Snackbar.make(view, ex.getMessage(), Snackbar.LENGTH_LONG).show();
             else
                 Helper.unexpectedError(getContext(), getViewLifecycleOwner(), ex);