From 2e7a742496c3593586435b7333f55e0af51c47f3 Mon Sep 17 00:00:00 2001 From: Jonatan Rhodin Date: Thu, 24 Oct 2024 16:48:26 +0200 Subject: [PATCH 1/2] Add always on warning to lan info dialog --- .../compose/dialog/info/LocalNetworkSharingInfoDialog.kt | 7 +++++++ .../net/mullvad/mullvadvpn/constant/StringConstant.kt | 1 + android/lib/resource/src/main/res/values/strings.xml | 1 + 3 files changed, 9 insertions(+) diff --git a/android/app/src/main/kotlin/net/mullvad/mullvadvpn/compose/dialog/info/LocalNetworkSharingInfoDialog.kt b/android/app/src/main/kotlin/net/mullvad/mullvadvpn/compose/dialog/info/LocalNetworkSharingInfoDialog.kt index 03713afba702..0cd5c5ff43ec 100644 --- a/android/app/src/main/kotlin/net/mullvad/mullvadvpn/compose/dialog/info/LocalNetworkSharingInfoDialog.kt +++ b/android/app/src/main/kotlin/net/mullvad/mullvadvpn/compose/dialog/info/LocalNetworkSharingInfoDialog.kt @@ -11,6 +11,7 @@ import com.ramcosta.composedestinations.navigation.EmptyDestinationsNavigator import com.ramcosta.composedestinations.spec.DestinationStyle import net.mullvad.mullvadvpn.R import net.mullvad.mullvadvpn.compose.component.textResource +import net.mullvad.mullvadvpn.constant.HTML_NEWLINE_STRING import net.mullvad.mullvadvpn.lib.theme.AppTheme @Preview @@ -28,6 +29,12 @@ fun LocalNetworkSharingInfo(navigator: DestinationsNavigator) { buildString { appendLine(stringResource(id = R.string.local_network_sharing_additional_info)) appendLine(textResource(id = R.string.local_network_sharing_ip_ranges)) + // A html linebreak is specifically added since a normal linebreak is + // removed by the html parser + appendLine(HTML_NEWLINE_STRING) + appendLine( + textResource(id = R.string.local_network_sharing_info_block_connections_warning) + ) }, onDismiss = dropUnlessResumed { navigator.navigateUp() }, ) diff --git a/android/app/src/main/kotlin/net/mullvad/mullvadvpn/constant/StringConstant.kt b/android/app/src/main/kotlin/net/mullvad/mullvadvpn/constant/StringConstant.kt index 472f1477324c..23f4189feb78 100644 --- a/android/app/src/main/kotlin/net/mullvad/mullvadvpn/constant/StringConstant.kt +++ b/android/app/src/main/kotlin/net/mullvad/mullvadvpn/constant/StringConstant.kt @@ -2,3 +2,4 @@ package net.mullvad.mullvadvpn.constant const val EMPTY_STRING = "" const val NEWLINE_STRING = "\n" +const val HTML_NEWLINE_STRING = "
" diff --git a/android/lib/resource/src/main/res/values/strings.xml b/android/lib/resource/src/main/res/values/strings.xml index bac4cc8b17aa..a9ce59fca2e3 100644 --- a/android/lib/resource/src/main/res/values/strings.xml +++ b/android/lib/resource/src/main/res/values/strings.xml @@ -395,4 +395,5 @@ App info Changelog Version + Attention: If \"Block connections without VPN\" is enabled, \"Local network sharing\" will not work. From c8678d1138abb7579e6c172fd3c3c07e2f15cb7a Mon Sep 17 00:00:00 2001 From: Jonatan Rhodin Date: Fri, 25 Oct 2024 11:43:37 +0200 Subject: [PATCH 2/2] Update translations --- gui/locales/messages.pot | 3 +++ 1 file changed, 3 insertions(+) diff --git a/gui/locales/messages.pot b/gui/locales/messages.pot index ec571e5364e7..517423d42078 100644 --- a/gui/locales/messages.pot +++ b/gui/locales/messages.pot @@ -2292,6 +2292,9 @@ msgstr "" msgid "At least one method needs to be enabled" msgstr "" +msgid "Attention: If \"Block connections without VPN\" is enabled, \"Local network sharing\" will not work." +msgstr "" + msgid "Attention: If \"Block connections without VPN\" is enabled, split apps will not have Internet. Always keep in mind that split tunneling is a privacy risk." msgstr ""