From 17f347cab50e556a25d7b9c072ee357b65c9f424 Mon Sep 17 00:00:00 2001
From: Paul <naydex.mc@gmail.com>
Date: Mon, 8 May 2017 10:17:41 +0000
Subject: [PATCH] Add loading on login activity while fetching tokens

---
 .../keylesspalace/tusky/LoginActivity.java    | 22 ++++++
 app/src/main/res/layout/activity_login.xml    | 78 ++++++++++++-------
 app/src/main/res/values-fr/strings.xml        |  2 +
 app/src/main/res/values/strings.xml           |  2 +
 4 files changed, 77 insertions(+), 27 deletions(-)

diff --git a/app/src/main/java/com/keylesspalace/tusky/LoginActivity.java b/app/src/main/java/com/keylesspalace/tusky/LoginActivity.java
index 8ec477484..9226dc6cf 100644
--- a/app/src/main/java/com/keylesspalace/tusky/LoginActivity.java
+++ b/app/src/main/java/com/keylesspalace/tusky/LoginActivity.java
@@ -32,6 +32,7 @@ import android.text.method.LinkMovementMethod;
 import android.view.View;
 import android.widget.Button;
 import android.widget.EditText;
+import android.widget.LinearLayout;
 import android.widget.TextView;
 
 import com.keylesspalace.tusky.entity.AccessToken;
