From 91eb01f58450a4f474306ab2af22846d8793ca4b Mon Sep 17 00:00:00 2001 From: Jonathan Klee Date: Mon, 17 Mar 2025 19:18:29 +0100 Subject: [PATCH] Do not scroll to top all the time Only when we have a new search and when inserting at position 0 --- .../java/foundation/e/apps/ui/search/SearchFragment.kt | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/app/src/main/java/foundation/e/apps/ui/search/SearchFragment.kt b/app/src/main/java/foundation/e/apps/ui/search/SearchFragment.kt index b05fcb277..4c0df8b7d 100644 --- a/app/src/main/java/foundation/e/apps/ui/search/SearchFragment.kt +++ b/app/src/main/java/foundation/e/apps/ui/search/SearchFragment.kt @@ -187,7 +187,6 @@ class SearchFragment : } updateSearchResult(listAdapter, apps ?: emptyList()) observeScrollOfSearchResult(listAdapter) - scrollToTop() } } @@ -205,8 +204,8 @@ class SearchFragment : * Compare lastSearch with searchText to avoid falsely updating to * current query text even before submitting the new search. */ - if (lastSearch != searchText) { - recyclerView?.scrollToPosition(0) + if (lastSearch != searchText && positionStart == 0) { + scrollToTop() lastSearch = searchText } } @@ -238,7 +237,7 @@ class SearchFragment : searchHintLayout?.visibility = View.GONE } - private fun scrollToTop() { + fun scrollToTop() { // Scroll to the top with some delays so that the recycler view has the time // to process the new results recyclerView?.postDelayed( -- GitLab