From 5b9d2175e2b2feb17083964251566efba58e4b80 Mon Sep 17 00:00:00 2001
From: arkon <eugcheung94@gmail.com>
Date: Sat, 27 Mar 2021 10:53:19 -0400
Subject: [PATCH] Reorganize reader sheet contents a bit

(cherry picked from commit c255f57d956a6f81d4a1ea6c22e1ca29d81d28ba)

# Conflicts:
#	app/src/main/res/layout/reader_general_settings.xml
---
 .../res/layout/reader_general_settings.xml    | 24 +--------------
 .../layout/reader_reading_mode_settings.xml   | 29 ++++++-------------
 2 files changed, 10 insertions(+), 43 deletions(-)

diff --git a/app/src/main/res/layout/reader_general_settings.xml b/app/src/main/res/layout/reader_general_settings.xml
index 26bcd765e..a64cac3a5 100644
--- a/app/src/main/res/layout/reader_general_settings.xml
+++ b/app/src/main/res/layout/reader_general_settings.xml
@@ -11,24 +11,10 @@
         android:layout_width="match_parent"
         android:layout_height="match_parent">
 
-        <!-- General reading mode preferences -->
-
-        <TextView
-            android:id="@+id/display_prefs"
-            android:layout_width="0dp"
-            android:layout_height="wrap_content"
-            android:layout_marginTop="24dp"
-            android:text="@string/pref_category_display"
-            android:textColor="?attr/colorAccent"
-            android:textStyle="bold"
-            app:layout_constraintStart_toStartOf="parent"
-            app:layout_constraintTop_toBottomOf="@+id/page_transitions" />
-
         <TextView
             android:id="@+id/rotation_mode_text"
             android:layout_width="0dp"
             android:layout_height="wrap_content"
-            android:layout_marginTop="20dp"
             android:text="@string/pref_rotation_type"
             app:layout_constraintBaseline_toBaselineOf="@id/rotation_mode"
             app:layout_constraintEnd_toStartOf="@id/verticalcenter"
@@ -38,11 +24,10 @@
             android:id="@+id/rotation_mode"
             android:layout_width="0dp"
             android:layout_height="wrap_content"
-            android:layout_marginTop="20dp"
             android:entries="@array/rotation_type"
             app:layout_constraintEnd_toEndOf="@id/spinner_end"
             app:layout_constraintStart_toEndOf="@id/verticalcenter"
-            app:layout_constraintTop_toBottomOf="@id/display_prefs" />
+            app:layout_constraintTop_toTopOf="parent" />
 
         <TextView
             android:id="@+id/background_color_text"
@@ -146,13 +131,6 @@
             android:textColor="?android:attr/textColorSecondary"
             app:layout_constraintTop_toBottomOf="@id/always_show_chapter_transition" />
 
-        <android.widget.Space
-            android:id="@+id/end_general_preferences"
-            android:layout_width="0dp"
-            android:layout_height="0dp"
-            app:layout_constraintBottom_toBottomOf="@id/auto_webtoon_mode"
-            tools:ignore="MissingConstraints" />
-
         <android.widget.Space
             android:id="@+id/spinner_end"
             android:layout_width="16dp"
diff --git a/app/src/main/res/layout/reader_reading_mode_settings.xml b/app/src/main/res/layout/reader_reading_mode_settings.xml
index cd6c17991..aee06f38d 100644
--- a/app/src/main/res/layout/reader_reading_mode_settings.xml
+++ b/app/src/main/res/layout/reader_reading_mode_settings.xml
@@ -11,30 +11,13 @@
         android:layout_width="match_parent"
         android:layout_height="match_parent">
 
-        <android.widget.Space
-            android:id="@+id/spinner_end"
-            android:layout_width="16dp"
-            android:layout_height="0dp"
-            app:layout_constraintStart_toEndOf="parent"
-            tools:ignore="MissingConstraints" />
-
         <!-- Series-specific preferences -->
 
-        <TextView
-            android:id="@+id/series_prefs"
-            android:layout_width="0dp"
-            android:layout_height="wrap_content"
-            android:text="@string/pref_category_for_this_series"
-            android:textColor="?attr/colorAccent"
-            android:textStyle="bold"
-            app:layout_constraintStart_toStartOf="parent"
-            app:layout_constraintTop_toTopOf="parent" />
-
         <TextView
             android:id="@+id/viewer_text"
             android:layout_width="0dp"
             android:layout_height="wrap_content"
-            android:text="@string/viewer"
+            android:text="@string/pref_category_for_this_series"
             app:layout_constraintBaseline_toBaselineOf="@id/viewer"
             app:layout_constraintEnd_toStartOf="@id/verticalcenter"
             app:layout_constraintStart_toStartOf="parent" />
@@ -43,11 +26,10 @@
             android:id="@+id/viewer"
             android:layout_width="0dp"
             android:layout_height="24dp"
-            android:layout_marginTop="20dp"
             android:entries="@array/viewers_selector"
             app:layout_constraintEnd_toEndOf="@id/spinner_end"
             app:layout_constraintStart_toEndOf="@id/verticalcenter"
-            app:layout_constraintTop_toBottomOf="@id/series_prefs" />
+            app:layout_constraintTop_toTopOf="parent" />
 
         <!-- Pager preferences -->
 
@@ -81,6 +63,13 @@
             android:orientation="vertical"
             app:layout_constraintGuide_percent="0.5" />
 
+        <android.widget.Space
+            android:id="@+id/spinner_end"
+            android:layout_width="16dp"
+            android:layout_height="0dp"
+            app:layout_constraintStart_toEndOf="parent"
+            tools:ignore="MissingConstraints" />
+
     </androidx.constraintlayout.widget.ConstraintLayout>
 
 </androidx.core.widget.NestedScrollView>