From 602013be6b087910f8784ae820f45397057e0d89 Mon Sep 17 00:00:00 2001
From: Yukako Murakami <Yukako.1.Murakami@sony.com>
Date: Mon, 12 Oct 2020 16:25:13 +0900
Subject: [PATCH] Apply scrollable to app pinning text area

Symptom:
When pinning an application, the instruction screen is not completely
displayed in landscape mode when display settings are largest.

Solution:
by applying scrollable to the layout of app pinning text area

Bug: 176455738
Change-Id: Idbd91f3103bf259168c35179f841b5490a5910a3
---
 .../screen_pinning_request_text_area.xml      | 119 +++++++++---------
 1 file changed, 63 insertions(+), 56 deletions(-)

diff --git a/packages/SystemUI/res/layout/screen_pinning_request_text_area.xml b/packages/SystemUI/res/layout/screen_pinning_request_text_area.xml
index 8dcddc28c3b7a..7880cbdd112c6 100644
--- a/packages/SystemUI/res/layout/screen_pinning_request_text_area.xml
+++ b/packages/SystemUI/res/layout/screen_pinning_request_text_area.xml
@@ -16,65 +16,72 @@
  * limitations under the License.
  */
 -->
-<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
-    android:id="@+id/screen_pinning_text_area"
+<ScrollView xmlns:android="http://schemas.android.com/apk/res/android"
     android:layout_width="wrap_content"
     android:layout_height="wrap_content"
-    android:background="?android:attr/colorAccent"
-    android:gravity="center_vertical">
+    android:fillViewport="true">
 
-    <TextView
-        android:id="@+id/screen_pinning_title"
-        android:layout_width="match_parent"
-        android:layout_height="wrap_content"
-        android:paddingEnd="48dp"
-        android:paddingStart="48dp"
-        android:paddingTop="43dp"
-        android:text="@string/screen_pinning_title"
-        android:textColor="@android:color/white"
-        android:textSize="24sp" />
-
-    <TextView
-        android:id="@+id/screen_pinning_description"
-        android:layout_width="match_parent"
-        android:layout_height="wrap_content"
-        android:layout_below="@id/screen_pinning_title"
-        android:paddingEnd="48dp"
-        android:paddingStart="48dp"
-        android:paddingTop="12.6dp"
-        android:text="@string/screen_pinning_description"
-        android:textColor="@android:color/white"
-        android:textSize="16sp" />
-
-    <Button
-        android:id="@+id/screen_pinning_ok_button"
-        style="@android:style/Widget.Material.Button"
+    <RelativeLayout
+        android:id="@+id/screen_pinning_text_area"
         android:layout_width="wrap_content"
-        android:layout_height="36dp"
-        android:layout_alignParentEnd="true"
-        android:layout_below="@+id/screen_pinning_description"
-        android:layout_marginEnd="40dp"
-        android:layout_marginTop="18dp"
-        android:background="@null"
-        android:paddingEnd="8dp"
-        android:paddingStart="8dp"
-        android:text="@string/screen_pinning_positive"
-        android:textColor="@android:color/white"
-        android:textSize="14sp" />
+        android:layout_height="wrap_content"
+        android:background="?android:attr/colorAccent"
+        android:gravity="center_vertical">
 
-    <Button
-        android:id="@+id/screen_pinning_cancel_button"
-        style="@android:style/Widget.Material.Button"
-        android:layout_width="wrap_content"
-        android:layout_height="36dp"
-        android:layout_alignTop="@id/screen_pinning_ok_button"
-        android:layout_marginEnd="4dp"
-        android:layout_toStartOf="@id/screen_pinning_ok_button"
-        android:background="@null"
-        android:paddingEnd="8dp"
-        android:paddingStart="8dp"
-        android:text="@string/screen_pinning_negative"
-        android:textColor="@android:color/white"
-        android:textSize="14sp" />
+        <TextView
+            android:id="@+id/screen_pinning_title"
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            android:paddingEnd="48dp"
+            android:paddingStart="48dp"
+            android:paddingTop="43dp"
+            android:text="@string/screen_pinning_title"
+            android:textColor="@android:color/white"
+            android:textSize="24sp" />
 
-</RelativeLayout>
+        <TextView
+            android:id="@+id/screen_pinning_description"
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            android:layout_below="@id/screen_pinning_title"
+            android:paddingEnd="48dp"
+            android:paddingStart="48dp"
+            android:paddingTop="12.6dp"
+            android:text="@string/screen_pinning_description"
+            android:textColor="@android:color/white"
+            android:textSize="16sp" />
+
+        <Button
+            android:id="@+id/screen_pinning_ok_button"
+            style="@android:style/Widget.Material.Button"
+            android:layout_width="wrap_content"
+            android:layout_height="36dp"
+            android:layout_alignParentEnd="true"
+            android:layout_below="@+id/screen_pinning_description"
+            android:layout_marginEnd="40dp"
+            android:layout_marginTop="18dp"
+            android:background="@null"
+            android:paddingEnd="8dp"
+            android:paddingStart="8dp"
+            android:text="@string/screen_pinning_positive"
+            android:textColor="@android:color/white"
+            android:textSize="14sp" />
+
+        <Button
+            android:id="@+id/screen_pinning_cancel_button"
+            style="@android:style/Widget.Material.Button"
+            android:layout_width="wrap_content"
+            android:layout_height="36dp"
+            android:layout_alignTop="@id/screen_pinning_ok_button"
+            android:layout_marginEnd="4dp"
+            android:layout_toStartOf="@id/screen_pinning_ok_button"
+            android:background="@null"
+            android:paddingEnd="8dp"
+            android:paddingStart="8dp"
+            android:text="@string/screen_pinning_negative"
+            android:textColor="@android:color/white"
+            android:textSize="14sp" />
+
+    </RelativeLayout>
+
+</ScrollView>