From 2d1e85f280c268294ef39dee1bfd19b22582941a Mon Sep 17 00:00:00 2001
From: len <len@kanade.eu>
Date: Thu, 9 Jun 2016 14:27:11 +0200
Subject: [PATCH] Fix scroll position with many categories. Closes #332

---
 .../eu/kanade/tachiyomi/ui/library/LibraryFragment.kt | 11 +++--------
 app/src/main/res/layout/activity_main.xml             |  2 --
 2 files changed, 3 insertions(+), 10 deletions(-)

diff --git a/app/src/main/java/eu/kanade/tachiyomi/ui/library/LibraryFragment.kt b/app/src/main/java/eu/kanade/tachiyomi/ui/library/LibraryFragment.kt
index 0558eb9f6..d03415cac 100644
--- a/app/src/main/java/eu/kanade/tachiyomi/ui/library/LibraryFragment.kt
+++ b/app/src/main/java/eu/kanade/tachiyomi/ui/library/LibraryFragment.kt
@@ -139,6 +139,7 @@ class LibraryFragment : BaseRxFragment<LibraryPresenter>(), ActionMode.Callback
     }
 
     override fun onDestroyView() {
+        tabs.setupWithViewPager(null)
         tabs.visibility = View.GONE
         super.onDestroyView()
     }
@@ -261,18 +262,12 @@ class LibraryFragment : BaseRxFragment<LibraryPresenter>(), ActionMode.Callback
 
         // Set the categories
         adapter.categories = categories
-        tabs.setupWithViewPager(view_pager)
         tabs.visibility = if (categories.size <= 1) View.GONE else View.VISIBLE
 
         // Restore active category.
         view_pager.setCurrentItem(activeCat, false)
-        if (tabs.tabCount > 0) {
-            // Prevent IndexOutOfBoundsException
-            if (tabs.tabCount <= view_pager.currentItem) {
-                view_pager.currentItem = (tabs.tabCount - 1)
-            }
-            tabs.getTabAt(view_pager.currentItem)?.select()
-        }
+        // Delay the scroll position to allow the view to be properly measured.
+        view_pager.post { tabs.setScrollPosition(view_pager.currentItem, 0f, true) }
 
         // Send the manga map to child fragments after the adapter is updated.
         presenter.libraryMangaSubject.onNext(LibraryMangaEvent(mangaMap))
diff --git a/app/src/main/res/layout/activity_main.xml b/app/src/main/res/layout/activity_main.xml
index f657d1dc0..10b7b3f8e 100644
--- a/app/src/main/res/layout/activity_main.xml
+++ b/app/src/main/res/layout/activity_main.xml
@@ -20,8 +20,6 @@
             <include layout="@layout/toolbar"/>
 
             <android.support.design.widget.TabLayout
-                xmlns:android="http://schemas.android.com/apk/res/android"
-                xmlns:app="http://schemas.android.com/apk/res-auto"
                 android:id="@+id/tabs"
                 android:layout_width="match_parent"
                 android:layout_height="wrap_content"