kiwix/kiwix-android
1
0
Fork 0
mirror of https://github.com/kiwix/kiwix-android.git synced 2025-09-07 22:31:17 -04:00
Code Issues Packages Projects Releases Wiki Activity

Fixed: UI was not updating when selecting items for deletion.

Browse Source
This commit is contained in:
MohitMaliFtechiz 2025-04-14 17:34:08 +05:30
parent 0aa39d8a2b
commit 01a3838c6a
1 changed files with 7 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
core/src/main/java/org/kiwix/kiwixmobile/core/page/PageFragment.kt
View File

@ -144,15 +144,6 @@ abstract class PageFragment : OnItemClickListener, BaseFragment(), FragmentActiv
val activity = requireActivity() as CoreMainActivity val activity = requireActivity() as CoreMainActivity
compositeDisposable.add(pageViewModel.effects.subscribe { it.invokeWith(activity) }) compositeDisposable.add(pageViewModel.effects.subscribe { it.invokeWith(activity) })
pageViewModel.state.observe(viewLifecycleOwner, Observer(::render)) pageViewModel.state.observe(viewLifecycleOwner, Observer(::render))
// // hides keyboard when scrolled
// fragmentPageBinding?.recyclerView?.addOnScrollListener(
// SimpleRecyclerViewScrollListener { _, newState ->
// if (newState == RecyclerView.SCROLL_STATE_DRAGGING) {
// fragmentPageBinding?.recyclerView?.closeKeyboard()
// }
// }
// )
} }
override fun onCreateView( override fun onCreateView(
@ -266,6 +257,13 @@ abstract class PageFragment : OnItemClickListener, BaseFragment(), FragmentActiv
private fun render(state: PageState<*>) { private fun render(state: PageState<*>) {
pageScreenState.value = pageScreenState.value.copy( pageScreenState.value = pageScreenState.value.copy(
switchIsEnabled = !state.isInSelectionState, switchIsEnabled = !state.isInSelectionState,
// First, assign the existing state to force Compose to recognize a change.
// This helps when internal properties of items (like `isSelected`) change,
// but the list reference itself remains the same — Compose won't detect it otherwise.
pageState = pageState.value
)
// Then, assign the actual updated state to trigger full recomposition.
pageScreenState.value = pageScreenState.value.copy(
pageState = state pageState = state
) )
if (state.isInSelectionState) { if (state.isInSelectionState) {