优化标签点击事件

Author: 杨宏伟

README.md

@@ -12,7 +12,7 @@ allprojects {
 	}
 
 dependencies {
-            implementation 'com.github.LuckyCodeer:TagLayout:1.0.3'
+            implementation 'com.github.LuckyCodeer:TagLayout:1.0.4'
 	}
 ```
 

app/src/main/java/com/yhw/taglayout/sample/ListActivity.kt

@@ -5,6 +5,7 @@ import android.view.LayoutInflater
 import android.view.View
 import android.view.ViewGroup
 import android.widget.TextView
+import android.widget.Toast
 import androidx.appcompat.app.AppCompatActivity
 import androidx.recyclerview.widget.RecyclerView
 import androidx.recyclerview.widget.RecyclerView.Adapter
@@ -77,6 +78,15 @@ class ListActivity : AppCompatActivity() {
         override fun onBindViewHolder(holder: MyViewHolder, position: Int) {
             holder.textView.text = list[position].name
             holder.tagLayout.setAdapter(MyTagAdapter(list[position].tagList))
+            holder.itemView.setOnClickListener {
+                Toast.makeText(holder.itemView.context, "" + position, Toast.LENGTH_SHORT).show()
+            }
+            /*holder.tagLayout.onItemClickListener = object : TagLayout.OnItemClickListener {
+                override fun onItemClick(position: Int, view: View) {
+                    Toast.makeText(holder.itemView.context, "点击了标签$position", Toast.LENGTH_SHORT)
+                        .show()
+                }
+            }*/
         }
 
     }
@@ -93,6 +103,7 @@ class ListActivity : AppCompatActivity() {
         }
 
         override fun onBindView(itemView: View, position: Int) {
+            itemView.setBackgroundResource(R.drawable.tag_normal_bg)
             val textView: TextView = itemView.findViewById(R.id.tv_title)
             textView.text = dataList[position]
         }

app/src/main/res/layout/tag_item_layout.xml

@@ -3,8 +3,6 @@
     android:layout_width="wrap_content"
     android:layout_height="wrap_content"
     android:layout_margin="10dp"
-    android:clickable="true"
-    android:focusable="true"
     android:background="@drawable/tag_selector_bg"
     android:orientation="vertical">
 

library/build.gradle

@@ -9,8 +9,8 @@ android {
     defaultConfig {
         minSdkVersion 16
         targetSdkVersion 30
-        versionCode 103
-        versionName "1.0.3"
+        versionCode 104
+        versionName "1.0.4"
 
         testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
         consumerProguardFiles "consumer-rules.pro"

library/src/main/java/com/yhw/taglayout/TagLayout.kt

@@ -6,13 +6,15 @@ import android.graphics.Rect
 import android.util.AttributeSet
 import android.util.Log
 import android.view.View
+import android.view.View.OnLongClickListener
 import android.view.ViewGroup
 import kotlin.math.max
 
 private const val TAG = "TagLayout"
 
 /**
  * 标签布局
+ * @author yhw
  */
 class TagLayout : ViewGroup {
     private val mViewRectList = mutableListOf<Rect>()
@@ -82,8 +84,6 @@ class TagLayout : ViewGroup {
             }
 
             if (!mChildViewList.contains(childView)) {
-                childView.isClickable = true
-                childView.isFocusable = true
                 if (choiceMode == ChoiceMode.SingleChoice.choiceMode) {
                     if (i in 0 until childCount && i == defChoicePosition)
                         childView.isSelected = true
@@ -101,28 +101,34 @@ class TagLayout : ViewGroup {
                 )
                 mViewRectList.add(rect)
 
-                childView.setOnClickListener {
-                    changedCheckedItemView(i)
-                    if (onItemClickListener != null) {
-                        onItemClickListener?.onItemClick(i, it)
-                    }
-                    if (choiceMode == ChoiceMode.SingleChoice.choiceMode) {
-                        this.defChoicePosition = i
-                        if (onSingleCheckedChangeListener != null) {
-                            onSingleCheckedChangeListener?.onCheckedChanged(defChoicePosition)
+                if (onItemClickListener != null || onSingleCheckedChangeListener != null || onMultipleCheckedChangeListener != null) {
+                    childView.isClickable = true
+                    childView.isFocusable = true
+                    childView.setOnClickListener {
+                        changedCheckedItemView(i)
+                        if (onItemClickListener != null) {
+                            onItemClickListener?.onItemClick(i, it)
                         }
-                    } else if (choiceMode == ChoiceMode.MultipleChoice.choiceMode) {
-                        if (onMultipleCheckedChangeListener != null) {
-                            onMultipleCheckedChangeListener?.onCheckedChanged(getCheckedList())
+                        if (choiceMode == ChoiceMode.SingleChoice.choiceMode) {
+                            this.defChoicePosition = i
+                            if (onSingleCheckedChangeListener != null) {
+                                onSingleCheckedChangeListener?.onCheckedChanged(defChoicePosition)
+                            }
+                        } else if (choiceMode == ChoiceMode.MultipleChoice.choiceMode) {
+                            if (onMultipleCheckedChangeListener != null) {
+                                onMultipleCheckedChangeListener?.onCheckedChanged(getCheckedList())
+                            }
                         }
                     }
                 }
 
-                childView.setOnLongClickListener {
-                    if (onItemLongClickListener != null) {
-                        onItemLongClickListener?.onItemLongClick(i, it)
-                    }
-                    true
+                if (onItemLongClickListener != null) {
+                    childView.isClickable = true
+                    childView.isFocusable = true
+                    childView.setOnLongClickListener(OnLongClickListener { v ->
+                        onItemLongClickListener?.onItemLongClick(i, v)
+                        true
+                    })
                 }
             }