From 5600283f073a923ea69600cf98d711f8f2180d64 Mon Sep 17 00:00:00 2001
From: manabu-nakamura <ma.nakamura@gmail.com>
Date: Wed, 5 Feb 2025 19:16:35 +0900
Subject: [PATCH] [Catalog] UIs are not broken

---
 .../res/layout/cat_navigationdrawer.xml       | 139 +++++++++---------
 .../layout/cat_navigationdrawer_custom.xml    |  73 ++++-----
 2 files changed, 111 insertions(+), 101 deletions(-)

diff --git a/catalog/java/io/material/catalog/navigationdrawer/res/layout/cat_navigationdrawer.xml b/catalog/java/io/material/catalog/navigationdrawer/res/layout/cat_navigationdrawer.xml
index 728b677ce4d..14d607a75c3 100644
--- a/catalog/java/io/material/catalog/navigationdrawer/res/layout/cat_navigationdrawer.xml
+++ b/catalog/java/io/material/catalog/navigationdrawer/res/layout/cat_navigationdrawer.xml
@@ -16,103 +16,108 @@
   -->
 
 <androidx.drawerlayout.widget.DrawerLayout
-    xmlns:android="http://schemas.android.com/apk/res/android"
-    xmlns:app="http://schemas.android.com/apk/res-auto"
-    android:id="@+id/drawer"
+  xmlns:android="http://schemas.android.com/apk/res/android"
+  xmlns:app="http://schemas.android.com/apk/res-auto"
+  android:id="@+id/drawer"
+  android:layout_width="match_parent"
+  android:layout_height="match_parent"
+  android:fitsSystemWindows="true">
+
+  <androidx.coordinatorlayout.widget.CoordinatorLayout
+    android:id="@+id/linear_layout"
     android:layout_width="match_parent"
     android:layout_height="match_parent"
+    android:orientation="vertical"
     android:fitsSystemWindows="true">
 
-  <androidx.coordinatorlayout.widget.CoordinatorLayout
-      android:id="@+id/linear_layout"
+    <com.google.android.material.appbar.AppBarLayout
       android:layout_width="match_parent"
-      android:layout_height="match_parent"
-      android:orientation="vertical"
+      android:layout_height="wrap_content"
       android:fitsSystemWindows="true">
 
-    <com.google.android.material.appbar.AppBarLayout
+      <androidx.appcompat.widget.Toolbar
+        android:id="@+id/toolbar"
+        style="?attr/catalogToolbarStyle"
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
-        android:fitsSystemWindows="true">
-
-      <androidx.appcompat.widget.Toolbar
-          android:id="@+id/toolbar"
-          style="?attr/catalogToolbarStyle"
-          android:layout_width="match_parent"
-          android:layout_height="wrap_content"
-          app:navigationIcon="@drawable/ic_drawer_menu_24px"
-          app:title="@string/cat_navigationdrawer_title"/>
+        app:navigationIcon="@drawable/ic_drawer_menu_24px"
+        app:title="@string/cat_navigationdrawer_title"/>
     </com.google.android.material.appbar.AppBarLayout>
 
-    <LinearLayout
-      android:layout_width="wrap_content"
-      android:layout_height="wrap_content"
-      android:layout_gravity="center"
-      android:gravity="center"
-      android:padding="64dp"
-      android:orientation="vertical">
+    <androidx.core.widget.NestedScrollView
+      android:layout_width="match_parent"
+      android:layout_height="match_parent"
+      app:layout_behavior="@string/appbar_scrolling_view_behavior">
 
-      <TextView
+      <LinearLayout
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
         android:gravity="center"
-        android:text="@string/cat_navigationdrawer_instructions"
-        android:textAppearance="?attr/textAppearanceBodyLarge" />
-
-      <Button
-        android:id="@+id/show_end_drawer_gravity"
-        android:layout_width="wrap_content"
-        android:layout_height="wrap_content"
-        android:layout_marginTop="32dp"
-        android:text="@string/cat_navigationdrawer_show_end_drawer" />
+        android:padding="64dp"
+        android:orientation="vertical">
 
