From c599c20696725d5ce39a2e668f043e6df52fa940 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Sylwester=20Zieli=C5=84ski?=
 <sylwester.zielinski@nordicsemi.no>
Date: Thu, 31 Mar 2022 14:44:55 +0200
Subject: [PATCH 1/3] Change DFU label depends on DFU app presence.

---
 app/src/main/AndroidManifest.xml                | 10 ++++++++++
 .../android/nrftoolbox/MainActivity.kt          | 12 ++++++++++++
 .../nrftoolbox/repository/ActivitySignals.kt    | 17 +++++++++++++++++
 .../android/nrftoolbox/view/HomeView.kt         |  8 +++++++-
 .../android/nrftoolbox/view/HomeViewState.kt    | 10 ++++++++--
 .../nrftoolbox/viewmodel/HomeViewModel.kt       |  7 +++++++
 app/src/main/res/values/strings.xml             |  1 +
 7 files changed, 62 insertions(+), 3 deletions(-)
 create mode 100644 app/src/main/java/no/nordicsemi/android/nrftoolbox/repository/ActivitySignals.kt

diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
index 23422dff..d022eac1 100644
--- a/app/src/main/AndroidManifest.xml
+++ b/app/src/main/AndroidManifest.xml
@@ -2,6 +2,16 @@
 <manifest xmlns:android="http://schemas.android.com/apk/res/android"
     package="no.nordicsemi.android.nrftoolbox">
 
+    <queries>
+        <package android:name="no.nordicsemi.android.dfu" />
+
+        <intent>
+            <action android:name="android.intent.action.VIEW" />
+            <category android:name="android.intent.category.BROWSABLE" />
+            <data android:scheme="https" />
+        </intent>
+    </queries>
+
     <application
         android:name=".NrfToolboxApplication"
         android:allowBackup="true"
diff --git a/app/src/main/java/no/nordicsemi/android/nrftoolbox/MainActivity.kt b/app/src/main/java/no/nordicsemi/android/nrftoolbox/MainActivity.kt
index 97b2eb73..cbf1de78 100644
--- a/app/src/main/java/no/nordicsemi/android/nrftoolbox/MainActivity.kt
+++ b/app/src/main/java/no/nordicsemi/android/nrftoolbox/MainActivity.kt
@@ -2,6 +2,7 @@ package no.nordicsemi.android.nrftoolbox
 
 import android.os.Bundle
 import androidx.activity.compose.setContent
+import androidx.activity.viewModels
 import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.material3.MaterialTheme
 import androidx.compose.material3.Surface
@@ -11,11 +12,17 @@ import no.nordicsemi.android.gls.GLSDestinations
 import no.nordicsemi.android.material.you.NordicActivity
 import no.nordicsemi.android.material.you.NordicTheme
 import no.nordicsemi.android.navigation.NavigationView
+import no.nordicsemi.android.nrftoolbox.repository.ActivitySignals
+import no.nordicsemi.android.nrftoolbox.viewmodel.HomeViewModel
 import no.nordicsemi.ui.scanner.ScannerDestinations
+import javax.inject.Inject
 
 @AndroidEntryPoint
 class MainActivity : NordicActivity() {
 
+    @Inject
+    lateinit var activitySignals: ActivitySignals
+
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)
 
@@ -30,4 +37,9 @@ class MainActivity : NordicActivity() {
             }
         }
     }
+
+    override fun onResume() {
+        super.onResume()
+        activitySignals.onResume()
+    }
 }
