From 77c0a93ac6dfdfece022928d48464630ee5e46b2 Mon Sep 17 00:00:00 2001
From: arkon <eugcheung94@gmail.com>
Date: Sat, 28 Aug 2021 12:28:15 -0400
Subject: [PATCH] Tweak theme preference item UI

---
 .../preference/ThemesPreferenceAdapter.kt     |  3 ---
 app/src/main/res/layout/pref_theme_item.xml   | 22 +++++++------------
 2 files changed, 8 insertions(+), 17 deletions(-)

diff --git a/app/src/main/java/eu/kanade/tachiyomi/widget/preference/ThemesPreferenceAdapter.kt b/app/src/main/java/eu/kanade/tachiyomi/widget/preference/ThemesPreferenceAdapter.kt
index c8b65edff2..20a4fdc0db 100644
--- a/app/src/main/java/eu/kanade/tachiyomi/widget/preference/ThemesPreferenceAdapter.kt
+++ b/app/src/main/java/eu/kanade/tachiyomi/widget/preference/ThemesPreferenceAdapter.kt
@@ -54,9 +54,6 @@ class ThemesPreferenceAdapter(private val clickListener: OnItemClickListener) :
         fun bind(appTheme: PreferenceValues.AppTheme) {
             binding.name.text = view.context.getString(appTheme.titleResId!!)
 
-            // Rounded corners
-            binding.coverContainer.clipToOutline = true
-
             val isSelected = preferences.appTheme().get() == appTheme
             binding.themeCard.isChecked = isSelected
             binding.themeCard.strokeColor = if (isSelected) selectedColor else unselectedColor
diff --git a/app/src/main/res/layout/pref_theme_item.xml b/app/src/main/res/layout/pref_theme_item.xml
index d8069bf0a5..aa48b76b65 100644
--- a/app/src/main/res/layout/pref_theme_item.xml
+++ b/app/src/main/res/layout/pref_theme_item.xml
@@ -37,7 +37,7 @@
 
             <ImageView
                 android:id="@+id/top_nav_text"
-                android:layout_width="65dp"
+                android:layout_width="64dp"
                 android:layout_height="16dp"
                 android:layout_marginStart="6dp"
                 android:src="@drawable/oval"
@@ -46,33 +46,27 @@
                 app:layout_constraintTop_toTopOf="@+id/top_nav"
                 app:tint="?attr/colorOnToolbar" />
 
-            <FrameLayout
+            <ImageView
                 android:id="@+id/cover_container"
                 android:layout_width="0dp"
                 android:layout_height="0dp"
                 android:layout_marginStart="6dp"
                 android:layout_marginTop="6dp"
-                android:background="@drawable/rounded_rectangle"
+                android:src="@drawable/rounded_rectangle"
+                app:layout_constraintTop_toBottomOf="@+id/top_nav"
                 app:layout_constraintDimensionRatio="2:3"
                 app:layout_constraintEnd_toEndOf="@+id/center_guideline"
                 app:layout_constraintStart_toStartOf="parent"
-                app:layout_constraintTop_toBottomOf="@+id/top_nav">
+                app:tint="?android:attr/divider" />
 
-                <View
-                    android:layout_width="match_parent"
-                    android:layout_height="match_parent"
-                    android:alpha="0.5"
-                    android:background="?attr/colorOnSurface" />
-
-            </FrameLayout>
-
-            <View
+            <ImageView
                 android:id="@+id/cover_badge"
                 android:layout_width="12dp"
                 android:layout_height="16dp"
                 android:layout_marginStart="4dp"
                 android:layout_marginTop="4dp"
-                android:background="?attr/colorAccent"
+                android:src="@drawable/rounded_rectangle"
+                app:tint="?attr/colorAccent"
                 app:layout_constraintStart_toStartOf="@+id/cover_container"
                 app:layout_constraintTop_toTopOf="@+id/cover_container" />