nagai/fune
3
0
Fork 0
forked from mirrors/gecko-dev
Code Pull requests Activity

Bug 1821468 - Improve cookie banner re-engagement dialog when titl is long

Browse source
This commit is contained in:
Arturo Mejia 2023-03-09 17:50:59 -05:00 committed by mergify[bot]
parent 4427f8a404
commit 1802511a9d
1 changed files with 19 additions and 26 deletions
repo.diff.show_diff_stats Download patch file Download diff file Expand all files Collapse all files

45
mobile/android/fenix/app/src/main/java/org/mozilla/fenix/settings/quicksettings/protections/cookiebanners/dialog/CookieBannerReEngagementDialogCompose.kt
View file

@ -42,7 +42,7 @@ import org.mozilla.fenix.theme.defaultTypography
private fun CookieBannerReEngagementDialogComposePreview() {
FirefoxTheme {
CookieBannerReEngagementDialogCompose(
dialogTitle = "Cookie banners begone!",
dialogTitle = "Allow Firefox to reject cookie banners?",
dialogText =
"Automatically reject cookie requests, when possible. Otherwise, " +
"accept all cookies to dismiss cookie banners.",
@ -82,35 +82,28 @@ fun CookieBannerReEngagementDialogCompose(
.background(color = FirefoxTheme.colors.layer1),
) {
Column {
Row(
IconButton(
modifier = Modifier
.fillMaxWidth(),
verticalAlignment = Alignment.CenterVertically,
horizontalArrangement = Arrangement.SpaceBetween,
.align(Alignment.End)
.size(48.dp),
onClick = onCloseButtonClicked,
) {
Text(
modifier = Modifier.padding(
top = 24.dp,
start = 24.dp,
end = 24.dp,
bottom = 8.dp,
),
color = FirefoxTheme.colors.textPrimary,
text = dialogTitle,
style = defaultTypography.headline7,
Icon(
painter = painterResource(R.drawable.mozac_ic_close),
contentDescription = stringResource(R.string.content_description_close_button),
tint = FirefoxTheme.colors.iconPrimary,
)
IconButton(
modifier = Modifier
.size(48.dp),
onClick = onCloseButtonClicked,
) {
Icon(
painter = painterResource(R.drawable.mozac_ic_close),
contentDescription = stringResource(R.string.content_description_close_button),
tint = FirefoxTheme.colors.iconPrimary,
)
}
}
Text(
modifier = Modifier.padding(
start = 24.dp,
end = 24.dp,
bottom = 8.dp,
),
color = FirefoxTheme.colors.textPrimary,
text = dialogTitle,
style = defaultTypography.headline7,
)
Text(
modifier = Modifier.padding(horizontal = 24.dp),
color = FirefoxTheme.colors.textPrimary,