diff --git a/uhabits-android/src/main/java/org/isoron/uhabits/activities/common/dialogs/CheckmarkDialog.kt b/uhabits-android/src/main/java/org/isoron/uhabits/activities/common/dialogs/CheckmarkDialog.kt index 16c18ce07..c80759bae 100644 --- a/uhabits-android/src/main/java/org/isoron/uhabits/activities/common/dialogs/CheckmarkDialog.kt +++ b/uhabits-android/src/main/java/org/isoron/uhabits/activities/common/dialogs/CheckmarkDialog.kt @@ -79,29 +79,25 @@ class CheckmarkDialog private fun setUpButtons(value: Int, color: Int) { val sres = StyledResources(context) val mediumContrastColor = sres.getColor(R.attr.contrast60) - setButtonAttrs(binding.yesBtn, color) setButtonAttrs(binding.noBtn, mediumContrastColor) - - if (preferences.isSkipEnabled) { - setButtonAttrs(binding.skippedBtn, color) - if (value == SKIP) binding.skippedBtn.performClick() - } - - if (preferences.areQuestionMarksEnabled) { - setButtonAttrs(binding.questionBtn, mediumContrastColor) - if (value == UNKNOWN) binding.questionBtn.performClick() - } - + setButtonAttrs(binding.skippedBtn, color, visible = preferences.isSkipEnabled) + setButtonAttrs(binding.questionBtn, mediumContrastColor, visible = preferences.areQuestionMarksEnabled) when (value) { - YES_MANUAL, YES_AUTO -> binding.yesBtn.performClick() - NO -> binding.noBtn.performClick() + UNKNOWN -> if (preferences.areQuestionMarksEnabled) { + binding.questionBtn.performClick() + } else { + binding.noBtn.performClick() + } + SKIP -> binding.skippedBtn.performClick() + YES_MANUAL -> binding.yesBtn.performClick() + YES_AUTO, NO -> binding.noBtn.performClick() } } - private fun setButtonAttrs(button: Button, color: Int) { + private fun setButtonAttrs(button: Button, color: Int, visible: Boolean = true) { button.apply { - visibility = View.VISIBLE + visibility = if (visible) View.VISIBLE else View.GONE typeface = fontAwesome setTextColor(color) setOnClickListener(this@CheckmarkDialog)