Skip to content

Commit a422b18

Browse files
committed
Limit width of landing screen buttons on tablets
1 parent 984ed7f commit a422b18

File tree

6 files changed

+15
-5
lines changed

6 files changed

+15
-5
lines changed

WordPress/src/jetpack/java/org/wordpress/android/ui/accounts/login/components/PrimaryButton.kt

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import androidx.compose.material.Text
88
import androidx.compose.runtime.Composable
99
import androidx.compose.ui.Modifier
1010
import androidx.compose.ui.res.colorResource
11+
import androidx.compose.ui.res.dimensionResource
1112
import androidx.compose.ui.res.stringResource
1213
import androidx.compose.ui.text.TextStyle
1314
import androidx.compose.ui.text.font.FontWeight
@@ -31,7 +32,7 @@ fun PrimaryButton(
3132
contentColor = colorResource(R.color.text_color_jetpack_login_splash_primary_button),
3233
),
3334
modifier = modifier
34-
.padding(horizontal = 20.dp)
35+
.padding(horizontal = dimensionResource(R.dimen.login_prologue_revamped_buttons_padding))
3536
.padding(top = Margin.ExtraLarge.value)
3637
.fillMaxWidth(),
3738
) {

WordPress/src/jetpack/java/org/wordpress/android/ui/accounts/login/components/SecondaryButton.kt

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ import androidx.compose.runtime.Composable
99
import androidx.compose.ui.Modifier
1010
import androidx.compose.ui.graphics.Color
1111
import androidx.compose.ui.res.colorResource
12+
import androidx.compose.ui.res.dimensionResource
1213
import androidx.compose.ui.res.stringResource
1314
import androidx.compose.ui.text.TextStyle
1415
import androidx.compose.ui.text.font.FontWeight
@@ -31,7 +32,7 @@ fun SecondaryButton(
3132
backgroundColor = Color.Transparent,
3233
),
3334
modifier = modifier
34-
.padding(horizontal = 20.dp)
35+
.padding(horizontal = dimensionResource(R.dimen.login_prologue_revamped_buttons_padding))
3536
.padding(bottom = 60.dp)
3637
.fillMaxWidth(),
3738
) {

WordPress/src/main/res/values-w600dp/dimens.xml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,4 +12,6 @@
1212
<dimen name="post_list_content_margin_standard">@dimen/content_margin_tablet</dimen>
1313
<dimen name="interests_content_margin">@dimen/margin_extra_extra_large</dimen>
1414

15+
<!-- Login Prologue -->
16+
<dimen name="login_prologue_revamped_buttons_padding">@dimen/login_prologue_revamped_buttons_padding_tablet</dimen>
1517
</resources>

WordPress/src/main/res/values/dimens.xml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -677,6 +677,8 @@
677677
<dimen name="login_prologue_content_area">320dp</dimen>
678678
<dimen name="login_prologue_background_circle_size">140dp</dimen>
679679
<dimen name="login_prologue_background_circle_size_large">160dp</dimen>
680+
<dimen name="login_prologue_revamped_buttons_padding">32dp</dimen>
681+
<dimen name="login_prologue_revamped_buttons_padding_tablet">250dp</dimen>
680682

681683
<!-- edit text with chips view -->
682684
<dimen name="edit_text_with_chips_margin_top">5dp</dimen>

WordPress/src/wordpress/java/org/wordpress/android/ui/accounts/login/compose/components/PrimaryButton.kt

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,13 +12,14 @@ import androidx.compose.runtime.Composable
1212
import androidx.compose.ui.Modifier
1313
import androidx.compose.ui.graphics.Color
1414
import androidx.compose.ui.res.colorResource
15+
import androidx.compose.ui.res.dimensionResource
1516
import androidx.compose.ui.text.TextStyle
1617
import androidx.compose.ui.text.font.FontWeight
17-
import androidx.compose.ui.text.font.FontWeight.Companion
1818
import androidx.compose.ui.tooling.preview.Preview
1919
import androidx.compose.ui.unit.dp
2020
import androidx.compose.ui.unit.sp
2121
import org.wordpress.android.R.color
22+
import org.wordpress.android.R.dimen
2223
import org.wordpress.android.ui.compose.theme.AppTheme
2324
import org.wordpress.android.ui.compose.unit.Margin
2425

@@ -46,7 +47,7 @@ fun PrimaryButton(
4647
.fillMaxWidth()
4748
.padding(
4849
vertical = Margin.Small.value,
49-
horizontal = Margin.ExtraExtraMediumLarge.value,
50+
horizontal = dimensionResource(dimen.login_prologue_revamped_buttons_padding),
5051
)
5152
) {
5253
Text(
@@ -58,6 +59,7 @@ fun PrimaryButton(
5859
)
5960
}
6061
}
62+
6163
@Preview(showBackground = true)
6264
@Preview(showBackground = true, uiMode = Configuration.UI_MODE_NIGHT_YES)
6365
@Composable

WordPress/src/wordpress/java/org/wordpress/android/ui/accounts/login/compose/components/SecondaryButton.kt

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,10 +10,12 @@ import androidx.compose.material.Text
1010
import androidx.compose.runtime.Composable
1111
import androidx.compose.ui.Modifier
1212
import androidx.compose.ui.graphics.Color
13+
import androidx.compose.ui.res.dimensionResource
1314
import androidx.compose.ui.text.TextStyle
1415
import androidx.compose.ui.tooling.preview.Preview
1516
import androidx.compose.ui.unit.dp
1617
import androidx.compose.ui.unit.sp
18+
import org.wordpress.android.R.dimen
1719
import org.wordpress.android.ui.compose.theme.AppTheme
1820
import org.wordpress.android.ui.compose.unit.Margin
1921

@@ -39,7 +41,7 @@ fun SecondaryButton(
3941
.fillMaxWidth()
4042
.padding(
4143
vertical = Margin.Small.value,
42-
horizontal = Margin.ExtraExtraMediumLarge.value,
44+
horizontal = dimensionResource(dimen.login_prologue_revamped_buttons_padding),
4345
)
4446
) {
4547
Text(

0 commit comments

Comments
 (0)