From 5ffb37961878cd726d8c1da48309878da79028c8 Mon Sep 17 00:00:00 2001
From: 1 <mrcasey@google.com>
Date: Thu, 11 May 2023 20:15:55 +0000
Subject: 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
---
 java/res/layout/chooser_headline_row.xml | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

(limited to 'java')

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"
-- 
cgit v1.2.3-59-g8ed1b