-      <LinearLayout
-        android:layout_width="wrap_content"
-        android:layout_height="wrap_content"
-        android:orientation="horizontal">
         <TextView
           android:layout_width="wrap_content"
           android:layout_height="wrap_content"
-          android:layout_gravity="end|center_vertical"
-          android:labelFor="@+id/bold_text_switch"
-          android:text="@string/cat_navigationdrawer_label_active_bold" />
-        <com.google.android.material.materialswitch.MaterialSwitch
-          android:id="@+id/bold_text_switch"
+          android:gravity="center"
+          android:text="@string/cat_navigationdrawer_instructions"
+          android:textAppearance="?attr/textAppearanceBodyLarge" />
+
+        <Button
+          android:id="@+id/show_end_drawer_gravity"
           android:layout_width="wrap_content"
           android:layout_height="wrap_content"
-          android:checked="true"
-          android:layout_marginStart="12dp" />
-      </LinearLayout>
+          android:layout_marginTop="32dp"
+          android:text="@string/cat_navigationdrawer_show_end_drawer" />
 
-      <LinearLayout
-        android:layout_width="wrap_content"
-        android:layout_height="wrap_content"
-        android:orientation="horizontal">
-        <TextView
+        <LinearLayout
           android:layout_width="wrap_content"
           android:layout_height="wrap_content"
-          android:layout_gravity="end|center_vertical"
-          android:labelFor="@+id/auto_close_switch"
-          android:text="@string/cat_navigationdrawer_label_auto_close" />
-        <com.google.android.material.materialswitch.MaterialSwitch
-          android:id="@+id/auto_close_switch"
+          android:orientation="horizontal">
+          <TextView
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:layout_gravity="end|center_vertical"
+            android:labelFor="@+id/bold_text_switch"
+            android:text="@string/cat_navigationdrawer_label_active_bold" />
+          <com.google.android.material.materialswitch.MaterialSwitch
+            android:id="@+id/bold_text_switch"
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:checked="true"
+            android:layout_marginStart="12dp" />
+        </LinearLayout>
+
+        <LinearLayout
           android:layout_width="wrap_content"
           android:layout_height="wrap_content"
-          android:checked="true"
-          android:layout_marginStart="12dp" />
-      </LinearLayout>
+          android:orientation="horizontal">
+          <TextView
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:layout_gravity="end|center_vertical"
+            android:labelFor="@+id/auto_close_switch"
+            android:text="@string/cat_navigationdrawer_label_auto_close" />
+          <com.google.android.material.materialswitch.MaterialSwitch
+            android:id="@+id/auto_close_switch"
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:checked="true"
+            android:layout_marginStart="12dp" />
+        </LinearLayout>
 
-    </LinearLayout>
+      </LinearLayout>
+    </androidx.core.widget.NestedScrollView>
   </androidx.coordinatorlayout.widget.CoordinatorLayout>
 
   <com.google.android.material.navigation.NavigationView
-      android:id="@+id/navigation_view_start"
-      android:layout_width="wrap_content"
-      android:layout_height="match_parent"
-      android:layout_gravity="start"
-      app:headerLayout="@layout/cat_navigationdrawer_header"
-      app:itemTextAppearanceActiveBoldEnabled="true"
-      app:menu="@menu/cat_navigationdrawer_menu"/>
+    android:id="@+id/navigation_view_start"
+    android:layout_width="wrap_content"
+    android:layout_height="match_parent"
+    android:layout_gravity="start"
+    app:headerLayout="@layout/cat_navigationdrawer_header"
+    app:itemTextAppearanceActiveBoldEnabled="true"
+    app:menu="@menu/cat_navigationdrawer_menu"/>
 
   <com.google.android.material.navigation.NavigationView
     android:id="@+id/navigation_view_end"
