Browse Source

Update tablet UI

- Only used when width is >= 720dp instead of 600dp (typically 10" tablets)
- Fix fastscroll in manga view (fixes #5267)
arkon 3 years ago
parent
commit
b03ebc1fa4

+ 1 - 1
app/build.gradle.kts

@@ -156,7 +156,7 @@ dependencies {
     implementation("androidx.work:work-runtime-ktx:2.7.0-alpha04")
 
     // UI library
-    implementation("com.google.android.material:material:1.4.0-beta01")
+    implementation("com.google.android.material:material:1.4.0-rc01")
 
     "standardImplementation"("com.google.firebase:firebase-core:19.0.0")
 

+ 2 - 2
app/src/main/java/eu/kanade/tachiyomi/util/system/ContextExtensions.kt

@@ -271,8 +271,8 @@ fun Context.createFileInCacheDir(name: String): File {
 }
 
 /**
- * We consider anything with a width of >= 600dp as a tablet, i.e. with layouts in layout-sw600dp.
+ * We consider anything with a width of >= 720dp as a tablet, i.e. with layouts in layout-w720dp.
  */
 fun Context.isTablet(): Boolean {
-    return resources.configuration.screenWidthDp >= 600
+    return (resources.displayMetrics.widthPixels / resources.displayMetrics.density) >= 720
 }

+ 1 - 0
app/src/main/res/layout-sw600dp/main_activity.xml → app/src/main/res/layout-w720dp/main_activity.xml

@@ -75,6 +75,7 @@
             style="@style/Widget.MaterialComponents.NavigationRailView.Colored"
             android:layout_width="wrap_content"
             android:layout_height="0dp"
+            app:elevation="0dp"
             app:itemIconTint="@color/nav_selector"
             app:itemRippleColor="?attr/rippleSecondaryColor"
             app:itemTextColor="@color/nav_selector"

+ 9 - 12
app/src/main/res/layout-sw600dp/manga_controller.xml → app/src/main/res/layout-w720dp/manga_controller.xml

@@ -50,18 +50,6 @@
                 app:layout_constraintTop_toTopOf="parent"
                 tools:listitem="@layout/chapters_item" />
 
-            <eu.kanade.tachiyomi.widget.MaterialFastScroll
-                android:id="@+id/fast_scroller"
-                android:layout_width="wrap_content"
-                android:layout_height="match_parent"
-                android:layout_centerHorizontal="true"
-                android:layout_gravity="end"
-                app:fastScrollerBubbleEnabled="false"
-                app:layout_constraintBottom_toBottomOf="parent"
-                app:layout_constraintEnd_toEndOf="parent"
-                app:layout_constraintTop_toTopOf="parent"
-                tools:visibility="visible" />
-
             <eu.kanade.tachiyomi.widget.ActionToolbar
                 android:id="@+id/action_toolbar"
                 android:layout_width="0dp"
@@ -76,4 +64,13 @@
 
     </eu.kanade.tachiyomi.widget.ThemedSwipeRefreshLayout>
 
+    <eu.kanade.tachiyomi.widget.MaterialFastScroll
+        android:id="@+id/fast_scroller"
+        android:layout_width="wrap_content"
+        android:layout_height="match_parent"
+        android:layout_centerHorizontal="true"
+        android:layout_gravity="end"
+        app:fastScrollerBubbleEnabled="false"
+        tools:visibility="visible" />
+
 </androidx.coordinatorlayout.widget.CoordinatorLayout>