Check only for Visible Item, not completely Visible; Fix #180

niftynei · niftynei · commit 026c4a211835 · 2016-10-20T13:00:23.000-04:00
diff --git a/app/src/main/java/com/zulip/android/activities/MessageListFragment.java b/app/src/main/java/com/zulip/android/activities/MessageListFragment.java
@@ -163,7 +163,7 @@ public void onScrollStateChanged(RecyclerView recyclerView, int newState) {
                             Log.i("scroll", "Starting request below");
                             loadMoreMessages(LoadPosition.BELOW);
                         }
-                        if (linearLayoutManager.findFirstCompletelyVisibleItemPosition() < near && !loadedToTop) {
+                        if (linearLayoutManager.findFirstVisibleItemPosition() < near && !loadedToTop) {
                             // At the top of the list
                             Log.i("scroll", "Starting request above");
                             loadMoreMessages(LoadPosition.ABOVE);

Original file line number	Diff line number	Diff line change
`@@ -163,7 +163,7 @@ public void onScrollStateChanged(RecyclerView recyclerView, int newState) {`
`163`	`163`	`Log.i("scroll", "Starting request below");`
`164`	`164`	`loadMoreMessages(LoadPosition.BELOW);`
`165`	`165`	`}`
`166`		`- if (linearLayoutManager.findFirstCompletelyVisibleItemPosition() < near && !loadedToTop) {`
	`166`	`+ if (linearLayoutManager.findFirstVisibleItemPosition() < near && !loadedToTop) {`
`167`	`167`	`// At the top of the list`
`168`	`168`	`Log.i("scroll", "Starting request above");`
`169`	`169`	`loadMoreMessages(LoadPosition.ABOVE);`