diff --git a/app/src/main/java/no/nordicsemi/android/nrftoolbox/repository/ActivitySignals.kt b/app/src/main/java/no/nordicsemi/android/nrftoolbox/repository/ActivitySignals.kt
new file mode 100644
index 00000000..fbd41e1c
--- /dev/null
+++ b/app/src/main/java/no/nordicsemi/android/nrftoolbox/repository/ActivitySignals.kt
@@ -0,0 +1,17 @@
+package no.nordicsemi.android.nrftoolbox.repository
+
+import kotlinx.coroutines.flow.MutableStateFlow
+import kotlinx.coroutines.flow.asStateFlow
+import javax.inject.Inject
+import javax.inject.Singleton
+
+@Singleton
+class ActivitySignals @Inject constructor() {
+
+    private val _onResumeTrigger = MutableStateFlow(false)
+    val state = _onResumeTrigger.asStateFlow()
+
+    fun onResume() {
+        _onResumeTrigger.value = !_onResumeTrigger.value
+    }
+}
diff --git a/app/src/main/java/no/nordicsemi/android/nrftoolbox/view/HomeView.kt b/app/src/main/java/no/nordicsemi/android/nrftoolbox/view/HomeView.kt
index 151800dc..7ba809af 100644
--- a/app/src/main/java/no/nordicsemi/android/nrftoolbox/view/HomeView.kt
+++ b/app/src/main/java/no/nordicsemi/android/nrftoolbox/view/HomeView.kt
@@ -14,6 +14,7 @@ import androidx.compose.ui.res.stringResource
 import androidx.compose.ui.text.style.TextAlign
 import androidx.compose.ui.unit.dp
 import androidx.hilt.navigation.compose.hiltViewModel
+import androidx.lifecycle.OnLifecycleEvent
 import no.nordicsemi.android.nrftoolbox.BuildConfig
 import no.nordicsemi.android.nrftoolbox.ProfileDestination
 import no.nordicsemi.android.nrftoolbox.R
