Handle Button click inside a row in RecyclerView

Question

I am using following code for handling row clicks   source   static class RecyclerTouchListener implements RecyclerView OnItemTouchListener        private GestureDetector gestureDetector      private ClickListener clickListener       public RecyclerTouchListener Context context  final RecyclerView recyclerView  final ClickListener clickListener            this clickListener   clickListener          gestureDetector   new GestureDetector context  new GestureDetector SimpleOnGestureListener                  Override             public boolean onSingleTapUp MotionEvent e                    return true                              Override             public void onLongPress MotionEvent e                    View child   recyclerView findChildViewUnder e getX    e getY                     if  child    null  amp  amp  clickListener    null                        clickListener onLongClick child  recyclerView getChildPosition child                                                            Override     public boolean onInterceptTouchEvent RecyclerView rv  MotionEvent e             View child   rv findChildViewUnder e getX    e getY             if  child    null  amp  amp  clickListener    null  amp  amp  gestureDetector onTouchEvent e                 clickListener onClick child  rv getChildPosition child                      return false              Override     public void onTouchEvent RecyclerView rv  MotionEvent e              This works however  if I want to have say a delete button on each row  I am not sure to how to implement that with this   I attached OnClick listener to delete button which works  deletes the row  but it also fires the onclick on full row   Can anybody help me in how to avoid full row click if a single button is clicked   Thanks

User · Answer

Just put an override method named getItemId  Get it by right click generate override methods getItemId Put this method in the Adapter class

User · Answer

I find that typically    I need to use multiple listeners because I have several buttons  I want my logic to be in the activity and not the adapter or viewholder    So  mark-keen s answer works well but having an interface provides more flexibility   public static class MyViewHolder extends RecyclerView ViewHolder        public ImageView iconImageView      public TextView iconTextView       public MyViewHolder final View itemView            super itemView            iconImageView    ImageView  itemView findViewById R id myRecyclerImageView           iconTextView    TextView  itemView findViewById R id myRecyclerTextView            iconTextView setOnClickListener new View OnClickListener                  Override             public void onClick View v                    onClickListener iconTextViewOnClick v  getAdapterPosition                                       iconImageView setOnClickListener new View OnClickListener                  Override             public void onClick View v                    onClickListener iconImageViewOnClick v  getAdapterPosition                                         Where onClickListener is defined in your adapter   public MyAdapterListener onClickListener   public interface MyAdapterListener        void iconTextViewOnClick View v  int position       void iconImageViewOnClick View v  int position       And probably set through your constructor   public MyAdapter ArrayList lt MyListItems gt  newRows  MyAdapterListener listener         rows   newRows      onClickListener   listener      Then you can handle the events in your Activity or wherever your RecyclerView is being used   mAdapter   new MyAdapter mRows  new MyAdapter MyAdapterListener                          Override                     public void iconTextViewOnClick View v  int position                            Log d TAG   iconTextViewOnClick at position   position                                               Override                     public void iconImageViewOnClick View v  int position                            Log d TAG   iconImageViewOnClick at position   position                                             mRecycler setAdapter mAdapter

User · Answer

You can check if you have any similar entries first  if you get a collection with size 0  start a new query to save   OR  more professional and faster way  create a cloud trigger  before save   check out this answer https   stackoverflow com a 35194514 1388852

User · Answer

this is how I handle multiple onClick events inside a recyclerView  Edit   Updated to include callbacks  as mentioned in other comments    I have used a WeakReference in the ViewHolder to eliminate a potential memory leak  Define interface   public interface ClickListener        void onPositionClicked int position            void onLongClicked int position      Then the Adapter   public class MyAdapter extends RecyclerView Adapter lt MyAdapter MyViewHolder gt             private final ClickListener listener      private final List lt MyItems gt  itemsList       public MyAdapter List lt MyItems gt  itemsList  ClickListener listener            this listener   listener          this itemsList   itemsList              Override public MyViewHolder onCreateViewHolder ViewGroup parent  int viewType            return new MyViewHolder LayoutInflater from parent getContext    inflate R layout my row layout   parent  false   listener               Override public void onBindViewHolder MyViewHolder holder  int position               bind layout and data etc               Override public int getItemCount             return itemsList size               public static class MyViewHolder extends RecyclerView ViewHolder implements View OnClickListener  View OnLongClickListener            private ImageView iconImageView          private TextView iconTextView          private WeakReference lt ClickListener gt  listenerRef           public MyViewHolder final View itemView  ClickListener listener                super itemView                listenerRef   new WeakReference lt  gt  listener               iconImageView    ImageView  itemView findViewById R id myRecyclerImageView               iconTextView    TextView  itemView findViewById R id myRecyclerTextView                itemView setOnClickListener this               iconTextView setOnClickListener this               iconImageView setOnLongClickListener this                         onClick Listener for view          Override         public void onClick View v                 if  v getId      iconTextView getId                      Toast makeText v getContext     quot ITEM PRESSED    quot    String valueOf getAdapterPosition     Toast LENGTH SHORT  show                  else                   Toast makeText v getContext     quot ROW PRESSED    quot    String valueOf getAdapterPosition     Toast LENGTH SHORT  show                                           listenerRef get   onPositionClicked getAdapterPosition                           onLongClickListener for view          Override         public boolean onLongClick View v                 final AlertDialog Builder builder   new AlertDialog Builder v getContext                 builder setTitle  quot Hello Dialog quot                        setMessage  quot LONG CLICK DIALOG WINDOW FOR ICON  quot    String valueOf getAdapterPosition                          setPositiveButton  quot OK quot   new DialogInterface OnClickListener                              Override                         public void onClick DialogInterface dialog  int which                                                                    builder create   show                listenerRef get   onLongClicked getAdapterPosition                 return true                     Then in your activity fragment - whatever you can implement   Clicklistener - or anonymous class if you wish like so   MyAdapter adapter   new MyAdapter myItems  new ClickListener                  Override public void onPositionClicked int position                       callback performed on click                             Override public void onLongClicked int position                       callback performed on click                            To get which item was clicked you match the view id i e  v getId      whateverItem getId   Hope this approach helps

