Fix click support. Showing 1 of many options

Author: jpshelley

demo/src/main/java/com/daimajia/swipedemo/RecyclerViewExample.java

@@ -18,6 +18,7 @@
 import com.daimajia.swipe.util.Attributes;
 import com.daimajia.swipedemo.adapter.RecyclerViewAdapter;
 import com.daimajia.swipedemo.adapter.RecyclerViewAdvancedAdapter;
+import com.daimajia.swipedemo.adapter.util.RecyclerItemClickListener;
 
 import org.lucasr.twowayview.ItemClickSupport;
 import org.lucasr.twowayview.TwoWayLayoutManager;
@@ -63,23 +64,19 @@ protected void onCreate(Bundle savedInstanceState) {
 
         mDataSet = new ArrayList<String>(Arrays.asList(adapterData));
         mAdapter = new RecyclerViewAdapter(this, mDataSet);
+
         ((RecyclerViewAdapter) mAdapter).setMode(Attributes.Mode.Single);
         recyclerView.setAdapter(mAdapter);
 
         /* Listeners */
-        ItemClickSupport itemClick = ItemClickSupport.addTo(recyclerView);
-        itemClick.setOnItemClickListener(onItemClickListener);
-        recyclerView.setOnTouchListener(new View.OnTouchListener() {
+        recyclerView.addOnItemTouchListener(new RecyclerItemClickListener(this, new RecyclerItemClickListener.OnItemClickListener() {
             @Override
-            public boolean onTouch(View v, MotionEvent event) {
-                Log.e("ListView", "OnTouch");
-                return false;
+            public void onItemClick(View view, int position) {
+                Toast.makeText(mContext, "Clicked:" + position, Toast.LENGTH_SHORT).show();
             }
-        });
+        }));
         recyclerView.setOnScrollListener(onScrollListener);
 
-        // TODO: Item Long Click is firing for every touch.
-//        itemClick.setOnItemLongClickListener(onItemLongClickListener);
 
         // TODO: Item Selection Support for RecyclerView
 //        recyclerView.setOnItemSelectedListener(new AdapterView.OnItemSelectedListener() {
@@ -96,27 +93,6 @@ public boolean onTouch(View v, MotionEvent event) {
 
     }
 
-    /**
-     * Substitute for our onItemClick listener for RecyclerView
-     */
-    ItemClickSupport.OnItemClickListener onItemClickListener = new ItemClickSupport.OnItemClickListener() {
-        @Override
-        public void onItemClick(RecyclerView parent, View child, int position, long id) {
-            Toast.makeText(mContext, "Clicked:" + position, Toast.LENGTH_SHORT).show();
-        }
-    };
-
-    /**
-     * Substitute for our onItemLongClick listener for RecyclerView
-     */
-    ItemClickSupport.OnItemLongClickListener onItemLongClickListener = new ItemClickSupport.OnItemLongClickListener() {
-        @Override
-        public boolean onItemLongClick(RecyclerView recyclerView, View view, int position, long id) {
-            Toast.makeText(mContext, "OnItemLongClickListener:" + position, Toast.LENGTH_SHORT).show();
-            return false;
-        }
-    };
-
     /**
      * Substitute for our onScrollListener for RecyclerView
      */

demo/src/main/java/com/daimajia/swipedemo/adapter/RecyclerViewAdapter.java

@@ -19,7 +19,7 @@
 
 public class RecyclerViewAdapter extends RecyclerSwipeAdapter {
 
-    public static class SimpleViewHolder extends ViewHolder {
+    public static class SimpleViewHolder extends ViewHolder{
         SwipeLayout swipeLayout;
         TextView textViewPos;
         TextView textViewData;
@@ -50,7 +50,7 @@ public ViewHolder createRecyclerViewHolder(ViewGroup parent, int viewType) {
     }
 
     @Override
-    public void bindRecyclerViewHolder(ViewHolder viewHolder, int position) {
+    public void bindRecyclerViewHolder(ViewHolder viewHolder, final int position) {
         SimpleViewHolder simpleViewHolder = (SimpleViewHolder) viewHolder;
         String item = mDataset.get(position);
         simpleViewHolder.swipeLayout.setShowMode(SwipeLayout.ShowMode.LayDown);

demo/src/main/java/com/daimajia/swipedemo/adapter/util/RecyclerItemClickListener.java

@@ -0,0 +1,40 @@
+package com.daimajia.swipedemo.adapter.util;
+
+import android.content.Context;
+import android.support.v7.widget.RecyclerView;
+import android.view.GestureDetector;
+import android.view.MotionEvent;
+import android.view.View;
+
+public class RecyclerItemClickListener implements RecyclerView.OnItemTouchListener {
+    private OnItemClickListener mListener;
+
+    public interface OnItemClickListener {
+        public void onItemClick(View view, int position);
+    }
+
+    GestureDetector mGestureDetector;
+
+    public RecyclerItemClickListener(Context context, OnItemClickListener listener) {
+        mListener = listener;
+        mGestureDetector = new GestureDetector(context, new GestureDetector.SimpleOnGestureListener() {
+            @Override
+            public boolean onSingleTapUp(MotionEvent e) {
+                return true;
+            }
+        });
+    }
+
+    @Override
+    public boolean onInterceptTouchEvent(RecyclerView view, MotionEvent e) {
+        View childView = view.findChildViewUnder(e.getX(), e.getY());
+        if (childView != null && mListener != null && mGestureDetector.onTouchEvent(e)) {
+            mListener.onItemClick(childView, view.getChildPosition(childView));
+        }
+        return false;
+    }
+
+    @Override
+    public void onTouchEvent(RecyclerView view, MotionEvent motionEvent) {
+    }
+}