From 7d75950624f56512a713a2b1ba4cbdc700d89673 Mon Sep 17 00:00:00 2001
From: Alexander-- <alexander.r@gmx.com>
Date: Thu, 14 Nov 2019 20:54:40 +0659
Subject: [PATCH] Disable srolling down comment list while comments are loading

Prevents comment list from losing focus to some outside View
when user tries to scroll down after reaching "end"
---
 .../fragments/list/BaseListInfoFragment.java  | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

diff --git a/app/src/main/java/org/schabi/newpipe/fragments/list/BaseListInfoFragment.java b/app/src/main/java/org/schabi/newpipe/fragments/list/BaseListInfoFragment.java
index 9a8e1fd17..7363d221c 100644
--- a/app/src/main/java/org/schabi/newpipe/fragments/list/BaseListInfoFragment.java
+++ b/app/src/main/java/org/schabi/newpipe/fragments/list/BaseListInfoFragment.java
@@ -10,6 +10,7 @@ import androidx.annotation.NonNull;
 import org.schabi.newpipe.extractor.ListExtractor;
 import org.schabi.newpipe.extractor.ListInfo;
 import org.schabi.newpipe.util.Constants;
+import org.schabi.newpipe.views.NewPipeRecyclerView;
 
 import java.util.Queue;
 
@@ -17,6 +18,8 @@ import icepick.State;
 import io.reactivex.Single;
 import io.reactivex.android.schedulers.AndroidSchedulers;
 import io.reactivex.disposables.Disposable;
+import io.reactivex.functions.Action;
+import io.reactivex.functions.Consumer;
 import io.reactivex.schedulers.Schedulers;
 
 public abstract class BaseListInfoFragment<I extends ListInfo>
@@ -136,9 +139,13 @@ public abstract class BaseListInfoFragment<I extends ListInfo>
         isLoading.set(true);
 
         if (currentWorker != null) currentWorker.dispose();
+
+        forbidDownwardFocusScroll();
+
         currentWorker = loadMoreItemsLogic()
                 .subscribeOn(Schedulers.io())
                 .observeOn(AndroidSchedulers.mainThread())
+                .doFinally(this::allowDownwardFocusScroll)
                 .subscribe((@io.reactivex.annotations.NonNull ListExtractor.InfoItemsPage InfoItemsPage) -> {
                     isLoading.set(false);
                     handleNextItems(InfoItemsPage);
@@ -148,6 +155,18 @@ public abstract class BaseListInfoFragment<I extends ListInfo>
                 });
     }
 
+    private void forbidDownwardFocusScroll() {
+        if (itemsList instanceof NewPipeRecyclerView) {
+            ((NewPipeRecyclerView) itemsList).setFocusScrollAllowed(false);
+        }
+    }
+
+    private void allowDownwardFocusScroll() {
+        if (itemsList instanceof NewPipeRecyclerView) {
+            ((NewPipeRecyclerView) itemsList).setFocusScrollAllowed(true);
+        }
+    }
+
     @Override
     public void handleNextItems(ListExtractor.InfoItemsPage result) {
         super.handleNextItems(result);