From eabaead700864820dfa1d8ac44b4ce84228b2177 Mon Sep 17 00:00:00 2001
From: Greyson Parrelli <greyson@signal.org>
Date: Tue, 14 Aug 2018 13:04:33 -0700
Subject: [PATCH] Revert "Fingerprint unlock UX improvements"

This reverts commit 8441ac4091fff3c6f6ae984d9b76c4b45b10c36b.

Fixes #8106
---
 .../ic_priority_high_white_48dp.png           | Bin 184 -> 0 bytes
 .../ic_priority_high_white_48dp.png           | Bin 144 -> 0 bytes
 .../ic_priority_high_white_48dp.png           | Bin 228 -> 0 bytes
 .../ic_priority_high_white_48dp.png           | Bin 335 -> 0 bytes
 .../ic_priority_high_white_48dp.png           | Bin 465 -> 0 bytes
 res/layout/prompt_passphrase_activity.xml     |  44 ++++++++----------
 res/values/strings.xml                        |   4 --
 res/values/styles.xml                         |   5 --
 res/values/themes.xml                         |   2 -
 .../securesms/PassphrasePromptActivity.java   |  25 ++++------
 10 files changed, 28 insertions(+), 52 deletions(-)
 delete mode 100644 res/drawable-hdpi/ic_priority_high_white_48dp.png
 delete mode 100644 res/drawable-mdpi/ic_priority_high_white_48dp.png
 delete mode 100644 res/drawable-xhdpi/ic_priority_high_white_48dp.png
 delete mode 100644 res/drawable-xxhdpi/ic_priority_high_white_48dp.png
 delete mode 100644 res/drawable-xxxhdpi/ic_priority_high_white_48dp.png

diff --git a/res/drawable-hdpi/ic_priority_high_white_48dp.png b/res/drawable-hdpi/ic_priority_high_white_48dp.png
deleted file mode 100644
index 66366c9d42b0098054f39a28407499cec4c197b3..0000000000000000000000000000000000000000
GIT binary patch
literal 0
HcmV?d00001

