Move `WireGuard key` option into `Advanced` screen

2 files changed, 23 insertions, 8 deletions

diff --git a/android/src/main/kotlin/net/mullvad/mullvadvpn/ui/AdvancedFragment.kt b/android/src/main/kotlin/net/mullvad/mullvadvpn/ui/AdvancedFragment.kt
index ec39ee2100..1921546951 100644
--- a/android/src/main/kotlin/net/mullvad/mullvadvpn/ui/AdvancedFragment.kt
+++ b/android/src/main/kotlin/net/mullvad/mullvadvpn/ui/AdvancedFragment.kt
@@ -1,6 +1,7 @@
 package net.mullvad.mullvadvpn.ui
 
 import android.os.Bundle
+import android.support.v4.app.Fragment
 import android.view.LayoutInflater
 import android.view.View
 import android.view.ViewGroup
@@ -17,6 +18,7 @@ private const val MAX_MTU_VALUE = 1420
 
 class AdvancedFragment : ServiceDependentFragment(OnNoService.GoBack) {
     private lateinit var wireguardMtuInput: CellInput
+    private lateinit var wireguardKeysMenu: View
 
     private var subscriptionId: Int? = null
     private var updateUiJob: Job? = null
@@ -45,6 +47,12 @@ class AdvancedFragment : ServiceDependentFragment(OnNoService.GoBack) {
             text = context.getString(R.string.wireguard_mtu_footer, MIN_MTU_VALUE, MAX_MTU_VALUE)
         }
 
+        wireguardKeysMenu = view.findViewById<View>(R.id.wireguard_keys).apply {
+            setOnClickListener {
+                openSubFragment(WireguardKeyFragment())
+            }
+        }
+
         settingsListener.subscribe({ settings -> updateUi(settings) })
 
         return view
@@ -63,4 +71,18 @@ class AdvancedFragment : ServiceDependentFragment(OnNoService.GoBack) {
         subscriptionId?.let { id -> settingsListener.unsubscribe(id) }
         updateUiJob?.cancel()
     }
+
+    private fun openSubFragment(fragment: Fragment) {
+        fragmentManager?.beginTransaction()?.apply {
+            setCustomAnimations(
+                R.anim.fragment_enter_from_right,
+                R.anim.fragment_half_exit_to_left,
+                R.anim.fragment_half_enter_from_left,
+                R.anim.fragment_exit_to_right
+            )
+            replace(R.id.main_fragment, fragment)
+            addToBackStack(null)
+            commit()
+        }
+    }
 }
diff --git a/android/src/main/kotlin/net/mullvad/mullvadvpn/ui/SettingsFragment.kt b/android/src/main/kotlin/net/mullvad/mullvadvpn/ui/SettingsFragment.kt
index 8b9a30bd32..55f8b24c89 100644
--- a/android/src/main/kotlin/net/mullvad/mullvadvpn/ui/SettingsFragment.kt
+++ b/android/src/main/kotlin/net/mullvad/mullvadvpn/ui/SettingsFragment.kt
@@ -27,7 +27,6 @@ class SettingsFragment : ServiceAwareFragment() {
     private lateinit var preferencesMenu: View
     private lateinit var advancedMenu: View
     private lateinit var remainingTimeLabel: RemainingTimeLabel
-    private lateinit var wireguardKeysMenu: View
 
     private var active = false
 
@@ -83,15 +82,10 @@ class SettingsFragment : ServiceAwareFragment() {
             }
         }
 
-        wireguardKeysMenu = view.findViewById<View>(R.id.wireguard_keys).apply {
-            setOnClickListener {
-                openSubFragment(WireguardKeyFragment())
-            }
-        }
-
         view.findViewById<View>(R.id.app_version).setOnClickListener {
             openLink(R.string.download_url)
         }
+
         view.findViewById<View>(R.id.report_a_problem).setOnClickListener {
             openSubFragment(ProblemReportFragment())
         }
@@ -181,7 +175,6 @@ class SettingsFragment : ServiceAwareFragment() {
         accountMenu.visibility = visibility
         preferencesMenu.visibility = visibility
         advancedMenu.visibility = visibility
-        wireguardKeysMenu.visibility = visibility
     }
 
     private fun updateVersionInfo() = GlobalScope.launch(Dispatchers.Main) {