[fenix] For https://github.com/mozilla-mobile/fenix/issues/3288 - call setChecked after calling setOnCheckedChangeListener (https://github.com/mozilla-mobile/fenix/pull/3357)

5 years ago · e1dc44b093
parent 8908bfd85c
commit e1dc44b093
1 changed files with 1 additions and 1 deletions
--- a/app/src/main/java/org/mozilla/fenix/collections/CollectionCreationTabListAdapter.kt
+++ b/app/src/main/java/org/mozilla/fenix/collections/CollectionCreationTabListAdapter.kt
@ -56,7 +56,6 @@ class CollectionCreationTabListAdapter(
    override fun onBindViewHolder(holder: TabViewHolder, position: Int) {
        val tab = tabs[position]
        val isSelected = selectedTabs.contains(tab)
-        holder.bind(tab, isSelected, hideCheckboxes)
        holder.view.tab_selected_checkbox.setOnCheckedChangeListener { _, isChecked ->
            val action = if (isChecked) {
                selectedTabs.add(tab)
@ -67,6 +66,7 @@ class CollectionCreationTabListAdapter(
            }
            actionEmitter.onNext(action)
        }
+        holder.bind(tab, isSelected, hideCheckboxes)
    }

    override fun getItemCount(): Int = tabs.size