From 0e892be48e20ce043c5932959156127e63700b80 Mon Sep 17 00:00:00 2001
From: mcclure <andi.m.mcclure@gmail.com>
Date: Sat, 15 Oct 2022 13:04:27 -0400
Subject: [PATCH] Add a description to the focus dialog (#2711)

* Show explanation atop focus set dialog

* Update app/src/main/res/layout/dialog_focus.xml

Per Connyduck suggestion

Co-authored-by: Konrad Pozniak <connyduck@users.noreply.github.com>

Co-authored-by: Konrad Pozniak <connyduck@users.noreply.github.com>
---
 app/src/main/res/layout/dialog_focus.xml | 38 +++++++++++++++++-------
 app/src/main/res/values/strings.xml      |  1 +
 2 files changed, 29 insertions(+), 10 deletions(-)

diff --git a/app/src/main/res/layout/dialog_focus.xml b/app/src/main/res/layout/dialog_focus.xml
index a5092892..65e44df3 100644
--- a/app/src/main/res/layout/dialog_focus.xml
+++ b/app/src/main/res/layout/dialog_focus.xml
@@ -1,17 +1,35 @@
 <?xml version="1.0" encoding="utf-8"?>
-<FrameLayout xmlns:android="http://schemas.android.com/apk/res/android"
+<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
     android:layout_width="match_parent"
-    android:layout_height="match_parent">
+    android:layout_height="match_parent"
+    android:orientation="vertical">
 
-    <!-- todo add padding -->
-    <ImageView
-        android:id="@+id/imageView"
+    <TextView
+        android:id="@+id/focusExplanation"
         android:layout_width="match_parent"
-        android:layout_height="match_parent" />
+        android:layout_height="wrap_content"
+        android:layout_gravity="center_horizontal"
+        android:padding="24dp"
+        android:gravity="center"
+        android:textSize="18sp"
+        android:text="@string/set_focus_description" />
 
-    <com.keylesspalace.tusky.components.compose.view.FocusIndicatorView
-        android:id="@+id/focusIndicator"
+    <FrameLayout
         android:layout_width="match_parent"
-        android:layout_height="match_parent" />
+        android:layout_height="wrap_content"
+        android:id="@+id/imageContainer">
 
-</FrameLayout>
\ No newline at end of file
+        <!-- todo add padding -->
+        <ImageView
+            android:id="@+id/imageView"
+            android:layout_width="match_parent"
+            android:layout_height="match_parent" />
+
+        <com.keylesspalace.tusky.components.compose.view.FocusIndicatorView
+            android:id="@+id/focusIndicator"
+            android:layout_width="match_parent"
+            android:layout_height="match_parent" />
+
+    </FrameLayout>
+
+</LinearLayout>
\ No newline at end of file
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 18c5c8bb..75824a9a 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -408,6 +408,7 @@
     <plurals name="hint_describe_for_visually_impaired">
         <item quantity="other">Describe for visually impaired\n(%d character limit)</item>
     </plurals>
+    <string name="set_focus_description">Tap or drag the circle to choose the focal point which will always be visible in thumbnails.</string>
     <string name="action_set_caption">Set caption</string>
     <string name="action_set_focus">Set focus point</string>
     <string name="action_edit_image">Edit image</string>