Skip to content

Commit 71123a7

Browse files
authored
Merge pull request #528 from Automattic/mlumeau/500-what-is-alt-text-link
QuickEditor: Clickable alt text help link brings to support page
2 parents a807c3c + 39aa55a commit 71123a7

6 files changed

+28
-4
lines changed

gravatar-quickeditor/src/main/java/com/gravatar/quickeditor/ui/alttext/AltTextPage.kt

Lines changed: 17 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ package com.gravatar.quickeditor.ui.alttext
22

33
import androidx.activity.compose.BackHandler
44
import androidx.compose.foundation.border
5+
import androidx.compose.foundation.clickable
56
import androidx.compose.foundation.layout.Arrangement
67
import androidx.compose.foundation.layout.Box
78
import androidx.compose.foundation.layout.Column
@@ -12,11 +13,11 @@ import androidx.compose.foundation.layout.size
1213
import androidx.compose.foundation.layout.wrapContentSize
1314
import androidx.compose.foundation.shape.RoundedCornerShape
1415
import androidx.compose.foundation.text.BasicTextField
16+
import androidx.compose.material3.Icon
1517
import androidx.compose.material3.LocalTextStyle
1618
import androidx.compose.material3.MaterialTheme
1719
import androidx.compose.material3.SnackbarHostState
1820
import androidx.compose.material3.Surface
19-
import androidx.compose.material3.Text
2021
import androidx.compose.runtime.Composable
2122
import androidx.compose.runtime.LaunchedEffect
2223
import androidx.compose.runtime.collectAsState
@@ -28,6 +29,8 @@ import androidx.compose.ui.Modifier
2829
import androidx.compose.ui.draw.clip
2930
import androidx.compose.ui.platform.LocalContext
3031
import androidx.compose.ui.platform.LocalDensity
32+
import androidx.compose.ui.platform.LocalUriHandler
33+
import androidx.compose.ui.res.painterResource
3134
import androidx.compose.ui.res.stringResource
3235
import androidx.compose.ui.tooling.preview.Preview
3336
import androidx.compose.ui.unit.dp
@@ -48,6 +51,10 @@ import kotlinx.coroutines.launch
4851
import kotlinx.coroutines.withContext
4952
import java.net.URL
5053

54+
/** Gravatar Alt Text help URL */
55+
private const val GRAVATAR_ALT_TEXT_HELP_URL: String =
56+
"https://support.gravatar.com/profiles/avatars/#add-alt-text-to-avatars"
57+
5158
@Composable
5259
internal fun AltTextPage(
5360
email: String,
@@ -118,6 +125,8 @@ internal fun AltTextPage(
118125
onEvent: (AltTextEvent) -> Unit,
119126
modifier: Modifier = Modifier,
120127
) {
128+
val uriHandler = LocalUriHandler.current
129+
121130
Surface(modifier = modifier.fillMaxWidth()) {
122131
Box(
123132
modifier = Modifier
@@ -138,9 +147,13 @@ internal fun AltTextPage(
138147
title = stringResource(R.string.gravatar_qe_avatar_alt_text_section_title),
139148
modifier = Modifier,
140149
)
141-
Text(
142-
text = stringResource(id = R.string.gravatar_qe_avatar_alt_text_section_what_is),
143-
color = MaterialTheme.colorScheme.primary,
150+
Icon(
151+
painter = painterResource(id = R.drawable.gravatar_alt_text_help),
152+
tint = MaterialTheme.colorScheme.primary,
153+
contentDescription = stringResource(id = R.string.gravatar_qe_avatar_alt_text_section_what_is),
154+
modifier = Modifier.clickable {
155+
uriHandler.openUri(GRAVATAR_ALT_TEXT_HELP_URL)
156+
},
144157
)
145158
}
146159

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
<vector xmlns:android="http://schemas.android.com/apk/res/android"
2+
android:width="18dp"
3+
android:height="18dp"
4+
android:viewportWidth="18"
5+
android:viewportHeight="18">
6+
<path
7+
android:pathData="M6.75,7.25C6.75,6.007 7.757,5 9,5C10.243,5 11.25,6.007 11.25,7.25C11.25,8.408 10.375,9.362 9.25,9.486C9.112,9.501 9,9.612 9,9.75V11M9,12V13.5M17,9C17,13.418 13.418,17 9,17C4.582,17 1,13.418 1,9C1,4.582 4.582,1 9,1C13.418,1 17,4.582 17,9Z"
8+
android:strokeWidth="1.5"
9+
android:fillColor="#00000000"
10+
android:strokeColor="#101517"/>
11+
</vector>

0 commit comments

Comments
 (0)