@@ -62,6 +63,9 @@ public class LoginActivity extends AppCompatActivity {
     private String clientId;
     private String clientSecret;
 
+    @BindView(R.id.login_input) LinearLayout input;
+    @BindView(R.id.login_loading) LinearLayout loading;
+
     @BindView(R.id.edit_text_domain) EditText editText;
     @BindView(R.id.button_login) Button button;
     @BindView(R.id.whats_an_instance) TextView whatsAnInstance;
@@ -326,6 +330,8 @@ public class LoginActivity extends AppCompatActivity {
                 domain = preferences.getString("domain", null);
                 clientId = preferences.getString("clientId", null);
                 clientSecret = preferences.getString("clientSecret", null);
+
+                setLoading(true);
                 /* Since authorization has succeeded, the final step to log in is to exchange
                  * the authorization code for an access token. */
                 Callback<AccessToken> callback = new Callback<AccessToken>() {
@@ -334,6 +340,8 @@ public class LoginActivity extends AppCompatActivity {
                         if (response.isSuccessful()) {
                             onLoginSuccess(response.body().accessToken);
                         } else {
+                            setLoading(false);
+
                             editText.setError(getString(R.string.error_retrieving_oauth_token));
                             Log.e(TAG, String.format("%s %s",
                                     getString(R.string.error_retrieving_oauth_token),
@@ -343,6 +351,7 @@ public class LoginActivity extends AppCompatActivity {
 
                     @Override
                     public void onFailure(Call<AccessToken> call, Throwable t) {
+                        setLoading(false);
                         editText.setError(getString(R.string.error_retrieving_oauth_token));
                         Log.e(TAG, String.format("%s %s",
                                 getString(R.string.error_retrieving_oauth_token),
@@ -355,21 +364,34 @@ public class LoginActivity extends AppCompatActivity {
             } else if (error != null) {
                 /* Authorization failed. Put the error response where the user can read it and they
                  * can try again. */
+                setLoading(false);
                 editText.setError(getString(R.string.error_authorization_denied));
                 Log.e(TAG, getString(R.string.error_authorization_denied) + error);
             } else {
+                setLoading(false);
                 // This case means a junk response was received somehow.
                 editText.setError(getString(R.string.error_authorization_unknown));
             }
         }
     }
 
+    private void setLoading(boolean loadingState) {
+        if (loadingState) {
+            loading.setVisibility(View.VISIBLE);
+            input.setVisibility(View.GONE);
+        } else {
+            loading.setVisibility(View.GONE);
+            input.setVisibility(View.VISIBLE);
+        }
+    }
+
     private void onLoginSuccess(String accessToken) {
         boolean committed = preferences.edit()
                 .putString("domain", domain)
                 .putString("accessToken", accessToken)
                 .commit();
         if (!committed) {
+            setLoading(false);
             editText.setError(getString(R.string.error_retrieving_oauth_token));
             return;
         }
diff --git a/app/src/main/res/layout/activity_login.xml b/app/src/main/res/layout/activity_login.xml
index 8df45c405..f65930e60 100644
--- a/app/src/main/res/layout/activity_login.xml
+++ b/app/src/main/res/layout/activity_login.xml
@@ -20,39 +20,63 @@
             android:src="@drawable/elephant_friend"
             android:contentDescription="@null" />
 
-        <android.support.design.widget.TextInputLayout
+        <LinearLayout
+            android:id="@+id/login_input"
+            android:layout_width="wrap_content"
             android:layout_height="wrap_content"
-            android:layout_width="250dp">
-            <android.support.design.widget.TextInputEditText
-                android:layout_width="match_parent"
+            android:orientation="vertical">
+            <android.support.design.widget.TextInputLayout
                 android:layout_height="wrap_content"
-                android:inputType="textUri"
-                android:hint="@string/hint_domain"
-                android:ems="10"
-                android:id="@+id/edit_text_domain" />
-        </android.support.design.widget.TextInputLayout>
+                android:layout_width="250dp">
+                <android.support.design.widget.TextInputEditText
+                    android:layout_width="match_parent"
+                    android:layout_height="wrap_content"
+                    android:inputType="textUri"
+                    android:hint="@string/hint_domain"
+                    android:ems="10"
+                    android:id="@+id/edit_text_domain" />
+            </android.support.design.widget.TextInputLayout>
 
-        <Button
-            android:id="@+id/button_login"
-            android:layout_width="250dp"
-            android:layout_height="wrap_content"
-            android:layout_marginTop="4dp"
-            android:textColor="@android:color/white"
-            android:text="@string/action_login" />
+            <Button
+                android:id="@+id/button_login"
+                android:layout_width="250dp"
+                android:layout_height="wrap_content"
+                android:layout_marginTop="4dp"
+                android:textColor="@android:color/white"
+                android:text="@string/action_login" />
 
-        <TextView
-            android:layout_width="250dp"
-            android:layout_height="wrap_content"
+            <TextView
+                android:layout_width="250dp"
+                android:layout_height="wrap_content"
+                android:visibility="gone"
+                android:id="@+id/text_error" />
+
+            <TextView
+                android:layout_width="250dp"
+                android:layout_height="wrap_content"
+                android:paddingTop="5dp"
+                android:textAlignment="center"
+                android:id="@+id/whats_an_instance"
+                android:text="@string/link_whats_an_instance" />
+        </LinearLayout>
+        <LinearLayout
+            android:id="@+id/login_loading"
             android:visibility="gone"
-            android:id="@+id/text_error" />
+            android:orientation="vertical"
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content">
+            <ProgressBar
+                android:layout_gravity="center"
+                android:layout_width="wrap_content"
+                android:layout_height="wrap_content" />
+            <TextView
+                android:paddingTop="10dp"
+                android:textAlignment="center"
+                android:layout_width="250dp"
+                android:layout_height="wrap_content"
+                android:text="@string/login_connection"/>
+        </LinearLayout>
 
-        <TextView
-            android:layout_width="250dp"
-            android:layout_height="wrap_content"
-            android:paddingTop="5dp"
-            android:textAlignment="center"
-            android:id="@+id/whats_an_instance"
-            android:text="@string/link_whats_an_instance" />
 
     </LinearLayout>
 </ScrollView>
\ No newline at end of file
diff --git a/app/src/main/res/values-fr/strings.xml b/app/src/main/res/values-fr/strings.xml
index 82250ab76..db0fe884e 100644
--- a/app/src/main/res/values-fr/strings.xml
+++ b/app/src/main/res/values-fr/strings.xml
@@ -111,6 +111,8 @@
 
     <string name="link_whats_an_instance">Qu’est-ce qu’une instance ?</string>
 
+    <string name="login_connection">Connexion en cours…</string>
+
     <string name="dialog_whats_an_instance">Indiquer ici l’adresse ou le domaine d’une instance, comme mastodon.social, icosahedron.website, social.tchncs.de,
         <a href="https://github.com/tootsuite/mastodon/blob/master/docs/Using-Mastodon/List-of-Mastodon-instances.md">et bien d’autres encore</a> (en anglais) !
         \n\nSi vous ne disposez d’aucun compte, vous pouvez renseigner le nom de l’instance que vous souhaitez rejoindre et y créer un compte.\n\nUne instance est l’endroit où votre compte est
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 7b99be651..b13921d57 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -122,6 +122,8 @@
 
     <string name="link_whats_an_instance">What\'s an instance?</string>
 
+    <string name="login_connection">Connecting…</string>
+
     <string name="dialog_whats_an_instance">The address or domain of any instance can be entered
         here, such as mastodon.social, icosahedron.website, social.tchncs.de, and
         <a href="https://github.com/tootsuite/mastodon/blob/master/docs/Using-Mastodon/List-of-Mastodon-instances.md">more!</a>