From 38b50984923fae619e54fc55fc0ef408028832c2 Mon Sep 17 00:00:00 2001 From: Marcel Hibbe Date: Wed, 16 Jul 2025 15:58:44 +0200 Subject: [PATCH] move saving of conversation list scroll-position to onPause otherwise, it's would not work when just reopening the app from background Signed-off-by: Marcel Hibbe --- .../ConversationsListActivity.kt | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/app/src/main/java/com/nextcloud/talk/conversationlist/ConversationsListActivity.kt b/app/src/main/java/com/nextcloud/talk/conversationlist/ConversationsListActivity.kt index cae0a5c4a..5a4d30ad6 100644 --- a/app/src/main/java/com/nextcloud/talk/conversationlist/ConversationsListActivity.kt +++ b/app/src/main/java/com/nextcloud/talk/conversationlist/ConversationsListActivity.kt @@ -345,6 +345,16 @@ class ConversationsListActivity : showSearchOrToolbar() } + override fun onPause() { + super.onPause() + val firstVisible = layoutManager?.findFirstVisibleItemPosition() ?: 0 + val firstItem = adapter?.getItem(firstVisible) + val firstTop = (firstItem as ConversationItem).mHolder?.itemView?.top + val firstOffset = firstTop?.minus(CONVERSATION_ITEM_HEIGHT) ?: 0 + + appPreferences.setConversationListPositionAndOffset(firstVisible, firstOffset) + } + // if edge to edge is used, add an empty item at the bottom of the list @Suppress("MagicNumber") private fun addEmptyItemForEdgeToEdgeIfNecessary() { @@ -1881,12 +1891,6 @@ class ConversationsListActivity : bundle.putString(BundleKeys.KEY_MESSAGE_ID, selectedMessageId) selectedMessageId = null } - val firstVisible = layoutManager?.findFirstVisibleItemPosition() ?: 0 - val firstItem = adapter?.getItem(firstVisible) - val firstTop = (firstItem as ConversationItem).mHolder?.itemView?.top - val firstOffset = firstTop?.minus(CONVERSATION_ITEM_HEIGHT) ?: 0 - - appPreferences.setConversationListPositionAndOffset(firstVisible, firstOffset) val intent = Intent(context, ChatActivity::class.java) intent.putExtras(bundle)