From f4d1567e3fc090e5206aa5f4f69de194074b6f48 Mon Sep 17 00:00:00 2001
From: Anupam Das <anupam.das.bwn@gmail.com>
Date: Tue, 13 Dec 2016 21:22:28 +0530
Subject: [PATCH] FIXED #255, Backpress should deselect any selected item if
 any

---
 .../toolkit/fragment/DraftsFragment.java      | 32 +++++++++++++++----
 .../toolkit/fragment/LoadApkFragment.java     | 31 ++++++++++++++----
 .../toolkit/fragment/LoadProjectFragment.java | 31 ++++++++++++++----
 3 files changed, 76 insertions(+), 18 deletions(-)

diff --git a/source-code/app/src/main/java/org/buildmlearn/toolkit/fragment/DraftsFragment.java b/source-code/app/src/main/java/org/buildmlearn/toolkit/fragment/DraftsFragment.java
index 513edb81..4b5519f2 100644
--- a/source-code/app/src/main/java/org/buildmlearn/toolkit/fragment/DraftsFragment.java
+++ b/source-code/app/src/main/java/org/buildmlearn/toolkit/fragment/DraftsFragment.java
@@ -11,6 +11,7 @@
 import android.support.v7.app.AlertDialog;
 import android.support.v7.app.AppCompatActivity;
 import android.util.Log;
+import android.view.KeyEvent;
 import android.view.LayoutInflater;
 import android.view.Menu;
 import android.view.MenuItem;
@@ -117,6 +118,21 @@ public boolean onItemLongClick(AdapterView<?> parent, View view, int position, l
                 return true;
             }
         });
+
+        getView().setFocusableInTouchMode(true);
+        getView().requestFocus();
+        getView().setOnKeyListener(new View.OnKeyListener() {
+            @Override
+            public boolean onKey(View v, int keyCode, KeyEvent event) {
+                if (keyCode == KeyEvent.KEYCODE_BACK) {
+                    if (mAdapter.selectedPositionsSize() > 0) {
+                        unselectAll();
+                        return true;
+                    }
+                }
+                return false;
+            }
+        });
     }
 
     /**
@@ -316,12 +332,7 @@ public void onClick(View v) {
                 break;
 
             case R.id.action_unselect_all:
-                for(int i=0;i<mAdapter.getCount();i++)
-                    if(mAdapter.isPositionSelected(i)) {
-                        mListView.getChildAt(i).setBackgroundColor(0);
-                        mAdapter.removeSelectedPosition(i);
-                    }
-                restoreColorScheme();
+                unselectAll();
                 break;
 
             case R.id.action_delete_all:
@@ -349,6 +360,15 @@ public void onClick(View v) {
         return super.onOptionsItemSelected(item);
     }
 
+    private void unselectAll() {
+        for (int i = 0; i < mAdapter.getCount(); i++)
+            if (mAdapter.isPositionSelected(i)) {
+                mListView.getChildAt(i).setBackgroundColor(0);
+                mAdapter.removeSelectedPosition(i);
+            }
+        restoreColorScheme();
+    }
+
     /**
      * @brief Removes selected project item
      */
diff --git a/source-code/app/src/main/java/org/buildmlearn/toolkit/fragment/LoadApkFragment.java b/source-code/app/src/main/java/org/buildmlearn/toolkit/fragment/LoadApkFragment.java
index a47d512b..4f30ef77 100644
--- a/source-code/app/src/main/java/org/buildmlearn/toolkit/fragment/LoadApkFragment.java
+++ b/source-code/app/src/main/java/org/buildmlearn/toolkit/fragment/LoadApkFragment.java
@@ -147,6 +147,21 @@ public boolean onItemLongClick(AdapterView<?> parent, View view, int position, l
                 return true;
             }
         });
+
+        getView().setFocusableInTouchMode(true);
+        getView().requestFocus();
+        getView().setOnKeyListener(new View.OnKeyListener() {
+            @Override
+            public boolean onKey(View v, int keyCode, KeyEvent event) {
+                if (keyCode == KeyEvent.KEYCODE_BACK) {
+                    if (mAdapter.selectedPositionsSize() > 0) {
+                        unselectAll();
+                        return true;
+                    }
+                }
+                return false;
+            }
+        });
     }
 
     /**
@@ -409,12 +424,7 @@ public boolean onKey(View v, int keyCode, KeyEvent event) {
                 break;
 
             case R.id.action_unselect_all:
-                for(int i=0;i<mAdapter.getCount();i++)
-                    if(mAdapter.isPositionSelected(i)) {
-                        mListView.getChildAt(i).setBackgroundColor(0);
-                        mAdapter.removeSelectedPosition(i);
-                    }
-                restoreColorScheme();
+                unselectAll();
                 break;
 
             default: //do nothing
@@ -423,6 +433,15 @@ public boolean onKey(View v, int keyCode, KeyEvent event) {
         return super.onOptionsItemSelected(item);
     }
 
+    public void unselectAll() {
+        for (int i = 0; i < mAdapter.getCount(); i++)
+            if (mAdapter.isPositionSelected(i)) {
+                mListView.getChildAt(i).setBackgroundColor(0);
+                mAdapter.removeSelectedPosition(i);
+            }
+        restoreColorScheme();
+    }
+
     /**
      * @brief Removes selected apk item
      */
diff --git a/source-code/app/src/main/java/org/buildmlearn/toolkit/fragment/LoadProjectFragment.java b/source-code/app/src/main/java/org/buildmlearn/toolkit/fragment/LoadProjectFragment.java
index 6a295fe8..810b6ce4 100644
--- a/source-code/app/src/main/java/org/buildmlearn/toolkit/fragment/LoadProjectFragment.java
+++ b/source-code/app/src/main/java/org/buildmlearn/toolkit/fragment/LoadProjectFragment.java
@@ -162,6 +162,21 @@ public boolean onItemLongClick(AdapterView<?> parent, View view, int position, l
                 return true;
             }
         });
+
+        getView().setFocusableInTouchMode(true);
+        getView().requestFocus();
+        getView().setOnKeyListener(new View.OnKeyListener() {
+            @Override
+            public boolean onKey(View v, int keyCode, KeyEvent event) {
+                if (keyCode == KeyEvent.KEYCODE_BACK) {
+                    if (mAdapter.selectedPositionsSize() > 0) {
+                        unselectAll();
+                        return true;
+                    }
+                }
+                return false;
+            }
+        });
     }
 
     /**
@@ -449,12 +464,7 @@ public boolean onKey(View v, int keyCode, KeyEvent event) {
                 break;
 
             case R.id.action_unselect_all:
-                for(int i=0;i<mAdapter.getCount();i++)
-                    if(mAdapter.isPositionSelected(i)) {
-                        mListView.getChildAt(i).setBackgroundColor(0);
-                        mAdapter.removeSelectedPosition(i);
-                    }
-                restoreColorScheme();
+                unselectAll();
                 break;
 
             default: //do nothing
@@ -463,6 +473,15 @@ public boolean onKey(View v, int keyCode, KeyEvent event) {
         return super.onOptionsItemSelected(item);
     }
 
+    private void unselectAll() {
+        for (int i = 0; i < mAdapter.getCount(); i++)
+            if (mAdapter.isPositionSelected(i)) {
+                mListView.getChildAt(i).setBackgroundColor(0);
+                mAdapter.removeSelectedPosition(i);
+            }
+        restoreColorScheme();
+    }
+
     /**
      * @brief Removes selected project item
      */