From 3287ce719bb1e298436be2747877da7fb4caa2b7 Mon Sep 17 00:00:00 2001
From: M66B <M66B@users.noreply.github.com>
Date: Wed, 12 Jun 2019 22:21:55 +0200
Subject: [PATCH] Improved displaying no messages

---
 .../email/BoundaryCallbackMessages.java       |  8 +++----
 .../eu/faircode/email/FragmentMessages.java   | 24 ++++++-------------
 2 files changed, 11 insertions(+), 21 deletions(-)

diff --git a/app/src/main/java/eu/faircode/email/BoundaryCallbackMessages.java b/app/src/main/java/eu/faircode/email/BoundaryCallbackMessages.java
index 27efbad998..4934711cdf 100644
--- a/app/src/main/java/eu/faircode/email/BoundaryCallbackMessages.java
+++ b/app/src/main/java/eu/faircode/email/BoundaryCallbackMessages.java
@@ -113,16 +113,16 @@ public class BoundaryCallbackMessages extends PagedList.BoundaryCallback<TupleMe
     @Override
     public void onZeroItemsLoaded() {
         Log.i("Boundary zero loaded");
-        queue_load();
+        queue_load(true);
     }
 
     @Override
     public void onItemAtEndLoaded(final TupleMessageEx itemAtEnd) {
         Log.i("Boundary at end");
-        queue_load();
+        queue_load(false);
     }
 
-    private void queue_load() {
+    private void queue_load(final boolean zero) {
         executor.submit(new Runnable() {
             private int fetched;
 
@@ -158,7 +158,7 @@ public class BoundaryCallbackMessages extends PagedList.BoundaryCallback<TupleMe
                         handler.post(new Runnable() {
                             @Override
                             public void run() {
-                                intf.onLoaded(fetched);
+                                intf.onLoaded(zero ? fetched : Integer.MAX_VALUE);
                             }
                         });
                 }
diff --git a/app/src/main/java/eu/faircode/email/FragmentMessages.java b/app/src/main/java/eu/faircode/email/FragmentMessages.java
index 033886eb69..e466c81837 100644
--- a/app/src/main/java/eu/faircode/email/FragmentMessages.java
+++ b/app/src/main/java/eu/faircode/email/FragmentMessages.java
@@ -2700,21 +2700,16 @@ public class FragmentMessages extends FragmentBase implements SharedPreferences.
         public void onLoading() {
             loading = true;
             pbWait.setVisibility(View.VISIBLE);
-            tvNoEmail.setVisibility(View.GONE);
         }
 
         @Override
         public void onLoaded(int fetched) {
             loading = false;
 
-            Integer submitted = (Integer) rvMessage.getTag();
-            if (submitted == null)
-                return;
-
             if (initialized && busy == 0)
                 pbWait.setVisibility(View.GONE);
-            if (submitted + fetched == 0)
-                tvNoEmail.setVisibility(View.VISIBLE);
+
+            tvNoEmail.setVisibility(fetched == 0 ? View.VISIBLE : View.GONE);
         }
 
         @Override
@@ -2752,18 +2747,13 @@ public class FragmentMessages extends FragmentBase implements SharedPreferences.
                 }
             });
 
-            rvMessage.setTag(messages.size());
-
             initialized = true;
-            if (!loading && busy == 0) {
+            if (!loading && busy == 0)
                 pbWait.setVisibility(View.GONE);
-                if (messages.size() == 0)
-                    tvNoEmail.setVisibility(View.VISIBLE);
-            }
-            if (messages.size() > 0) {
-                tvNoEmail.setVisibility(View.GONE);
-                grpReady.setVisibility(View.VISIBLE);
-            }
+
+            if (viewType != AdapterMessage.ViewType.SEARCH)
+                tvNoEmail.setVisibility(messages.size() == 0 && !loading ? View.VISIBLE : View.GONE);
+            grpReady.setVisibility(View.VISIBLE);
         }
     };