Merge pull request #1749 from Infomaniak/cleaning-new-message

Various cleaning in NewMessage
Infomaniak · Mar 12, 2024 · 072cb25 · 072cb25
2 parents 72825ce + 8e40142
commit 072cb25
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 7 deletions.
diff --git a/app/src/main/java/com/infomaniak/mail/ui/newMessage/NewMessageActivity.kt b/app/src/main/java/com/infomaniak/mail/ui/newMessage/NewMessageActivity.kt
@@ -42,7 +42,6 @@ import javax.inject.Inject
 class NewMessageActivity : BaseActivity() {
 
     private val binding by lazy { ActivityNewMessageBinding.inflate(layoutInflater) }
-    private val newMessageViewModel: NewMessageViewModel by viewModels()
 
     private val navController by lazy {
         (supportFragmentManager.findFragmentById(R.id.newMessageHostFragment) as NavHostFragment).navController

diff --git a/app/src/main/java/com/infomaniak/mail/ui/newMessage/RecipientFieldView.kt b/app/src/main/java/com/infomaniak/mail/ui/newMessage/RecipientFieldView.kt
@@ -214,10 +214,10 @@ class RecipientFieldView @JvmOverloads constructor(
     private fun setTextInputListeners() = with(binding.textInput) {
 
         fun performContactSearch(text: CharSequence) {
-            if ((text.trim().count()) > 0) {
-                contactAdapter.searchContacts(text)
-            } else {
+            if (text.isBlank()) {
                 contactAdapter.clear()
+            } else {
+                contactAdapter.searchContacts(text)
             }
         }
 

diff --git a/app/src/main/res/layout/fragment_new_message.xml b/app/src/main/res/layout/fragment_new_message.xml
@@ -111,7 +111,6 @@
                     </FrameLayout>
 
                     <com.google.android.material.divider.MaterialDivider
-                        android:id="@+id/divider1"
                         android:layout_width="0dp"
                         android:layout_height="wrap_content"
                         android:layout_marginHorizontal="@dimen/marginStandardMedium"
@@ -138,7 +137,6 @@
                         app:title="@string/toTitle" />
 
                     <com.google.android.material.divider.MaterialDivider
-                        android:id="@+id/divider2"
                         android:layout_width="match_parent"
                         android:layout_height="wrap_content"
                         android:layout_marginHorizontal="@dimen/marginStandardMedium"
@@ -188,7 +186,6 @@
                         app:title="@string/bccTitle" />
 
                     <com.google.android.material.divider.MaterialDivider
-                        android:id="@+id/divider4"
                         android:layout_width="match_parent"
                         android:layout_height="wrap_content"
                         android:layout_marginHorizontal="@dimen/marginStandardMedium"