From dbc17ae5158559094fd5f8a8afa159b84b345540 Mon Sep 17 00:00:00 2001 From: Benoit Marty Date: Tue, 22 Oct 2019 18:23:53 +0200 Subject: [PATCH] Use AppCompatEditText instead of EditText --- .../features/home/room/detail/RoomDetailFragment.kt | 10 +++++----- .../home/room/detail/composer/ComposerEditText.kt | 4 ++-- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/vector/src/main/java/im/vector/riotx/features/home/room/detail/RoomDetailFragment.kt b/vector/src/main/java/im/vector/riotx/features/home/room/detail/RoomDetailFragment.kt index e20ec63769..13d9ac4a3d 100644 --- a/vector/src/main/java/im/vector/riotx/features/home/room/detail/RoomDetailFragment.kt +++ b/vector/src/main/java/im/vector/riotx/features/home/room/detail/RoomDetailFragment.kt @@ -421,7 +421,7 @@ class RoomDetailFragment : if (text != composerLayout.composerEditText.text.toString()) { // Ignore update to avoid saving a draft composerLayout.composerEditText.setText(text) - composerLayout.composerEditText.setSelection(composerLayout.composerEditText.text.length) + composerLayout.composerEditText.setSelection(composerLayout.composerEditText.text?.length ?: 0) } } @@ -1135,21 +1135,21 @@ class RoomDetailFragment : val myDisplayName = session.getUser(session.myUserId)?.displayName if (myDisplayName == text) { // current user - if (composerLayout.composerEditText.text.isBlank()) { + if (composerLayout.composerEditText.text.isNullOrBlank()) { composerLayout.composerEditText.append(Command.EMOTE.command + " ") - composerLayout.composerEditText.setSelection(composerLayout.composerEditText.text.length) + composerLayout.composerEditText.setSelection(composerLayout.composerEditText.text?.length ?: 0) // vibrate = true } } else { // another user - if (composerLayout.composerEditText.text.isBlank()) { + if (composerLayout.composerEditText.text.isNullOrBlank()) { // Ensure displayName will not be interpreted as a Slash command if (text.startsWith("/")) { composerLayout.composerEditText.append("\\") } composerLayout.composerEditText.append(sanitizeDisplayName(text) + ": ") } else { - composerLayout.composerEditText.text.insert(composerLayout.composerEditText.selectionStart, sanitizeDisplayName(text) + " ") + composerLayout.composerEditText.text?.insert(composerLayout.composerEditText.selectionStart, sanitizeDisplayName(text) + " ") } // vibrate = true diff --git a/vector/src/main/java/im/vector/riotx/features/home/room/detail/composer/ComposerEditText.kt b/vector/src/main/java/im/vector/riotx/features/home/room/detail/composer/ComposerEditText.kt index 9c894df880..273aeecbfa 100644 --- a/vector/src/main/java/im/vector/riotx/features/home/room/detail/composer/ComposerEditText.kt +++ b/vector/src/main/java/im/vector/riotx/features/home/room/detail/composer/ComposerEditText.kt @@ -23,12 +23,12 @@ import android.os.Build import android.util.AttributeSet import android.view.inputmethod.EditorInfo import android.view.inputmethod.InputConnection -import android.widget.EditText +import androidx.appcompat.widget.AppCompatEditText import androidx.core.view.inputmethod.EditorInfoCompat import androidx.core.view.inputmethod.InputConnectionCompat class ComposerEditText @JvmOverloads constructor(context: Context, attrs: AttributeSet? = null, defStyleAttr: Int = android.R.attr.editTextStyle) - : EditText(context, attrs, defStyleAttr) { + : AppCompatEditText(context, attrs, defStyleAttr) { interface Callback { fun onRichContentSelected(contentUri: Uri): Boolean