From 2979b9e2d5708b75cf2f37cdfbe177dffd287d7f Mon Sep 17 00:00:00 2001
From: TrianguloY <correo--correo@hotmail.com>
Date: Sun, 21 May 2023 23:35:01 +0200
Subject: [PATCH] improve copy/share buttons

---
 .../urlchecker/modules/list/OpenModule.java   |  2 +-
 app/src/main/res/drawable/copy.xml            | 10 ++++++++++
 app/src/main/res/drawable/share.xml           | 10 ++++++++++
 app/src/main/res/layout/dialog_open.xml       | 20 ++++++++++++-------
 app/src/main/res/values/strings.xml           |  2 +-
 5 files changed, 35 insertions(+), 9 deletions(-)
 create mode 100644 app/src/main/res/drawable/copy.xml
 create mode 100644 app/src/main/res/drawable/share.xml

diff --git a/app/src/main/java/com/trianguloy/urlchecker/modules/list/OpenModule.java b/app/src/main/java/com/trianguloy/urlchecker/modules/list/OpenModule.java
index b8f107f..9b24b38 100644
--- a/app/src/main/java/com/trianguloy/urlchecker/modules/list/OpenModule.java
+++ b/app/src/main/java/com/trianguloy/urlchecker/modules/list/OpenModule.java
@@ -122,7 +122,7 @@ class OpenDialog extends AModuleDialog {
         btn_openWith = views.findViewById(R.id.open_with);
         btn_openWith.setOnClickListener(v -> showList());
 
-        // init copy to URL
+        // init copy
         var btn_copy = views.findViewById(R.id.copyUrl);
         btn_copy.setOnClickListener(v -> AndroidUtils.copyToClipboard(getActivity(), R.string.mOpen_clipboard, getUrl()));
 
diff --git a/app/src/main/res/drawable/copy.xml b/app/src/main/res/drawable/copy.xml
new file mode 100644
index 0000000..6dfd5a3
--- /dev/null
+++ b/app/src/main/res/drawable/copy.xml
@@ -0,0 +1,10 @@
+<vector xmlns:android="http://schemas.android.com/apk/res/android"
+    android:width="24dp"
+    android:height="24dp"
+    android:autoMirrored="true"
+    android:viewportWidth="24"
+    android:viewportHeight="24">
+    <path
+        android:fillColor="@color/app"
+        android:pathData="M16,1L4,1c-1.1,0 -2,0.9 -2,2v14h2L4,3h12L16,1zM19,5L8,5c-1.1,0 -2,0.9 -2,2v14c0,1.1 0.9,2 2,2h11c1.1,0 2,-0.9 2,-2L21,7c0,-1.1 -0.9,-2 -2,-2zM19,21L8,21L8,7h11v14z" />
+</vector>
diff --git a/app/src/main/res/drawable/share.xml b/app/src/main/res/drawable/share.xml
new file mode 100644
index 0000000..d9e26dc
--- /dev/null
+++ b/app/src/main/res/drawable/share.xml
@@ -0,0 +1,10 @@
+<vector xmlns:android="http://schemas.android.com/apk/res/android"
+    android:width="24dp"
+    android:height="24dp"
+    android:autoMirrored="true"
+    android:viewportWidth="24"
+    android:viewportHeight="24">
+    <path
+        android:fillColor="@color/app"
+        android:pathData="M18,16.08c-0.76,0 -1.44,0.3 -1.96,0.77L8.91,12.7c0.05,-0.23 0.09,-0.46 0.09,-0.7s-0.04,-0.47 -0.09,-0.7l7.05,-4.11c0.54,0.5 1.25,0.81 2.04,0.81 1.66,0 3,-1.34 3,-3s-1.34,-3 -3,-3 -3,1.34 -3,3c0,0.24 0.04,0.47 0.09,0.7L8.04,9.81C7.5,9.31 6.79,9 6,9c-1.66,0 -3,1.34 -3,3s1.34,3 3,3c0.79,0 1.5,-0.31 2.04,-0.81l7.12,4.16c-0.05,0.21 -0.08,0.43 -0.08,0.65 0,1.61 1.31,2.92 2.92,2.92 1.61,0 2.92,-1.31 2.92,-2.92s-1.31,-2.92 -2.92,-2.92z" />
+</vector>
diff --git a/app/src/main/res/layout/dialog_open.xml b/app/src/main/res/layout/dialog_open.xml
index 4603d57..8145748 100644
--- a/app/src/main/res/layout/dialog_open.xml
+++ b/app/src/main/res/layout/dialog_open.xml
@@ -11,7 +11,6 @@
         android:layout_height="match_parent"
         android:contentDescription="@string/mOpen_tabsDesc"
         android:minWidth="0dp"
-        android:minHeight="0dp"
         android:src="@drawable/ctabs_off" />
 
     <ImageButton
@@ -21,7 +20,6 @@
         android:layout_height="match_parent"
         android:contentDescription="@string/mOpen_incognitoDesc"
         android:minWidth="0dp"
-        android:minHeight="0dp"
         android:src="@drawable/no_incognito" />
 
     <LinearLayout
@@ -52,22 +50,30 @@
             android:background="@drawable/open_right"
             android:paddingRight="1dp"
             android:src="@drawable/arrow_down"
-            android:tint="?attr/colorAccent" />
+            android:tint="?attr/colorAccent"
+            android:paddingEnd="1dp" />
 
     </LinearLayout>
 
-    <Button
+    <ImageButton
         android:id="@+id/copyUrl"
         style="?android:attr/buttonBarButtonStyle"
         android:layout_width="wrap_content"
         android:layout_height="match_parent"
-        android:text="@string/mOpen_copyUrl" />
+        android:contentDescription="@string/mOpen_copyUrl"
+        android:minWidth="0dp"
+        android:minHeight="0dp"
+        android:scaleType="center"
+        android:src="@drawable/copy" />
 
 
-    <Button
+    <ImageButton
         android:id="@+id/share"
         style="?android:attr/buttonBarButtonStyle"
         android:layout_width="wrap_content"
         android:layout_height="match_parent"
-        android:text="@string/mOpen_share" />
+        android:minWidth="0dp"
+        android:minHeight="0dp"
+        android:contentDescription="@string/mOpen_share"
+        android:src="@drawable/share" />
 </LinearLayout>
\ No newline at end of file
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 394fe61..911fd7e 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -152,7 +152,7 @@ Note: if you edit the patterns, new built-in patterns from app updates will not
     <string name="mOpen_with">Open with %s</string>
     <string name="mOpen_open">Open</string>
     <string name="mOpen_share">Share</string>
-    <string name="mOpen_copyUrl">Copy URL</string>
+    <string name="mOpen_copyUrl">Copy</string>
     <string name="mOpen_clipboard">URL copied to clipboard</string>
     <string name="mOpen_noapps">No apps</string>
     <string name="mOpen_tabsDesc">Toggle Custom Tabs feature</string>