diff --git a/app/src/main/java/no/nordicsemi/android/nrftoolbox/FeatureButton.kt b/app/src/main/java/no/nordicsemi/android/nrftoolbox/FeatureButton.kt index 080fbf0e..28eadbae 100644 --- a/app/src/main/java/no/nordicsemi/android/nrftoolbox/FeatureButton.kt +++ b/app/src/main/java/no/nordicsemi/android/nrftoolbox/FeatureButton.kt @@ -30,23 +30,10 @@ fun FeatureButton( ) { ScreenSection(onClick = onClick) { Column( - modifier = Modifier - .fillMaxWidth() - .height(160.dp), + modifier = Modifier.fillMaxWidth(), horizontalAlignment = Alignment.CenterHorizontally, verticalArrangement = Arrangement.Center ) { - Box( modifier = Modifier.weight(1f)) { - Text( - text = stringResource(id = nameCode), - style = MaterialTheme.typography.labelMedium, - modifier = Modifier.fillMaxWidth().align(Alignment.BottomCenter), - textAlign = TextAlign.Center - ) - } - - Spacer(modifier = Modifier.height(8.dp)) - Image( painter = painterResource(iconId), contentDescription = stringResource(id = name), @@ -61,14 +48,12 @@ fun FeatureButton( Spacer(modifier = Modifier.height(8.dp)) - Box( modifier = Modifier.weight(1f)) { - Text( - text = stringResource(id = name), - style = MaterialTheme.typography.bodyMedium, - modifier = Modifier.fillMaxWidth().align(Alignment.TopCenter), - textAlign = TextAlign.Center - ) - } + Text( + text = stringResource(id = name), + style = MaterialTheme.typography.bodyMedium, + modifier = Modifier.fillMaxWidth().height(42.dp), + textAlign = TextAlign.Center + ) } } } diff --git a/app/src/main/java/no/nordicsemi/android/nrftoolbox/HomeView.kt b/app/src/main/java/no/nordicsemi/android/nrftoolbox/HomeView.kt index 30413644..d2155e55 100644 --- a/app/src/main/java/no/nordicsemi/android/nrftoolbox/HomeView.kt +++ b/app/src/main/java/no/nordicsemi/android/nrftoolbox/HomeView.kt @@ -1,28 +1,23 @@ package no.nordicsemi.android.nrftoolbox -import android.app.Activity import androidx.compose.foundation.layout.* import androidx.compose.foundation.rememberScrollState import androidx.compose.foundation.verticalScroll import androidx.compose.runtime.Composable import androidx.compose.ui.Alignment import androidx.compose.ui.Modifier -import androidx.compose.ui.platform.LocalContext import androidx.compose.ui.platform.LocalUriHandler import androidx.compose.ui.res.stringResource import androidx.compose.ui.unit.dp import androidx.hilt.navigation.compose.hiltViewModel -import no.nordicsemi.android.theme.view.CloseIconAppBar +import no.nordicsemi.android.theme.view.TitleAppBar @Composable fun HomeScreen() { val viewModel: HomeViewModel = hiltViewModel() Column { - val context = LocalContext.current - CloseIconAppBar(stringResource(id = R.string.app_name)) { - (context as? Activity)?.finish() - } + TitleAppBar(stringResource(id = R.string.app_name)) Column( modifier = Modifier @@ -104,7 +99,7 @@ fun HomeScreen() { val uriHandler = LocalUriHandler.current FeatureButton(R.drawable.ic_dfu, R.string.dfu_module, R.string.dfu_module_full) { - uriHandler.openUri("https://github.com/NordicSemiconductor/Android-nRF-Toolbox") + uriHandler.openUri("https://github.com/NordicSemiconductor/Android-DFU-Library") } 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 6a3412ad..09e9d06c 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -8,7 +8,7 @@ HTS Health Thermometer BPS - Blood pressure + Blood Pressure RSCS Running Speed and Cadence PRX diff --git a/lib_theme/src/main/java/no/nordicsemi/android/theme/view/TopAppBar.kt b/lib_theme/src/main/java/no/nordicsemi/android/theme/view/TopAppBar.kt index 2351e0cc..18292eb0 100644 --- a/lib_theme/src/main/java/no/nordicsemi/android/theme/view/TopAppBar.kt +++ b/lib_theme/src/main/java/no/nordicsemi/android/theme/view/TopAppBar.kt @@ -36,6 +36,20 @@ fun CloseIconAppBar(text: String, onClick: () -> Unit) { ) } +@Composable +fun TitleAppBar(text: String) { + SmallTopAppBar( + title = { Text(text) }, + colors = TopAppBarDefaults.smallTopAppBarColors( + scrolledContainerColor = MaterialTheme.colorScheme.primary, + containerColor = colorResource(id = R.color.appBarColor), + titleContentColor = MaterialTheme.colorScheme.onPrimary, + actionIconContentColor = MaterialTheme.colorScheme.onPrimary, + navigationIconContentColor = MaterialTheme.colorScheme.onPrimary, + ) + ) +} + @Composable fun BackIconAppBar(text: String, onClick: () -> Unit) { SmallTopAppBar( diff --git a/profile_dfu/src/main/java/no/nordicsemi/dfu/view/DFUSummaryView.kt b/profile_dfu/src/main/java/no/nordicsemi/dfu/view/DFUSummaryView.kt index 50eb077b..a7d66c3f 100644 --- a/profile_dfu/src/main/java/no/nordicsemi/dfu/view/DFUSummaryView.kt +++ b/profile_dfu/src/main/java/no/nordicsemi/dfu/view/DFUSummaryView.kt @@ -74,7 +74,7 @@ internal fun DeviceDetailsView(device: DiscoveredBluetoothDevice) { .weight(1f) ) { Text( - text = device.displayName(), + text = device.displayName() ?: "No name", style = MaterialTheme.typography.titleMedium ) Text(text = device.displayAddress(), style = MaterialTheme.typography.bodyMedium) diff --git a/settings.gradle b/settings.gradle index 150a45fe..bfe0b342 100644 --- a/settings.gradle +++ b/settings.gradle @@ -18,7 +18,7 @@ dependencyResolutionManagement { alias('nordic-dfu').to('no.nordicsemi.android:dfu:1.12.1-beta01') version('commonlibraries', '1.0.1') - alias('nordic-ui-scanner').to('no.nordicsemi.android.common', 'ui-scanner').versionRef('commonlibraries') + alias('nordic-ui-scanner').to('no.nordicsemi.android.common', 'uiscanner').versionRef('commonlibraries') alias('nordic-navigation').to('no.nordicsemi.android.common', 'navigation').versionRef('commonlibraries') alias('nordic-theme').to('no.nordicsemi.android.common', 'theme').versionRef('commonlibraries') @@ -91,10 +91,10 @@ include ':lib_service' include ':lib_theme' include ':lib_utils' -//if (file('../Android-Common-Libraries').exists()) { -// includeBuild('../Android-Common-Libraries') -//} -// -//if (file('../Android-BLE-Library').exists()) { -// includeBuild('../Android-BLE-Library') -//} +if (file('../Android-Common-Libraries').exists()) { + includeBuild('../Android-Common-Libraries') +} + +if (file('../Android-BLE-Library').exists()) { + includeBuild('../Android-BLE-Library') +}