From e0011a82355fab08e614463007cd0c93cc52f11b Mon Sep 17 00:00:00 2001
From: Felipe Leme <felipeal@google.com>
Date: Tue, 12 Apr 2016 09:39:45 -0700
Subject: [PATCH] Make "Take bugreport" dialog scrollable.

Without scrolling, such dialog might not fit on small screens, specially
on multi-window environments.

BUG: 28137501
Change-Id: I84ed1ff946d1023712efcf80f15bac4a57966f19
---
 res/layout/bugreport_options_dialog.xml | 118 ++++++++++++------------
 1 file changed, 60 insertions(+), 58 deletions(-)

diff --git a/res/layout/bugreport_options_dialog.xml b/res/layout/bugreport_options_dialog.xml
index d937bc3758d..fd00bc646f2 100644
--- a/res/layout/bugreport_options_dialog.xml
+++ b/res/layout/bugreport_options_dialog.xml
@@ -15,66 +15,68 @@
      limitations under the License.
 -->
 
-<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
-    android:layout_width="match_parent"
-    android:layout_height="match_parent"
-    android:orientation="horizontal"
-    android:paddingStart="20dp"
-    android:paddingEnd="24dp"
-    android:paddingTop="15dp"
-    >
-
-    <CheckedTextView
-        android:id="@+id/bugreport_option_interactive_title"
+<ScrollView xmlns:android="http://schemas.android.com/apk/res/android">
+    <RelativeLayout
         android:layout_width="match_parent"
-        android:layout_height="wrap_content"
-        android:checked="true"
-        android:drawableStart="?android:attr/listChoiceIndicatorSingle"
-        android:ellipsize="marquee"
-        android:gravity="center_vertical"
-        android:paddingEnd="?android:attr/dialogPreferredPadding"
-        android:text="@*android:string/bugreport_option_interactive_title"
-        android:textAppearance="?android:attr/textAppearanceMedium"
-        android:textColor="?android:attr/textColorAlertDialogListItem" />
+        android:layout_height="match_parent"
+        android:orientation="horizontal"
+        android:paddingStart="20dp"
+        android:paddingEnd="24dp"
+        android:paddingTop="15dp"
+        >
 
-    <TextView
-        android:id="@+id/bugreport_option_interactive_summary"
-        android:layout_width="match_parent"
-        android:layout_height="wrap_content"
-        android:layout_below="@+id/bugreport_option_interactive_title"
-        android:maxLines="10"
-        android:paddingBottom="10dp"
-        android:paddingStart="32dp"
-        android:paddingEnd="?android:attr/dialogPreferredPadding"
-        android:text="@*android:string/bugreport_option_interactive_summary"
-        android:textAppearance="?android:attr/textAppearanceListItemSecondary"
-        android:textColor="?android:attr/textColorSecondary" />
+        <CheckedTextView
+            android:id="@+id/bugreport_option_interactive_title"
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            android:checked="true"
+            android:drawableStart="?android:attr/listChoiceIndicatorSingle"
+            android:ellipsize="marquee"
+            android:gravity="center_vertical"
+            android:paddingEnd="?android:attr/dialogPreferredPadding"
+            android:text="@*android:string/bugreport_option_interactive_title"
+            android:textAppearance="?android:attr/textAppearanceMedium"
+            android:textColor="?android:attr/textColorAlertDialogListItem" />
 
-    <CheckedTextView
-        android:id="@+id/bugreport_option_full_title"
-        android:layout_width="match_parent"
-        android:layout_height="wrap_content"
-        android:layout_below="@+id/bugreport_option_interactive_summary"
-        android:checked="false"
-        android:drawableStart="?android:attr/listChoiceIndicatorSingle"
-        android:ellipsize="marquee"
-        android:gravity="center_vertical"
-        android:paddingEnd="?android:attr/dialogPreferredPadding"
-        android:text="@*android:string/bugreport_option_full_title"
-        android:textAppearance="?android:attr/textAppearanceMedium"
-        android:textColor="?android:attr/textColorAlertDialogListItem" />
+        <TextView
+            android:id="@+id/bugreport_option_interactive_summary"
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            android:layout_below="@+id/bugreport_option_interactive_title"
+            android:maxLines="10"
+            android:paddingBottom="10dp"
+            android:paddingStart="32dp"
+            android:paddingEnd="?android:attr/dialogPreferredPadding"
+            android:text="@*android:string/bugreport_option_interactive_summary"
+            android:textAppearance="?android:attr/textAppearanceListItemSecondary"
+            android:textColor="?android:attr/textColorSecondary" />
 
-    <TextView
-        android:id="@+id/bugreport_option_full_summary"
-        android:layout_width="match_parent"
-        android:layout_height="wrap_content"
-        android:layout_below="@+id/bugreport_option_full_title"
-        android:maxLines="10"
-        android:paddingBottom="10dp"
-        android:paddingStart="32dp"
-        android:paddingEnd="?android:attr/dialogPreferredPadding"
-        android:text="@*android:string/bugreport_option_full_summary"
-        android:textAppearance="?android:attr/textAppearanceListItemSecondary"
-        android:textColor="?android:attr/textColorSecondary" />
+        <CheckedTextView
+            android:id="@+id/bugreport_option_full_title"
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            android:layout_below="@+id/bugreport_option_interactive_summary"
+            android:checked="false"
+            android:drawableStart="?android:attr/listChoiceIndicatorSingle"
+            android:ellipsize="marquee"
+            android:gravity="center_vertical"
+            android:paddingEnd="?android:attr/dialogPreferredPadding"
+            android:text="@*android:string/bugreport_option_full_title"
+            android:textAppearance="?android:attr/textAppearanceMedium"
+            android:textColor="?android:attr/textColorAlertDialogListItem" />
 
-</RelativeLayout>
\ No newline at end of file
+        <TextView
+            android:id="@+id/bugreport_option_full_summary"
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            android:layout_below="@+id/bugreport_option_full_title"
+            android:maxLines="10"
+            android:paddingBottom="10dp"
+            android:paddingStart="32dp"
+            android:paddingEnd="?android:attr/dialogPreferredPadding"
+            android:text="@*android:string/bugreport_option_full_summary"
+            android:textAppearance="?android:attr/textAppearanceListItemSecondary"
+            android:textColor="?android:attr/textColorSecondary" />
+
+    </RelativeLayout>
+</ScrollView>