tried android_clickable=”true” in your xml?

As the documentation suggest for:

RecyclerView.Adapter.html#onBindViewHolder

Called by RecyclerView to display the data at the specified position.
This method should update the contents of the itemView to reflect the
item at the given position.

And for the RecyclerView.Adapter.html#onCreateViewHolder

Called when RecyclerView needs a new RecyclerView.ViewHolder of the
given type to represent an item.

This new ViewHolder should be constructed with a new View that can
represent the items of the given type. You can either create a new
View manually or inflate it from an XML layout file.

As onCreateViewHolder is a method where we return the type of ViewHolder only while in onBindViewHolder we update or initialise the data to display.

It means no data is binded in onCreateViewHolder and binding anything in this method will have no effect.

I am researching on how to add an onClick event on my recyclerview properly,

currently I am using the interface inside my customAdapter

class CategoryAdapter(val categoryList : List<CategoryObject>, val context: Context, val mItemClickListener: MainInterface) : RecyclerView.Adapter<CategoryAdapter.ViewHolder>() {

interface MainInterface {
    fun onCategoryItemClick(categoryKey: Int)
}

override fun getItemCount(): Int {
    return categoryList.size
}

override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): ViewHolder {
    return ViewHolder(LayoutInflater.from(context).inflate(R.layout.listview_category, parent, false))
}

override fun onBindViewHolder(holder: ViewHolder, position: Int) {
    holder?.categoryName?.text = categoryList[position].categoryName
//        holder?.categoryName.setOnClickListener{
//            mItemClickListener.onCategoryItemClick(position)
//        }
//        holder?.categoryName?.setOnClickListener { listener(categoryList[position]) }
}

inner class ViewHolder (view: View) : RecyclerView.ViewHolder(view) {
    val categoryName = view.lv_txt_category_name

    init {
        view.setOnClickListener {
            mItemClickListener.onCategoryItemClick(adapterPosition)
        }
    }
}

}  

viewHolder’s onClick does not register on my activity ovveride function.

but putting the onClick inside onBindViewHolder works perfectly,

I’m not sure which is more efficient, if the onBindViewHolder onClick is the right answer, then I’ll stick with it, but if the viewHolder is the right one, why it does not work properly?

Thanks in advance!

Update
This is the stackoverflow post I’m using to research things

RecyclerView itemClickListener in Kotlin