From 7cd944ffe7cbccd3835a086f61a3b94bddc8d3ac Mon Sep 17 00:00:00 2001
From: IndusAryan <125901294+IndusAryan@users.noreply.github.com>
Date: Tue, 15 Aug 2023 17:28:14 +0530
Subject: [PATCH] feat: filled icon for home and new drag handle in navbar
 options dialog

---
 .../libretube/ui/adapters/NavBarOptionsAdapter.kt      |  2 +-
 app/src/main/res/drawable/ic_drag_handle.xml           |  8 ++++++++
 app/src/main/res/drawable/ic_home_dark.xml             | 10 ++++++++++
 app/src/main/res/layout/nav_options_item.xml           |  7 ++++---
 4 files changed, 23 insertions(+), 4 deletions(-)
 create mode 100644 app/src/main/res/drawable/ic_drag_handle.xml
 create mode 100644 app/src/main/res/drawable/ic_home_dark.xml

diff --git a/app/src/main/java/com/github/libretube/ui/adapters/NavBarOptionsAdapter.kt b/app/src/main/java/com/github/libretube/ui/adapters/NavBarOptionsAdapter.kt
index f8ea9fe70..861b9ce16 100644
--- a/app/src/main/java/com/github/libretube/ui/adapters/NavBarOptionsAdapter.kt
+++ b/app/src/main/java/com/github/libretube/ui/adapters/NavBarOptionsAdapter.kt
@@ -30,7 +30,7 @@ class NavBarOptionsAdapter(
             title.text = item.title
             checkbox.isChecked = item.isVisible
             home.setImageResource(
-                if (item.itemId == selectedHomeTabId) R.drawable.ic_home else R.drawable.ic_home_outlined
+                if (item.itemId == selectedHomeTabId) R.drawable.ic_home_dark else R.drawable.ic_home_outlined
             )
             home.setOnClickListener {
                 if (selectedHomeTabId == item.itemId) {
diff --git a/app/src/main/res/drawable/ic_drag_handle.xml b/app/src/main/res/drawable/ic_drag_handle.xml
new file mode 100644
index 000000000..5caf6dfea
--- /dev/null
+++ b/app/src/main/res/drawable/ic_drag_handle.xml
@@ -0,0 +1,8 @@
+<vector android:height="24dp"
+    android:tint="?attr/colorControlNormal"
+    android:viewportHeight="24"
+    android:viewportWidth="24"
+    android:width="24dp"
+    xmlns:android="http://schemas.android.com/apk/res/android">
+    <path android:fillColor="@android:color/white" android:pathData="M19,9H5c-0.55,0 -1,0.45 -1,1s0.45,1 1,1h14c0.55,0 1,-0.45 1,-1s-0.45,-1 -1,-1zM5,15h14c0.55,0 1,-0.45 1,-1s-0.45,-1 -1,-1H5c-0.55,0 -1,0.45 -1,1s0.45,1 1,1z"/>
+</vector>
diff --git a/app/src/main/res/drawable/ic_home_dark.xml b/app/src/main/res/drawable/ic_home_dark.xml
new file mode 100644
index 000000000..7e3ff499f
--- /dev/null
+++ b/app/src/main/res/drawable/ic_home_dark.xml
@@ -0,0 +1,10 @@
+<vector xmlns:android="http://schemas.android.com/apk/res/android"
+    android:width="24dp"
+    android:height="24dp"
+    android:viewportWidth="48"
+    android:viewportHeight="48">
+
+    <path
+        android:fillColor="?attr/colorPrimaryDark"
+        android:pathData="M11,42Q9.75,42 8.875,41.125Q8,40.25 8,39V19.5Q8,18.8 8.325,18.15Q8.65,17.5 9.2,17.1L22.2,7.35Q22.6,7.05 23.05,6.9Q23.5,6.75 24,6.75Q24.5,6.75 24.95,6.9Q25.4,7.05 25.8,7.35L38.8,17.1Q39.35,17.5 39.675,18.15Q40,18.8 40,19.5V39Q40,40.25 39.125,41.125Q38.25,42 37,42H28V28H20V42Z" />
+</vector>
diff --git a/app/src/main/res/layout/nav_options_item.xml b/app/src/main/res/layout/nav_options_item.xml
index 5f191adae..ac4710ced 100644
--- a/app/src/main/res/layout/nav_options_item.xml
+++ b/app/src/main/res/layout/nav_options_item.xml
@@ -17,7 +17,8 @@
         android:layout_height="match_parent"
         android:background="?attr/selectableItemBackgroundBorderless"
         android:paddingHorizontal="10dp"
-        android:src="@drawable/ic_home_outlined" />
+        android:src="@drawable/ic_home_outlined"
+        tools:ignore="ContentDescription" />
 
     <com.google.android.material.checkbox.MaterialCheckBox
         android:id="@+id/checkbox"
@@ -51,8 +52,8 @@
         android:focusable="false"
         android:padding="16dp"
         android:tintMode="src_in"
-        app:srcCompat="@drawable/ic_drag"
+        app:srcCompat="@drawable/ic_drag_handle"
         app:tint="?attr/colorControlNormal"
         tools:ignore="ContentDescription" />
 
-</LinearLayout>
\ No newline at end of file
+</LinearLayout>