Change how public key is copied to the clipboard

1 files changed, 18 insertions, 10 deletions

diff --git a/android/src/main/kotlin/net/mullvad/mullvadvpn/ui/WireguardKeyFragment.kt b/android/src/main/kotlin/net/mullvad/mullvadvpn/ui/WireguardKeyFragment.kt
index e07cf0988f..7fa9f2e39d 100644
--- a/android/src/main/kotlin/net/mullvad/mullvadvpn/ui/WireguardKeyFragment.kt
+++ b/android/src/main/kotlin/net/mullvad/mullvadvpn/ui/WireguardKeyFragment.kt
@@ -84,6 +84,12 @@ class WireguardKeyFragment : ServiceDependentFragment(OnNoService.GoToLaunchScre
         publicKey = view.findViewById<TextView>(R.id.wireguard_public_key)
         publicKeyAge = view.findViewById<TextView>(R.id.wireguard_key_age)
 
+        view.findViewById<View>(R.id.public_key_container).apply {
+            setOnClickListener {
+                copyPublicKeyToClipboard()
+            }
+        }
+
         generateKeyButton = view.findViewById<Button>(R.id.generate_key).apply {
             setOnClickListener {
                 onGenerateKeyPress()
@@ -144,16 +150,6 @@ class WireguardKeyFragment : ServiceDependentFragment(OnNoService.GoToLaunchScre
                 publicKey.visibility = View.VISIBLE
                 publicKey.setText(publicKeyString.substring(0, 20) + "...")
 
-                publicKey.setOnClickListener {
-                    val label = parentActivity.getString(R.string.wireguard_key_copied_to_clibpoard)
-                    val clipboard = parentActivity
-                        .getSystemService(Context.CLIPBOARD_SERVICE) as ClipboardManager
-                    clipboard.setPrimaryClip(ClipData.newPlainText(label, publicKeyString))
-
-                    Toast.makeText(parentActivity, label, Toast.LENGTH_SHORT)
-                        .show()
-                }
-
                 publicKeyAge.setText(formatKeyDateCreated(key.dateCreated))
 
                 keyState.verified?.let { verified ->
@@ -229,6 +225,18 @@ class WireguardKeyFragment : ServiceDependentFragment(OnNoService.GoToLaunchScre
         }
     }
 
+    private fun copyPublicKeyToClipboard() {
+        val clipboard =
+            parentActivity.getSystemService(Context.CLIPBOARD_SERVICE) as ClipboardManager
+        val clipLabel = parentActivity.resources.getString(R.string.wireguard_public_key)
+        val clipData = ClipData.newPlainText(clipLabel, publicKey.text)
+
+        clipboard.primaryClip = clipData
+
+        Toast.makeText(parentActivity, R.string.copied_wireguard_public_key, Toast.LENGTH_SHORT)
+            .show()
+    }
+
     private fun onGenerateKeyPress() {
         currentJob?.cancel()