Merge pull request #1832 from Infomaniak/too-much-draft-saving

Only save Draft when adding Attachment
Infomaniak · May 1, 2024 · 3962e4e · 3962e4e
2 parents 8f4dcd3 + 5be7da0
commit 3962e4e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/app/src/main/java/com/infomaniak/mail/ui/main/folder/ThreadListMultiSelection.kt b/app/src/main/java/com/infomaniak/mail/ui/main/folder/ThreadListMultiSelection.kt
@@ -44,7 +44,7 @@ import com.infomaniak.mail.utils.extensions.updateNavigationBarColor
 class ThreadListMultiSelection {
 
     lateinit var mainViewModel: MainViewModel
-    lateinit var threadListFragment: ThreadListFragment
+    private lateinit var threadListFragment: ThreadListFragment
     lateinit var unlockSwipeActionsIfSet: () -> Unit
     lateinit var localSettings: LocalSettings
 

diff --git a/app/src/main/java/com/infomaniak/mail/ui/newMessage/NewMessageFragment.kt b/app/src/main/java/com/infomaniak/mail/ui/newMessage/NewMessageFragment.kt
@@ -475,7 +475,9 @@ class NewMessageFragment : Fragment() {
             if (isFirstTime) {
                 isFirstTime = false
                 observeImportAttachments()
-            } else {
+            } else if (attachments.count() > attachmentAdapter.itemCount) {
+                // If we are adding Attachments, directly save the Draft, so the Attachments' upload starts now.
+                // TODO: Only save Attachments, and not the whole Draft.
                 saveDraft()
             }