From 690cd5d695d57a540967c09ad822e9f360a472d6 Mon Sep 17 00:00:00 2001
From: afohrman <afohrman@google.com>
Date: Wed, 17 May 2023 17:04:16 +0000
Subject: [PATCH] [Side Sheet] Changed side sheet child containers in Catalog
 from LinearLayouts to FrameLayouts.

The LinearLayouts each contained only one child, so a FrameLayout is a better choice; more lightweight and slightly more performant. This is a visual no-op.

PiperOrigin-RevId: 532823326
---
 .../sidesheet/res/layout/cat_sidesheets.xml   | 20 +++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/catalog/java/io/material/catalog/sidesheet/res/layout/cat_sidesheets.xml b/catalog/java/io/material/catalog/sidesheet/res/layout/cat_sidesheets.xml
index afbc7e3c5db..22b69f3cc35 100644
--- a/catalog/java/io/material/catalog/sidesheet/res/layout/cat_sidesheets.xml
+++ b/catalog/java/io/material/catalog/sidesheet/res/layout/cat_sidesheets.xml
@@ -16,7 +16,7 @@
 <merge xmlns:android="http://schemas.android.com/apk/res/android"
   xmlns:app="http://schemas.android.com/apk/res-auto"
   xmlns:tools="http://schemas.android.com/tools">
-  <LinearLayout
+  <FrameLayout
     android:id="@+id/standard_side_sheet_container"
     style="@style/Widget.Material3.SideSheet"
     android:layout_width="256dp"
@@ -25,9 +25,9 @@
     app:layout_behavior="@string/side_sheet_behavior"
     tools:targetApi="lollipop">
     <include layout="@layout/cat_sidesheet_content" />
-  </LinearLayout>
+  </FrameLayout>
 
-  <LinearLayout
+  <FrameLayout
     android:id="@+id/standard_detached_side_sheet_container"
     style="@style/Widget.Material3.SideSheet.Detached"
     android:layout_width="256dp"
@@ -36,9 +36,9 @@
     app:layout_behavior="@string/side_sheet_behavior"
     tools:targetApi="lollipop">
     <include layout="@layout/cat_sidesheet_detached_content" />
-  </LinearLayout>
+  </FrameLayout>
 
-  <LinearLayout
+  <FrameLayout
     android:id="@+id/vertically_scrolling_side_sheet_container"
     style="@style/Widget.Material3.SideSheet"
     android:layout_width="256dp"
@@ -47,9 +47,9 @@
     app:layout_behavior="@string/side_sheet_behavior"
     tools:targetApi="lollipop">
     <include layout="@layout/cat_sidesheet_content_vertically_scrolling" />
-  </LinearLayout>
+  </FrameLayout>
 
-  <LinearLayout
+  <FrameLayout
     android:id="@+id/coplanar_side_sheet_container"
     style="@style/Widget.Material3.SideSheet"
     android:layout_width="256dp"
@@ -59,9 +59,9 @@
     app:layout_behavior="@string/side_sheet_behavior"
     tools:targetApi="lollipop">
     <include layout="@layout/cat_sidesheet_content_coplanar" />
-  </LinearLayout>
+  </FrameLayout>
 
-  <LinearLayout
+  <FrameLayout
     android:id="@+id/coplanar_detached_side_sheet_container"
     style="@style/Widget.Material3.SideSheet.Detached"
     android:layout_width="256dp"
@@ -71,5 +71,5 @@
     app:layout_behavior="@string/side_sheet_behavior"
     tools:targetApi="lollipop">
     <include layout="@layout/cat_sidesheet_content_coplanar_detached" />
-  </LinearLayout>
+  </FrameLayout>
 </merge>