diff options
| author | 2023-05-11 20:15:55 +0000 | |
|---|---|---|
| committer | 2023-05-11 20:15:55 +0000 | |
| commit | 5ffb37961878cd726d8c1da48309878da79028c8 (patch) | |
| tree | 09e52bb11116c0efe7222bc39281c2d1651dc33a /java | |
| parent | 7d2de04363c0e93bf68f71414e3dc226e7c2c7f9 (diff) | |
Modify headline margins to UX spec
If both modify share and include text are present, ensure 16dp of space
between them. Ensure 16dp of space beneath the headline UI in general.
Bug: 281541996
Test: View UI with modify share or include text or both and compare
margins to mocks.
Change-Id: Id2c1521d3a039806080c32f7e773ebc03bda7b3d
Diffstat (limited to 'java')
| -rw-r--r-- | java/res/layout/chooser_headline_row.xml | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/java/res/layout/chooser_headline_row.xml b/java/res/layout/chooser_headline_row.xml index 8bee807a..e1668d25 100644 --- a/java/res/layout/chooser_headline_row.xml +++ b/java/res/layout/chooser_headline_row.xml @@ -22,6 +22,7 @@ android:layout_width="match_parent" android:layout_height="wrap_content" android:paddingHorizontal="@dimen/chooser_edge_margin_normal" + android:layout_marginBottom="16dp" > <TextView android:id="@+id/headline" @@ -35,7 +36,6 @@ style="@style/TextAppearance.ChooserDefault" android:fontFamily="@androidprv:string/config_headlineFontFamily" android:textSize="18sp" - android:paddingBottom="16dp" /> <androidx.constraintlayout.widget.Barrier @@ -55,7 +55,6 @@ android:ellipsize="end" android:visibility="gone" android:paddingTop="3dp" - android:paddingBottom="16dp" style="@style/TextAppearance.ChooserDefault" android:drawableEnd="@drawable/chevron_right" /> @@ -67,6 +66,8 @@ android:layout_width="wrap_content" android:layout_height="wrap_content" android:maxWidth="@dimen/modify_share_text_toggle_max_width" + android:layout_marginTop="16dp" + app:layout_goneMarginTop="0dp" app:layout_constraintEnd_toEndOf="parent" app:layout_constraintTop_toBottomOf="@id/reselection_action" android:layout_alignWithParentIfMissing="true" |