Android custom view with button not clickable

Question

I've created a custom RelativeLayout view which I inflate with merge tags.
In this custom view I have a button which I want it to do something.
I've tried many things but the button simply refuses to be clicked.

The strange part is, I can find the views and change their visibility just fine.

Is it possible to click a button this way, or should it be done in a different way?

The things I've tried:

• Anonymous innerclass for onClickListener
• XML attribute for onClick
• View onClick with onClickListener(this)
• Check if it's clickable with code (it returns true)
• Added clickable(true) to both XML and code.
• Private context from constructor instead of getContext()
• Moved logic from init() to onFinishInflate()
• Used the view from inflater to find views
• Switch from inflate() to LayoutInflater

Inflated custom view:

<?xml version="1.0" encoding="utf-8"?>
<merge
    xmlns:android="http://schemas.android.com/apk/res/android"
    android:layout_width="match_parent"
    android:layout_height="match_parent">

    <TextView
        android:id="@+id/internet_view_text"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_centerInParent="true"
        android:text="@string/internet_offline"/>

    <custom.IconView
        android:id="@+id/internet_view_icon"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_centerInParent="true"
        android:text="ICON"
        android:visibility="invisible"
        android:layout_below="@+id/internet_view_text"/>

    <Button
        android:id="@+id/internet_view_button"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_centerInParent="true"
        android:text="BUTTON"
        android:clickable="true"
        android:layout_below="@+id/internet_view_text"/>
</merge>

Relevant part of the parent view:

<custom.NoInternetView
    android:id="@+id/webview_no_content"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:visibility="invisible"
    app:automaticReconnect="false"/>

The class file:

public class NoInternetView extends RelativeLayout implements View.OnClickListener {

    private static final String TAG = NoInternetView.class.getSimpleName();

    private static ConnectivityChangeListener listener;
    private static boolean automaticReconnect;
    private Context mContext;
    private View view;

    private Button button;

    public NoInternetView(Context context, AttributeSet attrs, int defStyleAttr) {
        super(context, attrs, defStyleAttr);
        mContext = context;
        TypedArray a = context.obtainStyledAttributes(attrs, R.styleable.NoInternetView, defStyleAttr, 0);
        automaticReconnect = a.getBoolean(R.styleable.NoInternetView_automaticReconnect, false);
        a.recycle();
        init();
    }

    public NoInternetView(Context context, AttributeSet attrs) {
        this(context, attrs, 0);
    }

    public NoInternetView(Context context) {
        this(context, null);
    }

    @Override
    protected void onFinishInflate() {
        Log.d(TAG, "onFinishInflate");
        super.onFinishInflate();

        button = (Button) view.findViewById(R.id.internet_view_button);
        button.setOnClickListener(new OnClickListener() {
            @Override
            public void onClick(View v) {
                Log.d(TAG, "Clicked on some Button");
                if (listener != null && NetworkHelper.hasAccess(getContext())) {
                    listener.connected();
                }
            }
        });
        button.setClickable(true);

        boolean clicked = button.callOnClick();
        Log.d(TAG, "clicked: "+clicked);

        TextView text = (TextView) view.findViewById(R.id.internet_view_icon);
        text.setOnClickListener(new OnClickListener() {
            @Override
            public void onClick(View v) {
                Log.d(TAG, "Clicked on some TextView");
                if (listener != null && NetworkHelper.hasAccess(getContext())) {
                    listener.connected();
                }
            }
        });

        //check if the attribute 'automaticReconnect' is set to true
        //if so, show an icon instead of a button
        if (automaticReconnect){
            button.setVisibility(INVISIBLE);
            view.findViewById(R.id.internet_view_icon).setVisibility(VISIBLE);
        }

    }

    public void init(){
        Log.d(TAG, "init");
        view = LayoutInflater.from(mContext).inflate(R.layout.no_internet_view, this, false);
    }

    @Override
    public boolean dispatchKeyEvent(KeyEvent event) {
        Log.d(TAG, "something happened?");
        return super.dispatchKeyEvent(event);
    }

    @Override
    public void onClick(View v) {
        Log.d(TAG, "Clicked on Button(TextView)");
        if (listener != null && NetworkHelper.hasAccess(getContext())){
            listener.connected();
        }
    }

Answer 1

So I got it to work. Here are the things I did to make it work.

1. Replace the merge tag with RelativeLayout, because you are inflating the layout to show inside a view. More info here: Android xml merge layout error on inflate

2. You have set android:visibility="invisible", change that to android:visibility="visible" (This is just to get the view visible, you can later change it programatically)

3. Add the below line to the init() method of your NoInternetView. You are inflating the layout with all the views, but only when you add this view, you will be able to see it.

   public void init(){
          Log.d(TAG, "init");
          view = LayoutInflater.from(mContext).inflate(R.layout.no_internet_view, this, false);
          this.addView(view);}
   

Once you are done with this, the button will be visible and clicking on the button will be clickable. You can either use anonymous onClickListener or make the view implement the onClickListener and override the onClick() method. Anything works.