literal 184
zcmeAS@N?(olHy`uVBq!ia0vp^9w5xY0wn)GsXhaw>OEZ?Ln;{GUTqa+R^VZ{Xm>5a
z;fB#%h2J6T&%b(CW`2;JiRVq@%$bEMj!i6_LMk2;6i^tyrkv?<QkOJVdpV`NSnFV#
zW9|EQJU4ak?eH$)yLyk^i*M?C=3dJy5f_fSKMN3lbYbiF#5*^aZu@0*>u~w;GiSsu
YoA`fPIr-Eqpi>w;UHx3vIVCg!07~IVU;qFB

diff --git a/res/drawable-mdpi/ic_priority_high_white_48dp.png b/res/drawable-mdpi/ic_priority_high_white_48dp.png
deleted file mode 100644
index a7191587d656e859367309418efbf242fc605e15..0000000000000000000000000000000000000000
GIT binary patch
literal 0
HcmV?d00001

literal 144
zcmeAS@N?(olHy`uVBq!ia0vp^1|ZDA0wn)(8}b0Da8DP<kP61Pmv%EYDDW@@iXZ%G
zEcr6xVa}iH|Mp5ZOlO#Tvvi}k0|Y!+e?Zu%s!4_Ol;m-7so=LPO$o6w{3@?sy*%(U
oyyTzqd`Fh4yZ$bFo^Hk<HSM+MmxH@j0Igu~boFyt=akR{0By1~9{>OV

diff --git a/res/drawable-xhdpi/ic_priority_high_white_48dp.png b/res/drawable-xhdpi/ic_priority_high_white_48dp.png
deleted file mode 100644
index f1dc48c8191f27a7d7edfd4832dac986ed6b3805..0000000000000000000000000000000000000000
GIT binary patch
literal 0
HcmV?d00001

literal 228
zcmeAS@N?(olHy`uVBq!ia0vp^2_VeD0wg^q?%xcgR(rZQhEy=Vz1Et0)PSMwpnBG_
zliLm^&hoVJe6fDV`ZEbH_i@%d>)xmDIcdorL7)cw-~c<*1eKLdm(u)BPSaH_zUA$^
z^yCEN>vJV9T`~TndVXt&(}@YwU%kHZJ3a2zng=>5=d#cKmk;<9_4GgUwmsL_61e}q
zQ7r4NIuZBN>&~?=(y!*%Jo|O$ho8O2Bo)tczQqe?%qlru1ag9>tDnm{r-UW|_{&(J

diff --git a/res/drawable-xxhdpi/ic_priority_high_white_48dp.png b/res/drawable-xxhdpi/ic_priority_high_white_48dp.png
deleted file mode 100644
index 5ca2452c82c3ec6c6588c13b44b00be1d5b8d82d..0000000000000000000000000000000000000000
GIT binary patch
literal 0
HcmV?d00001

literal 335
zcmeAS@N?(olHy`uVBq!ia0vp^6F``S8Ays|{O<-*1_3@Hu0Z-fDZtv7#Q>;>za+>n
zm_b@b$<IHpYR=p}FFybMH}$Ul8lcpEPZ!6K3dXlrx5lyrinx93b~amjfkE~H<1IEn
z<|z(79?TX;cKr6*cauxIJpJvzj-O0#{%~9PKku#OWjJ83=+Yse=+wd?s03wbLYNKt
z<{NhW-eQ`!>7TE^t;(`l-?j-0^NR0TYnVUF^8Kl9>yl@&TaSn&%U$y|eE9H`NVc5U
zvK!w%BzD#EhX(JdTv1=_5kG6g&0im1i+_@~n`rlSG4JMkQ~$qf*Y!CTto`cthby(8
f-tJdsa1gG2YPI^%i@+5?uP}JJ`njxgN@xNAEG?kE

diff --git a/res/drawable-xxxhdpi/ic_priority_high_white_48dp.png b/res/drawable-xxxhdpi/ic_priority_high_white_48dp.png
deleted file mode 100644
index 121e04946200def584e1b69ca62e9b2c443d29ae..0000000000000000000000000000000000000000
GIT binary patch
literal 0
HcmV?d00001

literal 465
zcmeAS@N?(olHy`uVBq!ia0vp^2SAvE1xWt5x}=AJfpLbXi(^Oy<C}}zj0Y4%SQfmF
z|FyQcUN%Gls00F5hb~ueU|?cFBXSfO7?}hVun`TfL|8Z+8W;!@4S6_>fEf#M!jD^9
z^AEQzl;Am}an?&qwCsY*ontYF3T4y%9k;KUqBV~_K+NO#Z?_7`3VUa-ca_T*E?xR?
zZnan2!heQOXPMq!volTT6+f$}_qWKPz?NSJW~ldlJYsRhH0_Ga>NK%eLc(?yu4gT-
zh;q%dyejOvLONv5Vy@CgP49CXQ-mrX%_@#{pC+;CSxh$DL}f3dQzrz!r`&Pxt5APz
zTjb*GCTzJ#buq6cqnzU>p>L8}+=kJDn<T$;8*Y?++kWQ1+d<*wRgaXeh#tFAcS_pZ
zvg_a)_hTP@d(`xHe0lR;RQT}D`+WzmRs2#=@tlM}FWG;v)cMA7>o;Fp4hjKJS3j3^
HP6<r_F~gR%

diff --git a/res/layout/prompt_passphrase_activity.xml b/res/layout/prompt_passphrase_activity.xml
index 59dd09ca92..ca8b673ac2 100644
--- a/res/layout/prompt_passphrase_activity.xml
+++ b/res/layout/prompt_passphrase_activity.xml
@@ -6,21 +6,32 @@
                 android:id="@+id/prompt_layout"
                 android:layout_width="match_parent"
                 android:layout_height="match_parent"
-                android:background="?attr/login_top_background"
                 android:orientation="vertical">
 
+    <View android:id="@+id/shim"
+          android:layout_width="match_parent"
+          android:layout_height="1dp"
+          android:layout_centerVertical="true"
+          android:visibility="invisible"/>
+
+    <FrameLayout
+            android:layout_width="match_parent"
+            android:layout_height="match_parent"
+            android:background="?login_top_background"
+            android:layout_above="@id/shim">
+
+    </FrameLayout>
+
     <android.support.v7.widget.Toolbar
             android:id="@+id/toolbar"
-            android:theme="?attr/actionBarStyle"
             android:layout_width="match_parent"
             android:layout_height="?attr/actionBarSize"
             android:layout_marginTop="20dp">
 
         <ImageView
-                android:layout_width="36dp"
-                android:layout_height="36dp"
+                android:layout_width="wrap_content"
+                android:layout_height="wrap_content"
                 android:src="@drawable/icon_transparent"
-                android:layout_centerVertical="true"
                 android:layout_gravity="center"/>
 
     </android.support.v7.widget.Toolbar>
@@ -28,7 +39,7 @@
 
     <LinearLayout
             android:layout_width="match_parent"
-            android:layout_height="280dp"
+            android:layout_height="wrap_content"
             android:layout_marginLeft="30dp"
             android:layout_marginRight="30dp"
             android:orientation="vertical"
@@ -39,20 +50,11 @@
 
         <TextView android:layout_width="match_parent"
                   android:layout_height="wrap_content"
-                  android:layout_marginBottom="2dp"
+                  android:layout_marginBottom="40dp"
                   android:text="@string/prompt_passphrase_activity__signal_is_locked"
-                  android:textColor="?android:textColorPrimary"
                   android:gravity="center_horizontal"
                   android:textSize="25sp"/>
 
-        <TextView android:layout_width="match_parent"
-            android:layout_height="wrap_content"
-            android:layout_marginBottom="20dp"
-            android:text="@string/PassphrasePromptActivity_confirm_fingerprint"
-            android:textColor="?android:textColorSecondary"
-            android:gravity="center_horizontal"
-            android:textSize="16sp"/>
-
         <ImageView android:id="@+id/fingerprint_auth_container"
                 android:src="@drawable/ic_fingerprint_white_48dp"
                 android:background="@drawable/circle_tintable"
@@ -61,17 +63,9 @@
                 android:layout_width="wrap_content"
                 android:layout_height="wrap_content"
                 android:layout_gravity="center_horizontal"
-                android:layout_marginBottom="25dp"
+                android:layout_marginBottom="60dp"
                 tools:visibility="visible"/>
 
-        <TextView android:id="@+id/fingerprint_status_text"
-            android:layout_width="match_parent"
-            android:layout_height="wrap_content"
-            android:text="@string/PassphrasePromptActivity_touch_sensor"
-            android:textColor="@color/gray27"
-            android:gravity="center_horizontal"
-            android:textSize="16sp"/>
-
         <TextView android:id="@+id/lock_screen_auth_container"
                   android:layout_width="match_parent"
                   android:layout_height="wrap_content"
diff --git a/res/values/strings.xml b/res/values/strings.xml
index 5b9e324d83..7c8f259e0b 100644
--- a/res/values/strings.xml
+++ b/res/values/strings.xml
@@ -464,10 +464,6 @@
     <string name="PassphrasePromptActivity_watermark_content_description">Signal icon</string>
     <string name="PassphrasePromptActivity_ok_button_content_description">Submit passphrase</string>
     <string name="PassphrasePromptActivity_invalid_passphrase_exclamation">Invalid passphrase!</string>
-    <string name="PassphrasePromptActivity_confirm_fingerprint">Confirm fingerprint to unlock</string>
-    <string name="PassphrasePromptActivity_touch_sensor">Touch sensor</string>
-    <string name="PassphrasePromptActivity_fingerprint_not_recognized_try_again">Fingerprint not recognized. Try again.</string>
-    <string name="PassphrasePromptActivity_fingerprint_recognized">Fingerprint recognized</string>
 
     <!-- PlayServicesProblemFragment -->
     <string name="PlayServicesProblemFragment_the_version_of_google_play_services_you_have_installed_is_not_functioning">The version of Google Play Services you have installed is not functioning correctly.  Please reinstall Google Play Services and try again.</string>
diff --git a/res/values/styles.xml b/res/values/styles.xml
index 9228e52b24..1a4cbf1bf6 100644
--- a/res/values/styles.xml
+++ b/res/values/styles.xml
@@ -72,11 +72,6 @@
         <item name="elevation">0dp</item>
     </style>
 
-    <style name="TextSecure.FlatDarkActionBar"
-        parent="@style/TextSecure.DarkActionBar">
-        <item name="elevation">0dp</item>
-    </style>
-
     <style name="TextSecure.DarkActionBar.TabBar"
         parent="@style/Widget.AppCompat.ActionBar.TabBar">
         <item name="background">@color/gray95</item>
diff --git a/res/values/themes.xml b/res/values/themes.xml
index d6a983cd55..be3beacb76 100644
--- a/res/values/themes.xml
+++ b/res/values/themes.xml
@@ -58,7 +58,6 @@
     </style>
 
     <style name="TextSecure.LightIntroTheme" parent="@style/Theme.AppCompat.Light">
-        <item name="actionBarStyle">@style/TextSecure.FlatLightActionBar</item>
         <item name="windowActionBar">false</item>
         <item name="windowNoTitle">true</item>
         <item name="windowActionModeOverlay">true</item>
@@ -78,7 +77,6 @@
     </style>
 
     <style name="TextSecure.DarkIntroTheme" parent="@style/Theme.AppCompat">
-        <item name="actionBarStyle">@style/TextSecure.FlatDarkActionBar</item>
         <item name="windowActionBar">false</item>
         <item name="windowNoTitle">true</item>
         <item name="windowActionModeOverlay">true</item>
diff --git a/src/org/thoughtcrime/securesms/PassphrasePromptActivity.java b/src/org/thoughtcrime/securesms/PassphrasePromptActivity.java
index 65def02d5e..b0aa6bb53a 100644
--- a/src/org/thoughtcrime/securesms/PassphrasePromptActivity.java
+++ b/src/org/thoughtcrime/securesms/PassphrasePromptActivity.java
@@ -73,7 +73,6 @@ public class PassphrasePromptActivity extends PassphraseActivity {
   private View            passphraseAuthContainer;
   private ImageView       fingerprintPrompt;
   private TextView        lockScreenButton;
-  private TextView        fingerprintStatusText;
 
   private EditText        passphraseText;
   private ImageButton     showButton;
@@ -106,9 +105,6 @@ public class PassphrasePromptActivity extends PassphraseActivity {
     dynamicTheme.onResume(this);
     dynamicLanguage.onResume(this);
 
-    fingerprintStatusText.setText(getResources().getString(R.string.PassphrasePromptActivity_touch_sensor));
-    fingerprintStatusText.setTextColor(getResources().getColor(R.color.gray27));
-
     setLockTypeVisibility();
 
     if (TextSecurePreferences.isScreenLockEnabled(this) && !authenticated && !failure) {
@@ -218,7 +214,6 @@ public class PassphrasePromptActivity extends PassphraseActivity {
     passphraseAuthContainer       = findViewById(R.id.password_auth_container);
     fingerprintPrompt             = findViewById(R.id.fingerprint_auth_container);
     lockScreenButton              = findViewById(R.id.lock_screen_auth_container);
-    fingerprintStatusText         = findViewById(R.id.fingerprint_status_text);
     fingerprintManager            = FingerprintManagerCompat.from(this);
     fingerprintCancellationSignal = new CancellationSignal();
     fingerprintListener           = new FingerprintListener();
@@ -345,22 +340,23 @@ public class PassphrasePromptActivity extends PassphraseActivity {
     @Override
     public void onAuthenticationError(int errMsgId, CharSequence errString) {
       Log.w(TAG, "Authentication error: " + errMsgId + " " + errString);
+      onAuthenticationFailed();
     }
 
     @Override
     public void onAuthenticationSucceeded(FingerprintManagerCompat.AuthenticationResult result) {
       Log.i(TAG, "onAuthenticationSucceeded");
       fingerprintPrompt.setImageResource(R.drawable.ic_check_white_48dp);
-      fingerprintPrompt.getBackground().setColorFilter(getResources().getColor(R.color.teal_500), PorterDuff.Mode.SRC_IN);
+      fingerprintPrompt.getBackground().setColorFilter(getResources().getColor(R.color.green_500), PorterDuff.Mode.SRC_IN);
       fingerprintPrompt.animate().setInterpolator(new BounceInterpolator()).scaleX(1.1f).scaleY(1.1f).setDuration(500).setListener(new AnimationCompleteListener() {
         @Override
         public void onAnimationEnd(Animator animation) {
           handleAuthenticated();
+
+          fingerprintPrompt.setImageResource(R.drawable.ic_fingerprint_white_48dp);
+          fingerprintPrompt.getBackground().setColorFilter(getResources().getColor(R.color.signal_primary), PorterDuff.Mode.SRC_IN);
         }
       }).start();
-
-      fingerprintStatusText.setText(getResources().getString(R.string.PassphrasePromptActivity_fingerprint_recognized));
-      fingerprintStatusText.setTextColor(getResources().getColor(R.color.teal_500));
     }
 
     @Override
@@ -368,12 +364,12 @@ public class PassphrasePromptActivity extends PassphraseActivity {
       Log.w(TAG, "onAuthenticatoinFailed()");
       FingerprintManagerCompat.AuthenticationCallback callback = this;
 
-      fingerprintPrompt.setImageResource(R.drawable.ic_priority_high_white_48dp);
-      fingerprintPrompt.getBackground().setColorFilter(getResources().getColor(R.color.deep_orange_600), PorterDuff.Mode.SRC_IN);
+      fingerprintPrompt.setImageResource(R.drawable.ic_close_white_48dp);
+      fingerprintPrompt.getBackground().setColorFilter(getResources().getColor(R.color.red_500), PorterDuff.Mode.SRC_IN);
 
-      TranslateAnimation shake = new TranslateAnimation(0, 20, 0, 0);
+      TranslateAnimation shake = new TranslateAnimation(0, 30, 0, 0);
       shake.setDuration(50);
-      shake.setRepeatCount(5);
+      shake.setRepeatCount(7);
       shake.setAnimationListener(new Animation.AnimationListener() {
         @Override
         public void onAnimationStart(Animation animation) {}
@@ -389,9 +385,6 @@ public class PassphrasePromptActivity extends PassphraseActivity {
       });
 
       fingerprintPrompt.startAnimation(shake);
-
-      fingerprintStatusText.setText(getResources().getString(R.string.PassphrasePromptActivity_fingerprint_not_recognized_try_again));
-      fingerprintStatusText.setTextColor(getResources().getColor(R.color.deep_orange_600));
     }
 
   }