Don't call setOnClickListener for an AdapterView. You probably want setOnItemClickListener instead about dialogplus HOT 13 CLOSED

In my case I try to add onClickListener for spinner and I've got the same problem. I've crated workaround with touch listener:

someSpinner.setOnTouchListener { _, event -> onTouchSomeSpinner(event)}

fun onTouchSomeSpinner(event: MotionEvent): Boolean {
        if(event.action == MotionEvent.ACTION_UP) {
            view.hideKeyBoard()
            view.analyticsEvent()
            ...
        }
        return false
    }

from dialogplus.

version 1.10

from dialogplus.

can you show the whole implementation of dialogplus?

from dialogplus.

           WheelView monthWheel = (WheelView)viewGroup.findViewById(R.id.month_wheel);

//
//
//
monthWheel.setScrollCycle(true);
NumberAdapter monthAdapter = new NumberAdapter(monthArray);
monthWheel.setAdapter(monthAdapter);
monthWheel.setOnItemSelectedListener(mWheelViewListener);
monthWheel.setUnselectedAlpha(0.5f);
monthWheel.setOnItemClickListener(new TosAdapterView.OnItemClickListener() {
@OverRide
public void onItemClick(TosAdapterView<?> tosAdapterView, View view, int i, long l) {

                }
            });

            DialogPlus dialogPlus = DialogPlus.newDialog(getActivity())
                    .setContentHolder(new ViewHolder(viewGroup))
                    .setGravity(Gravity.BOTTOM)
                    .setCancelable(true)
                    .setOnItemClickListener(new OnItemClickListener() {
                        @Override
                        public void onItemClick(DialogPlus dialogPlus, Object o, View view, int i) {

                        }
                    })
                    .create();
            dialogPlus.show();

from dialogplus.

When you use ViewHolder, you can't use OnItemClickListener, if you want to have click events of the individual views, use OnClickListener, or if you already handle it by yourself, just remove all click listeners from dialogplus.

from dialogplus.

yes I have removed all click listeners before. But it throw the exception too.

DialogPlus dialogPlus = DialogPlus.newDialog(getActivity())
                        .setContentHolder(new ViewHolder(viewGroup))
                        .setGravity(Gravity.BOTTOM)
                        .setCancelable(true)
                        .create();
                dialogPlus.show();

from dialogplus.

I think that, when I give a custom viewgroup, the dialogplus also set the click event all children. But when I set a spinner in it. spinner can not set click event. android throw a exception. Can I forbidden the click event?

from dialogplus.

Yes, it seems this is the exact reason and unfortunately there is no this option, but we should add this for these situations. You can only avoid this by creating spinner dynamically, dialogplus adds clicklistener only if the view has an id.

from dialogplus.

oh! may be I can remove the id and try find it by the tag or parent find chid?

from dialogplus.

That could also work but out of box, we already avoid to set clicklistener for adapterview. Need to check thorougly for this issue.

from dialogplus.

ok. I try to remove the id and get it work. Thanks.

from dialogplus.

cheers 👍

from dialogplus.

@wood23636 Can you show me your code,I can't solve it, Thanks.

from dialogplus.