* Do not destroy action mode if app goes to background

* Long pressing on an item should also select it * Navigating back while in action mode should destroy it first
ARK-Builders · Oct 26, 2024 · c0ef5dd · c0ef5dd
1 parent 78aa757
commit c0ef5dd
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 11 deletions.
diff --git a/app/src/main/java/dev/arkbuilders/arkmemo/ui/adapters/NotesListAdapter.kt b/app/src/main/java/dev/arkbuilders/arkmemo/ui/adapters/NotesListAdapter.kt
@@ -204,10 +204,16 @@ class NotesListAdapter(
         this.notes = notes.toMutableList()
     }
 
-    fun toggleActionMode() {
+    fun toggleActionMode(pos: Int) {
         mActionMode = !mActionMode
-        notes.forEach { it.selected = false }
-        selectedNoteCount.postValue(0)
+        var selectedCount = 0
+        notes.forEachIndexed { index, note ->
+            note.selected = mActionMode && index == pos
+            if (index == pos) {
+                selectedCount ++
+            }
+        }
+        selectedNoteCount.postValue(selectedCount)
         notifyDataSetChanged()
     }
 

diff --git a/app/src/main/java/dev/arkbuilders/arkmemo/ui/fragments/NotesFragment.kt b/app/src/main/java/dev/arkbuilders/arkmemo/ui/fragments/NotesFragment.kt
@@ -228,7 +228,12 @@ class NotesFragment: BaseFragment() {
         observePlayerSideEffect()
         notesAdapter?.setActivity(activity)
         notesAdapter?.onItemLongPressed = {pos, note ->
-            toggleActionMode()
+            toggleActionMode(pos = pos)
+        }
+        notesAdapter?.onItemClicked = {
+            if (mIsActionMode) {
+                toggleActionMode()
+            }
         }
         binding.rvPinnedNotes.apply {
             this.layoutManager = layoutManager
@@ -307,10 +312,6 @@ class NotesFragment: BaseFragment() {
                 notesAdapter?.notifyItemChanged(playingAudioPosition)
             }
         }
-
-        if (mIsActionMode) {
-            toggleActionMode()
-        }
     }
 
     override fun onPause() {
@@ -411,7 +412,7 @@ class NotesFragment: BaseFragment() {
         }
     }
 
-    private fun toggleActionMode() {
+    private fun toggleActionMode(pos: Int = -1) {
         if (mIsActionMode) {
             binding.groupActionModeTexts.gone()
             binding.layoutBottomControl.visible()
@@ -439,7 +440,7 @@ class NotesFragment: BaseFragment() {
                 showBatchDeletionDialog()
             }
         }
-        (binding.rvPinnedNotes.adapter as? NotesListAdapter)?.toggleActionMode()
+        (binding.rvPinnedNotes.adapter as? NotesListAdapter)?.toggleActionMode(pos)
         mIsActionMode = !mIsActionMode
     }
 
@@ -506,6 +507,10 @@ class NotesFragment: BaseFragment() {
             binding.edtSearch.text.clear()
             binding.edtSearch.clearFocus()
             binding.rvPinnedNotes.layoutManager?.scrollToPosition(lastNoteItemPosition)
+
+        } else if (mIsActionMode) {
+            toggleActionMode()
+
         } else {
             activity.onBackPressedDispatcher.onBackPressed()
         }

diff --git a/app/src/main/res/layout/fragment_home.xml b/app/src/main/res/layout/fragment_home.xml
@@ -3,7 +3,8 @@
     android:layout_width="match_parent"
     android:layout_height="match_parent"
     xmlns:app="http://schemas.android.com/apk/res-auto"
-    xmlns:tools="http://schemas.android.com/tools">
+    xmlns:tools="http://schemas.android.com/tools"
+    android:animateLayoutChanges="true">
 
     <androidx.appcompat.widget.AppCompatImageView
         android:layout_width="@dimen/home_toolbar_icon_size"