Removing redundant logic

Author: Scapesfear

AnkiDroid/src/main/java/com/ichi2/anki/CardBrowser.kt

@@ -1916,16 +1916,12 @@ open class CardBrowser :
 
     private fun saveScrollingState(id: CardOrNoteId) {
         viewModel.cards.indexOf(id).takeIf { it >= 0 }?.let { position ->
-            val layoutManager = cardsListView.layoutManager as LinearLayoutManager
-            val firstVisiblePosition = layoutManager.findFirstVisibleItemPosition()
-            val view = cardsListView.getChildAt(position - firstVisiblePosition)
             viewModel.lastSelectedPosition = position
-            viewModel.oldCardTopOffset = view?.top ?: 0
         }
     }
 
     private fun autoScrollTo(newPosition: Int) {
-        (cardsListView.layoutManager as LinearLayoutManager).scrollToPositionWithOffset(newPosition, viewModel.oldCardTopOffset)
+        (cardsListView.layoutManager as LinearLayoutManager).scrollToPosition(newPosition)
     }
 }
 

AnkiDroid/src/main/java/com/ichi2/anki/browser/CardBrowserViewModel.kt

@@ -97,7 +97,6 @@ class CardBrowserViewModel(
 ) : ViewModel(),
     SharedPreferencesProvider by preferences {
     var lastSelectedPosition: Int = 0
-    var oldCardTopOffset: Int = 0
 
     // TODO: abstract so we can use a `Context` and `pref_display_filenames_in_browser_key`
     val showMediaFilenames = sharedPrefs().getBoolean("card_browser_show_media_filenames", false)