Add feedback form link

fmasa · Sep 19, 2024 · be06659 · be06659
1 parent d65b87e
commit be06659
Show file tree

Hide file tree

Showing 3 changed files with 43 additions and 9 deletions.
diff --git a/...c/commonMain/kotlin/cz/frantisekmasa/wfrp_master/common/core/ui/primitives/UserTipCard.kt b/...c/commonMain/kotlin/cz/frantisekmasa/wfrp_master/common/core/ui/primitives/UserTipCard.kt
@@ -2,14 +2,17 @@ package cz.frantisekmasa.wfrp_master.common.core.ui.primitives
 
 import androidx.compose.foundation.BorderStroke
 import androidx.compose.foundation.clickable
+import androidx.compose.foundation.layout.Arrangement
 import androidx.compose.foundation.layout.Row
 import androidx.compose.foundation.layout.padding
 import androidx.compose.material.Card
 import androidx.compose.material.Icon
+import androidx.compose.material.IconButton
 import androidx.compose.material.MaterialTheme
 import androidx.compose.material.Text
 import androidx.compose.material.TextButton
 import androidx.compose.material.icons.Icons
+import androidx.compose.material.icons.rounded.Close
 import androidx.compose.material.icons.rounded.Highlight
 import androidx.compose.runtime.Composable
 import androidx.compose.runtime.getValue
@@ -24,6 +27,7 @@ import androidx.compose.ui.unit.dp
 import cz.frantisekmasa.wfrp_master.common.Str
 import cz.frantisekmasa.wfrp_master.common.core.domain.NamedEnum
 import cz.frantisekmasa.wfrp_master.common.core.domain.localizedName
+import cz.frantisekmasa.wfrp_master.common.core.shared.rememberUrlOpener
 import cz.frantisekmasa.wfrp_master.common.core.tips.DismissedUserTipsHolder
 import cz.frantisekmasa.wfrp_master.common.core.ui.dialogs.AlertDialog
 import cz.frantisekmasa.wfrp_master.common.core.ui.flow.collectWithLifecycle
@@ -92,23 +96,45 @@ fun UserTipCard(
                 MaterialTheme.colors.surface
             },
         border = if (isLightTheme) null else BorderStroke(1.dp, Theme.fixedColors.warning),
-        modifier = modifier.clickable { dialogVisible = true },
     ) {
+        val urlOpener = rememberUrlOpener()
         Row(
             verticalAlignment = Alignment.CenterVertically,
-            modifier = Modifier.padding(Spacing.small),
+            horizontalArrangement = Arrangement.SpaceBetween,
+            modifier =
+                modifier
+                    .let {
+                        if (tip.link != null) {
+                            it.clickable { urlOpener.open(tip.link, isGooglePlayLink = false) }
+                        } else {
+                            it
+                        }
+                    }
+                    .padding(Spacing.small),
         ) {
-            Icon(Icons.Rounded.Highlight, null, Modifier.padding(end = Spacing.medium))
-            Text(
-                tip.localizedName,
-                style = MaterialTheme.typography.body2,
-                fontStyle = FontStyle.Italic,
-            )
+            Row(
+                verticalAlignment = Alignment.CenterVertically,
+                modifier = Modifier.padding(Spacing.small),
+            ) {
+                Icon(Icons.Rounded.Highlight, null, Modifier.padding(end = Spacing.medium))
+                Text(
+                    tip.localizedName,
+                    style = MaterialTheme.typography.body2,
+                    fontStyle = FontStyle.Italic,
+                )
+            }
+            IconButton(onClick = { dialogVisible = true }) {
+                Icon(Icons.Rounded.Close, stringResource(Str.common_ui_button_dismiss))
+            }
         }
     }
 }
 
-enum class UserTip(override val translatableName: StringResource) : NamedEnum {
+enum class UserTip(
+    override val translatableName: StringResource,
+    val link: String? = null,
+) : NamedEnum {
     ARMOUR_TRAPPINGS(Str.armour_tip_trappings),
     COMPENDIUM_LINK_MOVED(Str.parties_messages_compendium_card_moved),
+    FEEDBACK_FORM(Str.common_ui_tip_feedback_form, "https://forms.gle/YJEECqLbn6sZWYCu9"),
 }
diff --git a/...on/src/commonMain/kotlin/cz/frantisekmasa/wfrp_master/common/partyList/PartyListScreen.kt b/...on/src/commonMain/kotlin/cz/frantisekmasa/wfrp_master/common/partyList/PartyListScreen.kt
@@ -4,6 +4,7 @@ import androidx.compose.foundation.layout.Box
 import androidx.compose.foundation.layout.Column
 import androidx.compose.foundation.layout.PaddingValues
 import androidx.compose.foundation.layout.fillMaxHeight
+import androidx.compose.foundation.layout.fillMaxWidth
 import androidx.compose.foundation.lazy.LazyColumn
 import androidx.compose.foundation.lazy.items
 import androidx.compose.material.Divider
@@ -51,6 +52,8 @@ import cz.frantisekmasa.wfrp_master.common.core.ui.primitives.FloatingActionsMen
 import cz.frantisekmasa.wfrp_master.common.core.ui.primitives.ItemIcon
 import cz.frantisekmasa.wfrp_master.common.core.ui.primitives.MenuState
 import cz.frantisekmasa.wfrp_master.common.core.ui.primitives.Spacing
+import cz.frantisekmasa.wfrp_master.common.core.ui.primitives.UserTip
+import cz.frantisekmasa.wfrp_master.common.core.ui.primitives.UserTipCard
 import cz.frantisekmasa.wfrp_master.common.core.ui.primitives.VISUAL_ONLY_ICON_DESCRIPTION
 import cz.frantisekmasa.wfrp_master.common.core.ui.primitives.rememberScreenModel
 import cz.frantisekmasa.wfrp_master.common.gameMaster.GameMasterScreen
@@ -254,6 +257,10 @@ fun PartyList(
         Modifier.fillMaxHeight(),
         contentPadding = PaddingValues(top = Spacing.medium, bottom = Spacing.bottomPaddingUnderFab),
     ) {
+        item {
+            UserTipCard(UserTip.FEEDBACK_FORM, modifier = Modifier.fillMaxWidth())
+        }
+
         items(parties, key = { it.id }) { item ->
             WithContextMenu(
                 onClick = { onClick(item) },

diff --git a/common/src/commonMain/moko-resources/base/strings.xml b/common/src/commonMain/moko-resources/base/strings.xml
@@ -295,6 +295,7 @@
     <string name="common_ui_markdown_supported_note">Markdown formatting is supported</string>
     <string name="common_ui_messages_no_items">No items</string>
     <string name="common_ui_select_all">Select All</string>
+    <string name="common_ui_tip_feedback_form">Do you have 3 minutes to share feedback?</string>
     <string name="common_ui_units_days">days</string>
     <string name="common_ui_units_hours">hours</string>
     <string name="common_ui_units_minutes">minutes</string>