From 4f0cc76a12d72bb61444f27311314bb475582636 Mon Sep 17 00:00:00 2001
From: "David A. Velasco" <dvelasco@solidgear.es>
Date: Tue, 5 May 2015 09:54:49 +0200
Subject: [PATCH] Removed unneeded toggling on cursor visibility

---
 res/layout/passcodelock.xml                                | 12 +++++++++---
 src/com/owncloud/android/ui/activity/PassCodeActivity.java |  9 ---------
 2 files changed, 9 insertions(+), 12 deletions(-)

diff --git a/res/layout/passcodelock.xml b/res/layout/passcodelock.xml
index 6a3c9581..38c358f0 100644
--- a/res/layout/passcodelock.xml
+++ b/res/layout/passcodelock.xml
@@ -58,15 +58,21 @@
 
         <EditText
             android:id="@+id/txt1"
-            style="@style/PassCodeStyle" />
+            style="@style/PassCodeStyle"
+            android:cursorVisible="true"
+            />
 
         <EditText
             android:id="@+id/txt2"
-            style="@style/PassCodeStyle" />
+            style="@style/PassCodeStyle"
+            android:cursorVisible="true"
+            />
 
         <EditText
             android:id="@+id/txt3"
-            style="@style/PassCodeStyle" />
+            style="@style/PassCodeStyle"
+            android:cursorVisible="true"
+            />
     </LinearLayout>
 
     <Button
diff --git a/src/com/owncloud/android/ui/activity/PassCodeActivity.java b/src/com/owncloud/android/ui/activity/PassCodeActivity.java
index 331ac5f6..1eb7f8be 100644
--- a/src/com/owncloud/android/ui/activity/PassCodeActivity.java
+++ b/src/com/owncloud/android/ui/activity/PassCodeActivity.java
@@ -174,10 +174,8 @@ public class PassCodeActivity extends SherlockFragmentActivity {
             @Override
             public void onFocusChange(View v, boolean hasFocus) {
                 /// TODO WIP: should take advantage of hasFocus to reduce processing
-                mPassCodeEditTexts[1].setCursorVisible(true);      // TODO WIP this could be made static, or just nothing, since default is true
                 if (mPassCodeEditTexts[0].getText().toString().equals("")) {    // TODO WIP validation could be done in a global way, with a single OnFocusChangeListener for all the input fields
                     mPassCodeEditTexts[1].setSelected(false);
-                    mPassCodeEditTexts[1].setCursorVisible(false); // TODO WIP don't think we really need to enable and disable it; fields without focus should not show it
                     mPassCodeEditTexts[0].requestFocus();
                     mPassCodeEditTexts[0].setSelected(true);   // TODO WIP really needed? is it for the colour highlight
                     mPassCodeEditTexts[0].setSelection(0);     // TODO WIP what is THIS for?
@@ -215,16 +213,13 @@ public class PassCodeActivity extends SherlockFragmentActivity {
 
             @Override
             public void onFocusChange(View v, boolean hasFocus) {
-                mPassCodeEditTexts[2].setCursorVisible(true);
                 if (mPassCodeEditTexts[0].getText().toString().equals("")) {
                     mPassCodeEditTexts[2].setSelected(false);
-                    mPassCodeEditTexts[2].setCursorVisible(false);
                     mPassCodeEditTexts[0].requestFocus();
                     mPassCodeEditTexts[0].setSelected(true);
                     mPassCodeEditTexts[0].setSelection(0);
                 } else if (mPassCodeEditTexts[1].getText().toString().equals("")) {
                     mPassCodeEditTexts[2].setSelected(false);
-                    mPassCodeEditTexts[2].setCursorVisible(false);
                     mPassCodeEditTexts[1].requestFocus();
                     mPassCodeEditTexts[1].setSelected(true);
                     mPassCodeEditTexts[1].setSelection(0);
@@ -261,23 +256,19 @@ public class PassCodeActivity extends SherlockFragmentActivity {
 
             @Override
             public void onFocusChange(View v, boolean hasFocus) {
-                mPassCodeEditTexts[3]. setCursorVisible(true);
 
                 if (mPassCodeEditTexts[0].getText().toString().equals("")) {
                     mPassCodeEditTexts[3].setSelected(false);
-                    mPassCodeEditTexts[3].setCursorVisible(false);
                     mPassCodeEditTexts[0].requestFocus();
                     mPassCodeEditTexts[0].setSelected(true);
                     mPassCodeEditTexts[0].setSelection(0);
                 } else if (mPassCodeEditTexts[1].getText().toString().equals("")) {
                     mPassCodeEditTexts[3].setSelected(false);
-                    mPassCodeEditTexts[3].setCursorVisible(false);
                     mPassCodeEditTexts[1].requestFocus();
                     mPassCodeEditTexts[1].setSelected(true);
                     mPassCodeEditTexts[1].setSelection(0);
                 } else if (mPassCodeEditTexts[2].getText().toString().equals("")) {
                     mPassCodeEditTexts[3].setSelected(false);
-                    mPassCodeEditTexts[3].setCursorVisible(false);
                     mPassCodeEditTexts[2].requestFocus();
                     mPassCodeEditTexts[2].setSelected(true);
                     mPassCodeEditTexts[2].setSelection(0);
-- 
2.11.0