[Side Sheet] Aligned close affordance and title text along the baseli…

…ne rather than the top. The close affordance icon and title text were previously aligned by the top. Since the icon button has extra spacing from insets and padding, the Views appeared to be misaligned. Aligning along the baseline fixes the visual misalignment. This change is in the catalog since the side sheet view is provided by the developer. PiperOrigin-RevId: 538570382
material-components · Jun 9, 2023 · 7a3f30b · 7a3f30b
1 parent 7ee0b72
commit 7a3f30b
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 5 deletions.
diff --git a/catalog/java/io/material/catalog/sidesheet/res/layout/cat_sidesheet_content.xml b/catalog/java/io/material/catalog/sidesheet/res/layout/cat_sidesheet_content.xml
@@ -33,7 +33,7 @@
     app:icon="@drawable/ic_close_vd_theme_24px"
     app:iconTint="?attr/colorOnSurfaceVariant"
     app:layout_constraintEnd_toEndOf="parent"
-    app:layout_constraintTop_toTopOf="@id/side_sheet_title_text"
+    app:layout_constraintBaseline_toBaselineOf="@id/side_sheet_title_text"
     tools:ignore="ContentDescription" />
 
   <TextView

diff --git a/catalog/java/io/material/catalog/sidesheet/res/layout/cat_sidesheet_content_coplanar.xml b/catalog/java/io/material/catalog/sidesheet/res/layout/cat_sidesheet_content_coplanar.xml
@@ -31,7 +31,7 @@
     app:icon="@drawable/ic_close_vd_theme_24px"
     app:iconTint="?attr/colorOnSurfaceVariant"
     app:layout_constraintEnd_toEndOf="parent"
-    app:layout_constraintTop_toTopOf="parent"
+    app:layout_constraintBaseline_toBaselineOf="@id/coplanar_side_sheet_title_text"
     tools:ignore="ContentDescription" />
 
   <TextView

diff --git a/...java/io/material/catalog/sidesheet/res/layout/cat_sidesheet_content_coplanar_detached.xml b/...java/io/material/catalog/sidesheet/res/layout/cat_sidesheet_content_coplanar_detached.xml
@@ -31,7 +31,7 @@
     app:icon="@drawable/ic_close_vd_theme_24px"
     app:iconTint="?attr/colorOnSurfaceVariant"
     app:layout_constraintEnd_toEndOf="parent"
-    app:layout_constraintTop_toTopOf="parent"
+    app:layout_constraintBaseline_toBaselineOf="@id/coplanar_detached_side_sheet_title_text"
     tools:ignore="ContentDescription" />
 
   <TextView

diff --git a/...a/io/material/catalog/sidesheet/res/layout/cat_sidesheet_content_vertically_scrolling.xml b/...a/io/material/catalog/sidesheet/res/layout/cat_sidesheet_content_vertically_scrolling.xml
@@ -31,7 +31,7 @@
     app:icon="@drawable/ic_close_vd_theme_24px"
     app:iconTint="?attr/colorOnSurfaceVariant"
     app:layout_constraintEnd_toEndOf="parent"
-    app:layout_constraintTop_toTopOf="parent"
+    app:layout_constraintBaseline_toBaselineOf="@id/vertically_scrolling_side_sheet_title_text"
     tools:ignore="ContentDescription" />
 
   <TextView

diff --git a/catalog/java/io/material/catalog/sidesheet/res/layout/cat_sidesheet_detached_content.xml b/catalog/java/io/material/catalog/sidesheet/res/layout/cat_sidesheet_detached_content.xml
@@ -33,7 +33,7 @@
     app:icon="@drawable/ic_close_vd_theme_24px"
     app:iconTint="?attr/colorOnSurfaceVariant"
     app:layout_constraintEnd_toEndOf="parent"
-    app:layout_constraintTop_toTopOf="@id/detached_side_sheet_title_text"
+    app:layout_constraintBaseline_toBaselineOf="@id/detached_side_sheet_title_text"
     tools:ignore="ContentDescription" />
 
   <TextView