From 1271d6fdec30108d9328045ce28fa68a8ff77384 Mon Sep 17 00:00:00 2001 From: Shangeeth Sivan <9254310+shangeethsivan@users.noreply.github.com> Date: Mon, 21 Oct 2019 22:27:29 +0530 Subject: [PATCH] [fenix] Closes https://github.com/mozilla-mobile/fenix/issues/5465: Added OnCheckListener to all checkboxes ,when there are no items checked delete button will be disabled (https://github.com/mozilla-mobile/fenix/pull/6098) --- .../settings/deletebrowsingdata/DeleteBrowsingDataFragment.kt | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/mobile/android/fenix/app/src/main/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteBrowsingDataFragment.kt b/mobile/android/fenix/app/src/main/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteBrowsingDataFragment.kt index 02fa80081e31..f4a133e024b6 100644 --- a/mobile/android/fenix/app/src/main/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteBrowsingDataFragment.kt +++ b/mobile/android/fenix/app/src/main/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteBrowsingDataFragment.kt @@ -51,6 +51,10 @@ class DeleteBrowsingDataFragment : Fragment(R.layout.fragment_delete_browsing_da }) } + getCheckboxes().forEach { + it.onCheckListener = { _ -> updateDeleteButton() } + } + getCheckboxes().forEach { it.isChecked = true } view.delete_data?.setOnClickListener { @@ -58,6 +62,13 @@ class DeleteBrowsingDataFragment : Fragment(R.layout.fragment_delete_browsing_da } } + private fun updateDeleteButton() { + val enabled = getCheckboxes().any { it.isChecked } + + view?.delete_data?.isEnabled = enabled + view?.delete_data?.alpha = if (enabled) ENABLED_ALPHA else DISABLED_ALPHA + } + override fun onResume() { super.onResume() (activity as AppCompatActivity).apply { -- 2.11.4.GIT