Set tick color per color card

Added fix for white tick on white color issue

#12

colorpicker/src/main/kotlin/com/github/dhaval2404/colorpicker/MaterialColorPickerBottomSheet.kt

@@ -33,6 +33,7 @@ class MaterialColorPickerBottomSheet : BottomSheetDialogFragment() {
     private var colorShape: ColorShape = ColorShape.CIRCLE
     private var colorSwatch: ColorSwatch = ColorSwatch._300
     private var colors: List<String>? = null
+    private var isTickColorPerCard: Boolean = false
 
     companion object {
 
@@ -44,6 +45,7 @@ class MaterialColorPickerBottomSheet : BottomSheetDialogFragment() {
         private const val EXTRA_COLOR_SHAPE = "extra.color_shape"
         private const val EXTRA_COLOR_SWATCH = "extra.color_swatch"
         private const val EXTRA_COLORS = "extra.colors"
+        private const val EXTRA_IS_TICK_COLOR_PER_CARD = "extra.is_tick_color_per_card"
 
         fun getInstance(dialog: MaterialColorPickerDialog): MaterialColorPickerBottomSheet {
             val bundle = Bundle().apply {
@@ -54,6 +56,7 @@ class MaterialColorPickerBottomSheet : BottomSheetDialogFragment() {
                 putString(EXTRA_DEFAULT_COLOR, dialog.defaultColor)
                 putParcelable(EXTRA_COLOR_SWATCH, dialog.colorSwatch)
                 putParcelable(EXTRA_COLOR_SHAPE, dialog.colorShape)
+                putBoolean(EXTRA_IS_TICK_COLOR_PER_CARD, dialog.isTickColorPerCard)
 
                 var list: ArrayList<String>? = null
                 if (dialog.colors != null) {
@@ -99,6 +102,7 @@ class MaterialColorPickerBottomSheet : BottomSheetDialogFragment() {
             colorShape = it.getParcelable(EXTRA_COLOR_SHAPE)!!
 
             colors = it.getStringArrayList(EXTRA_COLORS)
+            isTickColorPerCard = it.getBoolean(EXTRA_IS_TICK_COLOR_PER_CARD)
         }
 
         title?.let { titleTxt.text = it }
@@ -108,6 +112,7 @@ class MaterialColorPickerBottomSheet : BottomSheetDialogFragment() {
         val colorList = colors ?: ColorUtil.getColors(context!!, colorSwatch.value)
         val adapter = MaterialColorPickerAdapter(colorList)
         adapter.setColorShape(colorShape)
+        adapter.setTickColorPerCard(isTickColorPerCard)
         if (!defaultColor.isNullOrBlank()) {
             adapter.setDefaultColor(defaultColor!!)
         }

colorpicker/src/main/kotlin/com/github/dhaval2404/colorpicker/MaterialColorPickerDialog.kt

@@ -34,7 +34,8 @@ class MaterialColorPickerDialog private constructor(
     val defaultColor: String?,
     val colorSwatch: ColorSwatch,
     var colorShape: ColorShape,
-    val colors: List<String>? = null
+    val colors: List<String>? = null,
+    var isTickColorPerCard: Boolean = false
 ) {
 
     class Builder(val context: Context) {
@@ -48,6 +49,7 @@ class MaterialColorPickerDialog private constructor(
         private var colorSwatch: ColorSwatch = ColorSwatch._300
         private var colorShape: ColorShape = ColorShape.CIRCLE
         private var colors: List<String>? = null
+        private var isTickColorPerCard: Boolean = false
 
         /**
          * Set Dialog Title
@@ -233,6 +235,30 @@ class MaterialColorPickerDialog private constructor(
             return this
         }
 
+
+        /**
+         * Set tick icon color, Default will be false
+         *
+         * If false,
+         *     First the majority of color(dark/light) will be calculated
+         *     If dark color count > light color count
+         *          tick color will be WHITE
+         *     else
+         *          tick color will be BLACK
+         *     Here, Tick color will be same card,
+         *     Which might create issue with black and white color in list
+         *
+         * If true,
+         *      based on the each color(dark/light) the card tick color will be decided
+         *      Here, Tick color will be different for each card
+         *
+         * @param tickColorPerCard Boolean
+         */
+        fun setTickColorPerCard(tickColorPerCard: Boolean): Builder {
+            this.isTickColorPerCard = tickColorPerCard
+            return this
+        }
+
         /**
          * Creates an {@link MaterialColorPickerDialog} with the arguments supplied to this
          * builder.
@@ -252,7 +278,8 @@ class MaterialColorPickerDialog private constructor(
                 defaultColor = defaultColor,
                 colorShape = colorShape,
                 colorSwatch = colorSwatch,
-                colors = colors
+                colors = colors,
+                isTickColorPerCard = isTickColorPerCard
             )
         }
 
@@ -299,6 +326,7 @@ class MaterialColorPickerDialog private constructor(
         val colorList = colors ?: ColorUtil.getColors(context, colorSwatch.value)
         val adapter = MaterialColorPickerAdapter(colorList)
         adapter.setColorShape(colorShape)
+        adapter.setTickColorPerCard(isTickColorPerCard)
         if (!defaultColor.isNullOrBlank()) {
             adapter.setDefaultColor(defaultColor)
         }

colorpicker/src/main/kotlin/com/github/dhaval2404/colorpicker/adapter/MaterialColorPickerAdapter.kt

@@ -3,6 +3,7 @@ package com.github.dhaval2404.colorpicker.adapter
 import android.graphics.Color
 import android.view.View
 import android.view.ViewGroup
+import android.widget.Chronometer
 import androidx.recyclerview.widget.RecyclerView
 import com.github.dhaval2404.colorpicker.model.ColorShape
 import com.github.dhaval2404.colorpicker.util.ColorUtil
@@ -22,6 +23,7 @@ class MaterialColorPickerAdapter(private val colors: List<String>) :
     private var isDarkColor = false
     private var color = ""
     private var colorShape = ColorShape.CIRCLE
+    private var isTickColorPerCard = false
 
     init {
         val darkColors = colors.count { ColorUtil.isDarkColor(it) }
@@ -36,6 +38,10 @@ class MaterialColorPickerAdapter(private val colors: List<String>) :
         this.color = color
     }
 
+    fun setTickColorPerCard(tickColorPerCard: Boolean) {
+        this.isTickColorPerCard = tickColorPerCard
+    }
+
     fun getSelectedColor() = color
 
     fun getItem(position: Int) = colors[position]
@@ -80,7 +86,13 @@ class MaterialColorPickerAdapter(private val colors: List<String>) :
 
             val isChecked = color == this@MaterialColorPickerAdapter.color
             checkIcon.setVisibility(isChecked)
-            checkIcon.setColorFilter(if (isDarkColor) Color.WHITE else Color.BLACK)
+
+            var darkColor = isDarkColor
+            if (isTickColorPerCard) {
+                darkColor = ColorUtil.isDarkColor(color)
+            }
+
+            checkIcon.setColorFilter(if (darkColor) Color.WHITE else Color.BLACK)
         }
 
     }