From 3c598f459ae25aecb0991fd32154cdab904b6c58 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Rom=C3=A1n?= <46288465+D3xron@users.noreply.github.com>
Date: Sat, 15 Jan 2022 22:58:39 +0000
Subject: [PATCH] Adding more size to few things (#495)

* Adding more size to few things

* Fixing margins and paddings

* Reverting categories_item

* Increasing size of innerContainer
---
 .../biometric/BiometricTimesHolder.kt         |  2 +-
 .../tachiyomi/ui/category/repos/RepoHolder.kt |  2 +-
 .../category/sources/SourceCategoryHolder.kt  |  2 +-
 .../res/layout/migration_bottom_sheet.xml     |  1 +
 .../main/res/layout/migration_manga_card.xml  |  4 +-
 app/src/main/res/layout/reader_activity.xml   | 50 ++++++++++---------
 6 files changed, 33 insertions(+), 28 deletions(-)

diff --git a/app/src/main/java/eu/kanade/tachiyomi/ui/category/biometric/BiometricTimesHolder.kt b/app/src/main/java/eu/kanade/tachiyomi/ui/category/biometric/BiometricTimesHolder.kt
index 1f03b6912..2d1817278 100644
--- a/app/src/main/java/eu/kanade/tachiyomi/ui/category/biometric/BiometricTimesHolder.kt
+++ b/app/src/main/java/eu/kanade/tachiyomi/ui/category/biometric/BiometricTimesHolder.kt
@@ -24,7 +24,7 @@ class BiometricTimesHolder(view: View, val adapter: BiometricTimesAdapter) : Fle
      */
     @OptIn(ExperimentalTime::class)
     fun bind(timeRange: TimeRange) {
-        binding.innerContainer.minimumHeight = 48.dpToPx
+        binding.innerContainer.minimumHeight = 60.dpToPx
 
         // Set capitalized title.
         binding.title.text = timeRange.getFormattedString(itemView.context)
diff --git a/app/src/main/java/eu/kanade/tachiyomi/ui/category/repos/RepoHolder.kt b/app/src/main/java/eu/kanade/tachiyomi/ui/category/repos/RepoHolder.kt
index 554b3dea0..bcdcd55c6 100644
--- a/app/src/main/java/eu/kanade/tachiyomi/ui/category/repos/RepoHolder.kt
+++ b/app/src/main/java/eu/kanade/tachiyomi/ui/category/repos/RepoHolder.kt
@@ -22,7 +22,7 @@ class RepoHolder(view: View, val adapter: RepoAdapter) : FlexibleViewHolder(view
      * @param category The category to bind.
      */
     fun bind(category: String) {
-        binding.innerContainer.minimumHeight = 48.dpToPx
+        binding.innerContainer.minimumHeight = 60.dpToPx
 
         // Set capitalized title.
         binding.title.text = category
diff --git a/app/src/main/java/eu/kanade/tachiyomi/ui/category/sources/SourceCategoryHolder.kt b/app/src/main/java/eu/kanade/tachiyomi/ui/category/sources/SourceCategoryHolder.kt
index 95c920cd0..7bd8a12cb 100644
--- a/app/src/main/java/eu/kanade/tachiyomi/ui/category/sources/SourceCategoryHolder.kt
+++ b/app/src/main/java/eu/kanade/tachiyomi/ui/category/sources/SourceCategoryHolder.kt
@@ -22,7 +22,7 @@ class SourceCategoryHolder(view: View, val adapter: SourceCategoryAdapter) : Fle
      * @param category The category to bind.
      */
     fun bind(category: String) {
-        binding.innerContainer.minimumHeight = 48.dpToPx
+        binding.innerContainer.minimumHeight = 60.dpToPx
 
         // Set capitalized title.
         binding.title.text = category
diff --git a/app/src/main/res/layout/migration_bottom_sheet.xml b/app/src/main/res/layout/migration_bottom_sheet.xml
index 785b26675..7b76b286f 100644
--- a/app/src/main/res/layout/migration_bottom_sheet.xml
+++ b/app/src/main/res/layout/migration_bottom_sheet.xml
@@ -36,6 +36,7 @@
             android:layout_height="wrap_content"
             android:layout_marginStart="8dp"
             android:layout_marginTop="8dp"
+            android:layout_marginEnd="8dp"
             android:scrollbars="none"
             app:layout_constraintEnd_toEndOf="parent"
             app:layout_constraintStart_toStartOf="parent"
diff --git a/app/src/main/res/layout/migration_manga_card.xml b/app/src/main/res/layout/migration_manga_card.xml
index d906f2099..bc47510ab 100644
--- a/app/src/main/res/layout/migration_manga_card.xml
+++ b/app/src/main/res/layout/migration_manga_card.xml
@@ -114,7 +114,7 @@
             android:ellipsize="end"
             android:maxLines="1"
             android:paddingTop="4dp"
-            android:paddingBottom="2dp"
+            android:paddingBottom="1dp"
             android:textAppearance="?attr/textAppearanceTitleSmall"
             android:textIsSelectable="false"
             app:layout_constraintLeft_toLeftOf="parent"
@@ -125,7 +125,7 @@
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
             android:clickable="false"
-            android:paddingTop="2dp"
+            android:paddingTop="1dp"
             android:paddingBottom="4dp"
             android:textAppearance="?attr/textAppearanceBodyMedium"
             android:textColor="?android:attr/textColorSecondary"
diff --git a/app/src/main/res/layout/reader_activity.xml b/app/src/main/res/layout/reader_activity.xml
index 0621f0cf7..b4d711b3f 100755
--- a/app/src/main/res/layout/reader_activity.xml
+++ b/app/src/main/res/layout/reader_activity.xml
@@ -82,41 +82,43 @@
 
                     <com.google.android.material.switchmaterial.SwitchMaterial
                         android:id="@+id/eh_autoscroll"
-                        android:layout_width="wrap_content"
+                        android:layout_width="0dp"
                         android:layout_height="match_parent"
                         android:layout_weight="1"
                         android:gravity="center_horizontal|center_vertical"
-                        android:padding="8dp"
+                        android:paddingStart="16dp"
+                        android:paddingEnd="16dp"
                         android:text="@string/eh_autoscroll"
-                        android:textColor="?attr/colorOnSurface"/>
+                        android:textColor="?attr/colorOnSurface"
+                        android:textSize="13sp" />
 
                     <EditText
                         android:id="@+id/eh_autoscroll_freq"
-                        android:layout_width="50dp"
-                        android:layout_height="wrap_content"
-                        android:layout_weight="0.2"
-                        android:ems="10"
-                        android:gravity="start|center_vertical"
+                        android:layout_width="100dp"
+                        android:layout_height="match_parent"
+                        android:backgroundTint="?attr/colorAccent"
+                        android:gravity="center_horizontal|center_vertical"
                         android:importantForAutofill="no"
                         android:inputType="numberDecimal"
-                        android:maxLength="10"/>
+                        android:maxLength="10"
+                        android:singleLine="true" />
 
                     <Button
                         android:id="@+id/eh_autoscroll_help"
                         style="@style/Widget.MaterialComponents.Button.TextButton"
-                        android:layout_width="40dp"
-                        android:layout_height="wrap_content"
+                        android:layout_width="50dp"
+                        android:layout_height="match_parent"
                         android:text="\?"
                         android:textColor="?attr/colorOnSurface"
                         android:textSize="15sp"
                         android:textStyle="bold"
-                        tools:ignore="HardcodedText"/>
+                        tools:ignore="HardcodedText" />
 
                 </LinearLayout>
 
                 <LinearLayout
                     android:layout_width="match_parent"
-                    android:layout_height="match_parent"
+                    android:layout_height="0dp"
                     android:layout_weight="1"
                     android:orientation="horizontal">
 
@@ -124,43 +126,45 @@
                         android:id="@+id/eh_retry_all"
                         style="@style/Widget.MaterialComponents.Button.TextButton"
                         android:layout_width="match_parent"
-                        android:layout_height="wrap_content"
+                        android:layout_height="match_parent"
                         android:layout_weight="1"
                         android:fontFamily="sans-serif"
                         android:text="@string/eh_retry_all"
-                        android:textColor="?attr/colorOnSurface"/>
+                        android:textColor="?attr/colorOnSurface"
+                        android:textSize="13sp" />
 
                     <Button
                         android:id="@+id/eh_retry_all_help"
                         style="@style/Widget.MaterialComponents.Button.TextButton"
-                        android:layout_width="40dp"
-                        android:layout_height="wrap_content"
+                        android:layout_width="50dp"
+                        android:layout_height="match_parent"
                         android:text="\?"
                         android:textColor="?attr/colorOnSurface"
                         android:textSize="15sp"
                         android:textStyle="bold"
-                        tools:ignore="HardcodedText"/>
+                        tools:ignore="HardcodedText" />
 
                     <Button
                         android:id="@+id/eh_boost_page"
                         style="@style/Widget.MaterialComponents.Button.TextButton"
                         android:layout_width="match_parent"
-                        android:layout_height="wrap_content"
+                        android:layout_height="match_parent"
                         android:layout_weight="1"
                         android:fontFamily="sans-serif"
                         android:text="@string/eh_boost_page"
-                        android:textColor="?attr/colorOnSurface"/>
+                        android:textColor="?attr/colorOnSurface"
+                        android:textSize="13sp" />
 
                     <Button
                         android:id="@+id/eh_boost_page_help"
                         style="@style/Widget.MaterialComponents.Button.TextButton"
-                        android:layout_width="40dp"
-                        android:layout_height="wrap_content"
+                        android:layout_width="50dp"
+                        android:layout_height="match_parent"
                         android:text="\?"
                         android:textColor="?attr/colorOnSurface"
                         android:textSize="15sp"
                         android:textStyle="bold"
-                        tools:ignore="HardcodedText"/>
+                        tools:ignore="HardcodedText" />
                 </LinearLayout>
 
             </LinearLayout>