Merge pull request #295 from BuildmLearn/backpress-fix

FIXED #255, Backpress should deselect any selected item if any

Author: opticod

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
      */

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
      */

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
      */