From cdb88ec87ebe293a12dcbf15438e974ac03890c6 Mon Sep 17 00:00:00 2001 From: Emily Kager Date: Thu, 13 Jun 2019 09:57:04 -0700 Subject: [PATCH] [fenix] For https://github.com/mozilla-mobile/fenix/issues/3433 - Add checkbox hide changes to change payload --- .../fenix/collections/CollectionCreationTabListAdapter.kt | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/app/src/main/java/org/mozilla/fenix/collections/CollectionCreationTabListAdapter.kt b/app/src/main/java/org/mozilla/fenix/collections/CollectionCreationTabListAdapter.kt index 50ad90233d..3e3bebfdf0 100644 --- a/app/src/main/java/org/mozilla/fenix/collections/CollectionCreationTabListAdapter.kt +++ b/app/src/main/java/org/mozilla/fenix/collections/CollectionCreationTabListAdapter.kt @@ -48,6 +48,8 @@ class CollectionCreationTabListAdapter( } else if (checkChanged.shouldBeUnchecked) { holder.view.tab_selected_checkbox.isChecked = false } + holder.view.tab_selected_checkbox.visibility = + if (checkChanged.shouldHideCheckBox) View.GONE else View.VISIBLE } } } @@ -123,14 +125,14 @@ private class TabDiffUtil( val shouldBeChecked = newSelected.contains(new[newItemPosition]) && !oldSelected.contains(old[oldItemPosition]) val shouldBeUnchecked = !newSelected.contains(new[newItemPosition]) && oldSelected.contains(old[oldItemPosition]) - return CheckChanged(shouldBeChecked, shouldBeUnchecked) + return CheckChanged(shouldBeChecked, shouldBeUnchecked, newHideCheckboxes) } override fun getOldListSize(): Int = old.size override fun getNewListSize(): Int = new.size } -data class CheckChanged(val shouldBeChecked: Boolean, val shouldBeUnchecked: Boolean) +data class CheckChanged(val shouldBeChecked: Boolean, val shouldBeUnchecked: Boolean, val shouldHideCheckBox: Boolean) class TabViewHolder( val view: View,