Android：onClick 使用 TextView 和 Button 的 LinearLayout

Question

I have a Fragment that uses the following XML layout:

<?xml version="1.0" encoding="utf-8"?>
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:background="@drawable/card"
    android:clickable="true"
    android:onClick="editActions"
    android:orientation="vertical" >

    <TextView
        android:id="@+id/title"
        style="@style/CardTitle"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:duplicateParentState="true"
        android:text="@string/title_workstation" />

    <Button
        android:id="@+id/factory_button_edit"
        style="?android:attr/buttonStyleSmall"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:duplicateParentState="true"
        android:text="@string/label_edit" />

</LinearLayout>

As you can see, I have an onClick parameter set on LinearLayout . Now on the TextView this one is triggered correctly, and on all empty area. Just on the Button it doesn't invoke the onClick method that I set.

Is this normal? What do I have to do so that the onClick method is invoked everywhere on the LinearLayout ?

Answer 1

This is what you need in your LinearLayout :

android:onClick="editActions" 
android:clickable="true"

and also in your class which call that xml some method:

public void editActions(View view){

           //something TODO

    }

Answer 2

add onClick Action for your button dirrectly in code like this:

Button btn = findViewById(R.id.factory_button_edit);
btn.setOnClickListener(button_click);

      OnClickListener button_click = new OnClickListener() {

        @Override
        public void onClick(View v) {
            editActions;
        }
    };

or xml add

android:onClick="editActions"

to your button

Answer 3

I think you should use a TouchDelegate :

Android provides the TouchDelegate class to make it possible for a parent to extend the touchable area of a child view beyond the child's bounds. This is useful when the child has to be small, but should have a larger touch region. You can also use this approach to shrink the child's touch region if need be.

Answer 4

To make your layout clickable with its children you need add this option for every child :

 android:clickable="false"

Then click handling will go up to parent .

Answer 5

try to do that

<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:background="@drawable/card"
    android:clickable="true"
    android:onClick="editActions"
    android:orientation="vertical" >

    <TextView
        android:id="@+id/title"
        style="@style/CardTitle"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="@string/title_workstation" />

    <Button
        android:id="@+id/factory_button_edit"
        style="?android:attr/buttonStyleSmall"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:onClick="clickMe"
        android:text="@string/label_edit" />

</LinearLayout>

and on your Activity or fragement

    public void editActions(View v) {
        Toast.makeText(this, "Layout", Toast.LENGTH_SHORT).show();
    }

    public void clickMe(View v) {
        Toast.makeText(this, "clickMe", Toast.LENGTH_SHORT).show();
    }

Answer 6

Well, if this a bar with some children and you want to have only this bar clickable with children area - try add this option for every child in parent:

android:clickable="false"

and of course add setOnClickListener for main layout, example on my blog

Example:

<LinearLayout  
      android:id="@+id/parent"  
      android:layout_width="match_parent"  
      android:layout_height="wrap_content"  
      android:orientation="horizontal">  
      <ImageButton  
           android:clickable="false"  
           android:layout_width="40dp"  
           android:layout_height="40dp"  
           android:text="Cinema"  
           android:textSize="13sp"  
           android:layout_marginRight="10dp" />  
      <Button  
           android:clickable="false"  
           android:layout_width="match_parent"  
           android:layout_height="40dp"  
           android:text="Check in"  
           android:textSize="13sp"/>  
 </LinearLayout> 

Answer 7

我想你应该为你的 Button 删除这个参数：

android:duplicateParentState="true"

Answer 8

View.OnClickListener myViewHandler = new View.OnClickListener() {
        public void onClick(View v) {
                RelativeLayout pressedRelViewx = (RelativeLayout) v;
                TextView pressedTextView = (TextView) pressedRelViewx.getChildAt(0);
                String text = pressedTextView.getText().toString();
                Toast.makeText(getApplicationContext() , "My Text "+ text , Toast.LENGTH_LONG).show() ;
        }
    };


Relative_Layout = (RelativeLayout)findViewById(R.id.my_container);
Relative_Layout.setOnClickListener(myViewHandler);

Answer 9

ok this may be late , but it will help have others, who are looking for

how to set onClick Listener for layouts [android].

LinearLayout mLinearLayout = findViewById(R.id.your_linear_layout_name);

mLinearLayout.setonClickListener(new View.onClickListener(){
   @Overide
   public void onClick(View v){
     //do want to do here

    }

});

Note:

you can not perform onClick Listener on button when button is inside the linear layout which is already listening to onClick , and if do that first parent(Linear Layout) is get clicked. It is best practice to do not preform nested onClick Listeners.

Answer 10

To eliminate copy-paste code, you can set a OnClickListener to all of the chidlren View 's with code:

private void setClickListenerRecursively(View view, View.OnClickListener listener){
    view.setOnClickListener(listener);

    if(view instanceof ViewGroup){
        ViewGroup viewGroup = (ViewGroup) view;
        for(int i = 0; i < viewGroup.getChildCount(); i++){
            View child = viewGroup.getChildAt(i);
            setClickListenerRecursively(child, listener);
        }
    }
}