From a85387c7c84727ce4d39cba1b8f09e6574f4d895 Mon Sep 17 00:00:00 2001
From: FooIbar <118464521+FooIbar@users.noreply.github.com>
Date: Wed, 28 Dec 2022 20:06:30 +0800
Subject: [PATCH] Add edit button for download label

---
 .../hippo/ehviewer/ui/scene/DownloadsScene.java  |  6 ++++--
 app/src/main/res/layout/item_drawer_list.xml     | 16 +++++++++++++---
 2 files changed, 17 insertions(+), 5 deletions(-)

diff --git a/app/src/main/java/com/hippo/ehviewer/ui/scene/DownloadsScene.java b/app/src/main/java/com/hippo/ehviewer/ui/scene/DownloadsScene.java
index 7368e002b..78eb4cbf3 100644
--- a/app/src/main/java/com/hippo/ehviewer/ui/scene/DownloadsScene.java
+++ b/app/src/main/java/com/hippo/ehviewer/ui/scene/DownloadsScene.java
@@ -875,11 +875,13 @@ public class DownloadsScene extends ToolbarScene
     private static class DownloadLabelHolder extends RecyclerView.ViewHolder {
 
         private final TextView label;
+        private final ImageView edit;
         private final ImageView option;
 
         private DownloadLabelHolder(View itemView) {
             super(itemView);
             label = (TextView) ViewUtils.$$(itemView, R.id.tv_key);
+            edit = (ImageView) ViewUtils.$$(itemView, R.id.iv_edit);
             option = (ImageView) ViewUtils.$$(itemView, R.id.iv_option);
         }
     }
@@ -937,8 +939,8 @@ public class DownloadsScene extends ToolbarScene
                     }
                 });
                 if (position > 0) {
-                    holder.option.setVisibility(View.VISIBLE);
-                    holder.option.setOnClickListener(v -> {
+                    holder.edit.setVisibility(View.VISIBLE);
+                    holder.edit.setOnClickListener(v -> {
                         if (context != null) {
                             EditTextDialogBuilder builder = new EditTextDialogBuilder(
                                     context, label, getString(R.string.download_labels));
diff --git a/app/src/main/res/layout/item_drawer_list.xml b/app/src/main/res/layout/item_drawer_list.xml
index 7543d669e..9ca1cdf20 100644
--- a/app/src/main/res/layout/item_drawer_list.xml
+++ b/app/src/main/res/layout/item_drawer_list.xml
@@ -29,10 +29,20 @@
         android:layout_height="wrap_content"
         android:layout_gravity="center_vertical"
         android:layout_weight="1"
-        android:paddingLeft="?attr/listPreferredItemPaddingLeft"
+        android:paddingStart="?attr/listPreferredItemPaddingLeft"
         android:textAppearance="?attr/textAppearanceListItemSmall"
         android:textColor="?android:attr/textColorPrimary" />
 
+    <ImageView
+        android:id="@+id/iv_edit"
+        android:layout_width="wrap_content"
+        android:layout_height="match_parent"
+        android:layout_gravity="center_vertical"
+        android:paddingStart="?attr/listPreferredItemPaddingLeft"
+        android:paddingEnd="?attr/listPreferredItemPaddingRight"
+        android:visibility="gone"
+        app:srcCompat="@drawable/v_pencil_dark_x24" />
+
     <ImageView
         android:id="@+id/iv_option"
         android:layout_width="wrap_content"
@@ -40,8 +50,8 @@
         android:layout_gravity="center_vertical"
         android:clickable="true"
         android:focusable="true"
-        android:paddingLeft="?attr/listPreferredItemPaddingLeft"
-        android:paddingRight="?attr/listPreferredItemPaddingRight"
+        android:paddingStart="?attr/listPreferredItemPaddingLeft"
+        android:paddingEnd="?attr/listPreferredItemPaddingRight"
         app:srcCompat="@drawable/ic_baseline_reorder_24" />
 
 </LinearLayout>
\ No newline at end of file