diff --git a/catalog/java/io/material/catalog/navigationdrawer/res/layout/cat_navigationdrawer_custom.xml b/catalog/java/io/material/catalog/navigationdrawer/res/layout/cat_navigationdrawer_custom.xml
index becac3b10a1..a13aa2c75b0 100644
--- a/catalog/java/io/material/catalog/navigationdrawer/res/layout/cat_navigationdrawer_custom.xml
+++ b/catalog/java/io/material/catalog/navigationdrawer/res/layout/cat_navigationdrawer_custom.xml
@@ -16,57 +16,62 @@
   -->
 
 <androidx.drawerlayout.widget.DrawerLayout
-    xmlns:android="http://schemas.android.com/apk/res/android"
-    xmlns:app="http://schemas.android.com/apk/res-auto"
-    android:id="@+id/drawer"
+  xmlns:android="http://schemas.android.com/apk/res/android"
+  xmlns:app="http://schemas.android.com/apk/res-auto"
+  android:id="@+id/drawer"
+  android:layout_width="match_parent"
+  android:layout_height="match_parent"
+  android:fitsSystemWindows="true">
+
+  <androidx.coordinatorlayout.widget.CoordinatorLayout
+    android:id="@+id/linear_layout"
     android:layout_width="match_parent"
     android:layout_height="match_parent"
+    android:orientation="vertical"
     android:fitsSystemWindows="true">
 
-  <androidx.coordinatorlayout.widget.CoordinatorLayout
-      android:id="@+id/linear_layout"
+    <com.google.android.material.appbar.AppBarLayout
       android:layout_width="match_parent"
-      android:layout_height="match_parent"
-      android:orientation="vertical"
+      android:layout_height="wrap_content"
       android:fitsSystemWindows="true">
 
-    <com.google.android.material.appbar.AppBarLayout
+      <androidx.appcompat.widget.Toolbar
+        android:id="@+id/toolbar"
+        style="?attr/catalogToolbarStyle"
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
-        android:fitsSystemWindows="true">
-
-      <androidx.appcompat.widget.Toolbar
-          android:id="@+id/toolbar"
-          style="?attr/catalogToolbarStyle"
-          android:layout_width="match_parent"
-          android:layout_height="wrap_content"
-          app:navigationIcon="@drawable/ic_drawer_menu_24px"
-          app:title="@string/cat_navigationdrawer_title"/>
+        app:navigationIcon="@drawable/ic_drawer_menu_24px"
+        app:title="@string/cat_navigationdrawer_title"/>
     </com.google.android.material.appbar.AppBarLayout>
 
-    <LinearLayout
-      android:layout_width="wrap_content"
-      android:layout_height="wrap_content"
-      android:layout_gravity="center"
-      android:gravity="center"
-      android:padding="64dp"
-      android:orientation="vertical">
+    <androidx.core.widget.NestedScrollView
+      android:layout_width="match_parent"
+      android:layout_height="match_parent"
+      app:layout_behavior="@string/appbar_scrolling_view_behavior">
 
-      <TextView
+      <LinearLayout
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
         android:gravity="center"
-        android:text="@string/cat_navigationdrawer_instructions"
-        android:textAppearance="?attr/textAppearanceBodyLarge" />
+        android:padding="64dp"
+        android:orientation="vertical">
 
-      <Button
-        android:id="@+id/show_end_drawer_gravity"
-        android:layout_width="wrap_content"
-        android:layout_height="wrap_content"
-        android:layout_marginTop="32dp"
-        android:text="@string/cat_navigationdrawer_show_end_drawer" />
+        <TextView
+          android:layout_width="wrap_content"
+          android:layout_height="wrap_content"
+          android:gravity="center"
+          android:text="@string/cat_navigationdrawer_instructions"
+          android:textAppearance="?attr/textAppearanceBodyLarge" />
+
+        <Button
+          android:id="@+id/show_end_drawer_gravity"
+          android:layout_width="wrap_content"
+          android:layout_height="wrap_content"
+          android:layout_marginTop="32dp"
+          android:text="@string/cat_navigationdrawer_show_end_drawer" />
 
-    </LinearLayout>
+      </LinearLayout>
+    </androidx.core.widget.NestedScrollView>
   </androidx.coordinatorlayout.widget.CoordinatorLayout>
 
   <RelativeLayout