@@ -124,7 +125,12 @@ fun HomeScreen() {
                 val uriHandler = LocalUriHandler.current
                 val context = LocalContext.current
                 val packageManger = context.packageManager
-                FeatureButton(R.drawable.ic_dfu, R.string.dfu_module, R.string.dfu_module_full, null, R.string.dfu_module_info) {
+
+                val description = packageManger.getLaunchIntentForPackage(DFU_PACKAGE_NAME)?.let {
+                    R.string.dfu_module_info
+                } ?: R.string.dfu_module_install
+
+                FeatureButton(R.drawable.ic_dfu, R.string.dfu_module, R.string.dfu_module_full, null, description) {
                     val intent = packageManger.getLaunchIntentForPackage(DFU_PACKAGE_NAME)
                     if (intent != null) {
                         context.startActivity(intent)
diff --git a/app/src/main/java/no/nordicsemi/android/nrftoolbox/view/HomeViewState.kt b/app/src/main/java/no/nordicsemi/android/nrftoolbox/view/HomeViewState.kt
index 77e32767..d319e128 100644
--- a/app/src/main/java/no/nordicsemi/android/nrftoolbox/view/HomeViewState.kt
+++ b/app/src/main/java/no/nordicsemi/android/nrftoolbox/view/HomeViewState.kt
@@ -7,5 +7,11 @@ data class HomeViewState(
     val isRSCSModuleRunning: Boolean = false,
     val isPRXModuleRunning: Boolean = false,
     val isCGMModuleRunning: Boolean = false,
-    val isUARTModuleRunning: Boolean = false
-)
+    val isUARTModuleRunning: Boolean = false,
+    val refreshToggle: Boolean = false
+) {
+
+    fun copyWithRefresh(): HomeViewState {
+        return copy(refreshToggle = !refreshToggle)
+    }
+}
diff --git a/app/src/main/java/no/nordicsemi/android/nrftoolbox/viewmodel/HomeViewModel.kt b/app/src/main/java/no/nordicsemi/android/nrftoolbox/viewmodel/HomeViewModel.kt
index 4b56b554..fd22f98f 100644
--- a/app/src/main/java/no/nordicsemi/android/nrftoolbox/viewmodel/HomeViewModel.kt
+++ b/app/src/main/java/no/nordicsemi/android/nrftoolbox/viewmodel/HomeViewModel.kt
@@ -1,5 +1,6 @@
 package no.nordicsemi.android.nrftoolbox.viewmodel
 
+import android.util.Log
 import androidx.lifecycle.ViewModel
 import androidx.lifecycle.viewModelScope
 import dagger.hilt.android.lifecycle.HiltViewModel
@@ -13,6 +14,7 @@ import no.nordicsemi.android.hrs.service.HRSRepository
 import no.nordicsemi.android.hts.repository.HTSRepository
 import no.nordicsemi.android.navigation.NavigationManager
 import no.nordicsemi.android.nrftoolbox.ProfileDestination
+import no.nordicsemi.android.nrftoolbox.repository.ActivitySignals
 import no.nordicsemi.android.nrftoolbox.view.HomeViewState
 import no.nordicsemi.android.prx.repository.PRXRepository
 import no.nordicsemi.android.rscs.repository.RSCSRepository
@@ -22,6 +24,7 @@ import javax.inject.Inject
 @HiltViewModel
 class HomeViewModel @Inject constructor(
     private val navigationManager: NavigationManager,
+    private val activitySignals: ActivitySignals,
     cgmRepository: CGMRepository,
     cscRepository: CSCRepository,
     hrsRepository: HRSRepository,
@@ -62,6 +65,10 @@ class HomeViewModel @Inject constructor(
         uartRepository.isRunning.onEach {
             _state.value = _state.value.copy(isUARTModuleRunning = it)
         }.launchIn(viewModelScope)
+
+        activitySignals.state.onEach {
+            _state.value = _state.value.copyWithRefresh()
+        }.launchIn(viewModelScope)
     }
 
     fun openProfile(destination: ProfileDestination) {
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index ddfb0591..8517ced2 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -20,6 +20,7 @@
     <string name="dfu_module">DFU</string>
     <string name="dfu_module_full">Device Firmware Update</string>
     <string name="dfu_module_info">Available in separate application.</string>
+    <string name="dfu_module_install">Install from Google Play Store.</string>
 
     <string name="viewmodel_profiles">ViewModel profiles</string>
     <string name="service_profiles">Service profiles</string>

From b37e11d1cd225d289a1dd92a73e7027e420d05da Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Sylwester=20Zieli=C5=84ski?=
 <sylwester.zielinski@nordicsemi.no>
Date: Thu, 31 Mar 2022 14:57:12 +0200
Subject: [PATCH 2/3] Change button colors in dark theme

---
 .../android/nrftoolbox/view/HomeView.kt          | 16 ++++++++--------
 app/src/main/res/values/strings.xml              |  6 +++---
 .../android/uart/view/UARTAddMacroDialog.kt      |  2 ++
 3 files changed, 13 insertions(+), 11 deletions(-)

diff --git a/app/src/main/java/no/nordicsemi/android/nrftoolbox/view/HomeView.kt b/app/src/main/java/no/nordicsemi/android/nrftoolbox/view/HomeView.kt
index 7ba809af..99fc714f 100644
--- a/app/src/main/java/no/nordicsemi/android/nrftoolbox/view/HomeView.kt
+++ b/app/src/main/java/no/nordicsemi/android/nrftoolbox/view/HomeView.kt
@@ -52,14 +52,14 @@ fun HomeScreen() {
 
                 Spacer(modifier = Modifier.height(16.dp))
 
-                FeatureButton(R.drawable.ic_bps, R.string.bps_module, R.string.bps_module_full) {
-                    viewModel.openProfile(ProfileDestination.BPS)
+                FeatureButton(R.drawable.ic_gls, R.string.gls_module, R.string.gls_module_full) {
+                    viewModel.openProfile(ProfileDestination.GLS)
                 }
 
                 Spacer(modifier = Modifier.height(16.dp))
 
-                FeatureButton(R.drawable.ic_gls, R.string.gls_module, R.string.gls_module_full) {
-                    viewModel.openProfile(ProfileDestination.GLS)
+                FeatureButton(R.drawable.ic_bps, R.string.bps_module, R.string.bps_module_full) {
+                    viewModel.openProfile(ProfileDestination.BPS)
                 }
 
                 Spacer(modifier = Modifier.height(16.dp))
@@ -96,14 +96,14 @@ fun HomeScreen() {
 
                 Spacer(modifier = Modifier.height(16.dp))
 
-                FeatureButton(R.drawable.ic_prx, R.string.prx_module, R.string.prx_module_full, state.isPRXModuleRunning) {
-                    viewModel.openProfile(ProfileDestination.PRX)
+                FeatureButton(R.drawable.ic_cgm, R.string.cgm_module, R.string.cgm_module_full, state.isCGMModuleRunning) {
+                    viewModel.openProfile(ProfileDestination.CGMS)
                 }
 
                 Spacer(modifier = Modifier.height(16.dp))
 
-                FeatureButton(R.drawable.ic_cgm, R.string.cgm_module, R.string.cgm_module_full, state.isCGMModuleRunning) {
-                    viewModel.openProfile(ProfileDestination.CGMS)
+                FeatureButton(R.drawable.ic_prx, R.string.prx_module, R.string.prx_module_full, state.isPRXModuleRunning) {
+                    viewModel.openProfile(ProfileDestination.PRX)
                 }
 
                 Spacer(modifier = Modifier.height(16.dp))
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 8517ced2..e982745d 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -16,11 +16,11 @@
     <string name="cgm_module">CGMS</string>
     <string name="cgm_module_full">Continuous Glucose</string>
     <string name="uart_module">UART</string>
-    <string name="uart_module_full">Serial port over BLE</string>
+    <string name="uart_module_full">Universal Asynchronous Receiver/Transmitter (UART)</string>
     <string name="dfu_module">DFU</string>
     <string name="dfu_module_full">Device Firmware Update</string>
-    <string name="dfu_module_info">Available in separate application.</string>
-    <string name="dfu_module_install">Install from Google Play Store.</string>
+    <string name="dfu_module_info">Open DFU application.</string>
+    <string name="dfu_module_install">Download from Google Play.</string>
 
     <string name="viewmodel_profiles">ViewModel profiles</string>
     <string name="service_profiles">Service profiles</string>
diff --git a/profile_uart/src/main/java/no/nordicsemi/android/uart/view/UARTAddMacroDialog.kt b/profile_uart/src/main/java/no/nordicsemi/android/uart/view/UARTAddMacroDialog.kt
index 8a50932d..b2045215 100644
--- a/profile_uart/src/main/java/no/nordicsemi/android/uart/view/UARTAddMacroDialog.kt
+++ b/profile_uart/src/main/java/no/nordicsemi/android/uart/view/UARTAddMacroDialog.kt
@@ -22,6 +22,7 @@ import androidx.compose.ui.Alignment
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.draw.clip
 import androidx.compose.ui.graphics.Color
+import androidx.compose.ui.graphics.ColorFilter
 import androidx.compose.ui.res.painterResource
 import androidx.compose.ui.res.stringResource
 import androidx.compose.ui.unit.dp
@@ -90,6 +91,7 @@ internal fun UARTAddMacroDialog(macro: UARTMacro?, onEvent: (UARTViewEvent) -> U
                         Image(
                             painter = painterResource(id = icon.toResId()),
                             contentDescription = stringResource(id = R.string.uart_macro_icon),
+                            colorFilter = ColorFilter.tint(MaterialTheme.colorScheme.onPrimary),
                             modifier = Modifier
                                 .size(40.dp)
                                 .clip(RoundedCornerShape(10.dp))

From 047e524f6ea3f8c347e99db4845b7d95f87d26d2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Sylwester=20Zieli=C5=84ski?=
 <sylwester.zielinski@nordicsemi.no>
Date: Thu, 31 Mar 2022 14:59:36 +0200
Subject: [PATCH 3/3] Improve add configuration dialog

---
 .../android/uart/view/UARTAddConfigurationDialog.kt       | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/profile_uart/src/main/java/no/nordicsemi/android/uart/view/UARTAddConfigurationDialog.kt b/profile_uart/src/main/java/no/nordicsemi/android/uart/view/UARTAddConfigurationDialog.kt
index 4f13112c..5e922bd1 100644
--- a/profile_uart/src/main/java/no/nordicsemi/android/uart/view/UARTAddConfigurationDialog.kt
+++ b/profile_uart/src/main/java/no/nordicsemi/android/uart/view/UARTAddConfigurationDialog.kt
@@ -39,14 +39,10 @@ internal fun UARTAddConfigurationDialog(onEvent: (UARTViewEvent) -> Unit, onDism
                         .padding(16.dp)
                 )
 
-//                Spacer(modifier = Modifier.height(16.dp))
-
                 NameInput(name, isError)
 
-//                Spacer(modifier = Modifier.height(16.dp))
-
                 Row(
-                    modifier = Modifier.fillMaxWidth(),
+                    modifier = Modifier.fillMaxWidth().padding(horizontal = 16.dp),
                     horizontalArrangement = Arrangement.End
                 ) {
                     TextButton(onClick = { onDismiss() }) {
@@ -96,8 +92,6 @@ private fun NameInput(
             style = MaterialTheme.typography.labelMedium,
             color = MaterialTheme.colorScheme.error
         )
-
-        Spacer(modifier = Modifier.size(16.dp))
     }
 }