From fae653540b00d03f9ee8172fed7efe774d7106aa Mon Sep 17 00:00:00 2001
From: Nicholas Tinsley <nicholas@signal.org>
Date: Thu, 13 Jul 2023 11:49:08 -0400
Subject: [PATCH] Make Safety Number Changed dialog scrollable.

This helps with smaller screens.
---
 ...afety_number_changed_learn_more_dialog.xml | 60 ++++++++++---------
 1 file changed, 32 insertions(+), 28 deletions(-)

diff --git a/app/src/main/res/layout/safety_number_changed_learn_more_dialog.xml b/app/src/main/res/layout/safety_number_changed_learn_more_dialog.xml
index 44e98a49f7..631a9a9a86 100644
--- a/app/src/main/res/layout/safety_number_changed_learn_more_dialog.xml
+++ b/app/src/main/res/layout/safety_number_changed_learn_more_dialog.xml
@@ -1,37 +1,41 @@
 <?xml version="1.0" encoding="utf-8"?>
-<LinearLayout
-    xmlns:android="http://schemas.android.com/apk/res/android"
-    tools:viewBindingIgnore="true"
+<ScrollView 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"
     android:layout_width="match_parent"
-    android:layout_height="wrap_content"
-    android:orientation="vertical"
-    android:padding="24dp">
+    android:layout_height="wrap_content">
 
-  <ImageView
-      android:layout_width="match_parent"
-      android:layout_height="wrap_content"
-      android:layout_marginBottom="10dp"
-      android:importantForAccessibility="no"
-      app:srcCompat="@drawable/safety_number_change_banner" />
+    <LinearLayout
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:orientation="vertical"
+        android:padding="24dp"
+        tools:viewBindingIgnore="true">
 
-  <TextView
-      android:id="@+id/safety_number_learn_more_title"
-      android:layout_width="match_parent"
-      android:layout_height="wrap_content"
-      android:layout_marginBottom="10dp"
-      style="@style/Signal.Text.Headline"
-      android:fontFamily="sans-serif-medium"
-      android:textSize="20sp"
-      tools:text="Your safety number with Peter Parker changed"/>
+        <ImageView
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            android:layout_marginBottom="10dp"
+            android:importantForAccessibility="no"
+            app:srcCompat="@drawable/safety_number_change_banner" />
 
-  <TextView
-      android:id="@+id/safety_number_learn_more_body"
-      android:layout_width="match_parent"
-      android:layout_height="wrap_content"
-      style="@style/Signal.Text.Body"
-      tools:text="Your Safety Number with Peter Parker changed, likely because they reinstalled Signal or changed devices. Tap Verify to confirm the new safety number. This is optional."/>
+        <TextView
+            android:id="@+id/safety_number_learn_more_title"
+            style="@style/Signal.Text.Headline"
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            android:layout_marginBottom="10dp"
+            android:fontFamily="sans-serif-medium"
+            android:textSize="20sp"
+            tools:text="Your safety number with Peter Parker changed" />
+
+        <TextView
+            android:id="@+id/safety_number_learn_more_body"
+            style="@style/Signal.Text.Body"
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            tools:text="Your Safety Number with Peter Parker changed, likely because they reinstalled Signal or changed devices. Tap Verify to confirm the new safety number. This is optional." />
 
 
-</LinearLayout>
\ No newline at end of file
+    </LinearLayout>
+</ScrollView>
\ No newline at end of file