* 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 * Update ripple effect for note item
ARK-Builders · Oct 26, 2024 · b85a4ac · b85a4ac
1 parent 78aa757
commit b85a4ac
Show file tree

Hide file tree

Showing 5 changed files with 33 additions and 12 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/drawable/ripple_note_item.xml b/app/src/main/res/drawable/ripple_note_item.xml
@@ -0,0 +1,9 @@
+<?xml version="1.0" encoding="utf-8"?>
+<ripple xmlns:android="http://schemas.android.com/apk/res/android" android:color="@android:color/darker_gray">
+    <item android:id="@android:id/background">
+        <shape android:shape="rectangle">
+            <solid android:color="@color/bg_primary"/>
+            <corners android:radius="@dimen/corner_radius_big"/>
+        </shape>
+    </item>
+</ripple>
diff --git a/app/src/main/res/layout/adapter_text_note.xml b/app/src/main/res/layout/adapter_text_note.xml
@@ -6,7 +6,7 @@
     xmlns:tools="http://schemas.android.com/tools"
     android:clickable="true"
     android:layout_marginBottom="@dimen/note_item_padding"
-    android:foreground="?attr/selectableItemBackgroundBorderless">
+    android:background="@drawable/ripple_note_item">
 
     <CheckBox
         android:layout_width="wrap_content"

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"