User · Answer

Just wanted to add another solution if you already have a recycler touch listener and want to handle all of the touch events in it rather than dealing with the button touch event separately in the view holder  The key thing this adapted version of the class does is return the button view in the onItemClick   callback when it s tapped  as opposed to the item container  You can then test for the view being a button  and carry out a different action  Note  long tapping on the button is interpreted as a long tap on the whole row still   public class RecyclerItemClickListener implements RecyclerView OnItemTouchListener       public static interface OnItemClickListener               public void onItemClick View view  int position           public void onItemLongClick View view  int position              private OnItemClickListener mListener      private GestureDetector mGestureDetector       public RecyclerItemClickListener Context context  final RecyclerView recyclerView  OnItemClickListener listener                mListener   listener           mGestureDetector   new GestureDetector context  new GestureDetector SimpleOnGestureListener                          Override             public boolean onSingleTapUp MotionEvent e                                   Important  x and y are translated coordinates here                 final ViewGroup childViewGroup    ViewGroup  recyclerView findChildViewUnder e getX    e getY                      if  childViewGroup    null  amp  amp  mListener    null                        final List lt View gt  viewHierarchy   new ArrayList lt View gt                            Important  x and y are raw screen coordinates here                     getViewHierarchyUnderChild childViewGroup  e getRawX    e getRawY    viewHierarchy                        View touchedView   childViewGroup                      if  viewHierarchy size    gt  0                            touchedView   viewHierarchy get 0                                             mListener onItemClick touchedView  recyclerView getChildPosition childViewGroup                        return true                                     return false                              Override             public void onLongPress MotionEvent e                                View childView   recyclerView findChildViewUnder e getX    e getY                      if childView    null  amp  amp  mListener    null                                        mListener onItemLongClick childView  recyclerView getChildPosition childView                                                           public void getViewHierarchyUnderChild ViewGroup root  float x  float y  List lt View gt  viewHierarchy            int   location   new int 2           final int childCount   root getChildCount             for  int i   0  i  lt  childCount    i                final View child   root getChildAt i               child getLocationOnScreen location               final int childLeft   location 0   childRight   childLeft   child getWidth                final int childTop   location 1   childBottom   childTop   child getHeight                 if  child isShown    amp  amp  x  gt   childLeft  amp  amp  x  lt   childRight  amp  amp  y  gt   childTop  amp  amp  y  lt   childBottom                    viewHierarchy add 0  child                             if  child instanceof ViewGroup                    getViewHierarchyUnderChild  ViewGroup  child  x  y  viewHierarchy                                       Override     public boolean onInterceptTouchEvent RecyclerView view  MotionEvent e                mGestureDetector onTouchEvent e            return false              Override     public void onTouchEvent RecyclerView view  MotionEvent motionEvent          Override     public void onRequestDisallowInterceptTouchEvent boolean disallowIntercept               Then using it from activity   fragment   recyclerView addOnItemTouchListener createItemClickListener recyclerView         public RecyclerItemClickListener createItemClickListener final RecyclerView recyclerView            return new RecyclerItemClickListener  context  recyclerView  new RecyclerItemClickListener OnItemClickListener                  Override             public void onItemClick View view  int position                    if  view instanceof AppCompatButton                               tapped on the button  so go do something                   else                              tapped on the item container  row   so do something different                                               Override             public void onItemLongClick View view  int position

User · Answer

You need to return true inside onInterceptTouchEvent   when you handle click event

User · Answer

I wanted a solution that did not create any extra objects  ie listeners  that would have to be garbage collected later  and did not require nesting a view holder inside an adapter class   In the ViewHolder class  private static class MyViewHolder extends RecyclerView ViewHolder implements View OnClickListener            private final TextView        declare the fields in your view         private ClickHandler ClickHandler           public MyHolder final View itemView                super itemView               nameField    TextView  itemView findViewById R id name                 find other fields here                Button myButton    Button  itemView findViewById R id my button               myButton setOnClickListener this                                  Override         public void onClick final View view                if  clickHandler    null                    clickHandler onMyButtonClicked getAdapterPosition                               Points to note  the ClickHandler interface is defined  but not initialized here  so there is no assumption in the onClick method that it was ever initialized   The ClickHandler interface looks like this   private interface ClickHandler       void onMyButtonClicked final int position        In the adapter  set an instance of  ClickHandler  in the constructor  and override onBindViewHolder  to initialize  clickHandler  on the view holder   private class MyAdapter extends           private final ClickHandler clickHandler       public MyAdapter final ClickHandler clickHandler            super               this clickHandler   clickHandler              Override     public void onBindViewHolder final MyViewHolder viewHolder  final int position            super onBindViewHolder viewHolder  position           viewHolder clickHandler   this clickHandler          Note  I know that viewHolder clickHandler is potentially getting set multiple times with the exact same value  but this is cheaper than checking for null and branching  and there is no memory cost  just an extra instruction   Finally  when you create the adapter  you are forced to pass a ClickHandlerinstance to the constructor  as so   adapter   new MyAdapter new ClickHandler          Override     public void onMyButtonClicked final int position            final MyModel model   adapter getItem position             do something with the model where the button was clicked             Note that adapter is a member variable here  not a local variable

[android] Handle Button click inside a row in RecyclerView

Examples related to android

Examples related to android-recyclerview