[Bugfix #5029] disable emoji keyboard not applies to reply

Signed-off-by: eidonia <bastien_rambeaud@hotmail.com>
This commit is contained in:
eidonia 2022-08-06 01:32:21 +02:00
parent 81897a179e
commit cd71abeb9f
6 changed files with 19 additions and 3 deletions

1
changelog.d/5029.bugfix Normal file
View File

@ -0,0 +1 @@
Disable emoji keyboard not applies in reply

View File

@ -353,6 +353,8 @@ class TimelineFragment @Inject constructor(
private var lockSendButton = false private var lockSendButton = false
private val currentCallsViewPresenter = CurrentCallsViewPresenter() private val currentCallsViewPresenter = CurrentCallsViewPresenter()
private val isEmojiKeyboardVisible: Boolean
get() = vectorPreferences.showEmojiKeyboard()
private val lazyLoadedViews = RoomDetailLazyLoadedViews() private val lazyLoadedViews = RoomDetailLazyLoadedViews()
private val emojiPopup: EmojiPopup by lifecycleAwareLazy { private val emojiPopup: EmojiPopup by lifecycleAwareLazy {
@ -1571,6 +1573,10 @@ class TimelineFragment @Inject constructor(
attachmentTypeSelector.show(views.composerLayout.views.attachmentButton) attachmentTypeSelector.show(views.composerLayout.views.attachmentButton)
} }
override fun onExpandOrCompactChange() {
views.composerLayout.views.composerEmojiButton.isVisible = isEmojiKeyboardVisible
}
override fun onSendMessage(text: CharSequence) { override fun onSendMessage(text: CharSequence) {
sendTextMessage(text) sendTextMessage(text)
} }

View File

@ -46,6 +46,7 @@ class MessageComposerView @JvmOverloads constructor(
fun onCloseRelatedMessage() fun onCloseRelatedMessage()
fun onSendMessage(text: CharSequence) fun onSendMessage(text: CharSequence)
fun onAddAttachment() fun onAddAttachment()
fun onExpandOrCompactChange()
} }
val views: ComposerLayoutBinding val views: ComposerLayoutBinding
@ -96,6 +97,7 @@ class MessageComposerView @JvmOverloads constructor(
} }
currentConstraintSetId = R.layout.composer_layout_constraint_set_compact currentConstraintSetId = R.layout.composer_layout_constraint_set_compact
applyNewConstraintSet(animate, transitionComplete) applyNewConstraintSet(animate, transitionComplete)
callback?.onExpandOrCompactChange()
} }
fun expand(animate: Boolean = true, transitionComplete: (() -> Unit)? = null) { fun expand(animate: Boolean = true, transitionComplete: (() -> Unit)? = null) {
@ -105,6 +107,7 @@ class MessageComposerView @JvmOverloads constructor(
} }
currentConstraintSetId = R.layout.composer_layout_constraint_set_expanded currentConstraintSetId = R.layout.composer_layout_constraint_set_expanded
applyNewConstraintSet(animate, transitionComplete) applyNewConstraintSet(animate, transitionComplete)
callback?.onExpandOrCompactChange()
} }
fun setTextIfDifferent(text: CharSequence?): Boolean { fun setTextIfDifferent(text: CharSequence?): Boolean {

View File

@ -119,8 +119,10 @@
android:background="?android:attr/selectableItemBackground" android:background="?android:attr/selectableItemBackground"
android:contentDescription="@string/a11y_open_emoji_picker" android:contentDescription="@string/a11y_open_emoji_picker"
android:src="@drawable/ic_insert_emoji" android:src="@drawable/ic_insert_emoji"
android:visibility="invisible"
app:tint="?vctr_content_tertiary" app:tint="?vctr_content_tertiary"
tools:ignore="MissingConstraints,MissingPrefix" /> tools:ignore="MissingConstraints,MissingPrefix"
tools:visibility="visible" />
<ImageButton <ImageButton
android:id="@+id/sendButton" android:id="@+id/sendButton"

View File

@ -155,13 +155,15 @@
android:background="?android:attr/selectableItemBackground" android:background="?android:attr/selectableItemBackground"
android:contentDescription="@string/a11y_open_emoji_picker" android:contentDescription="@string/a11y_open_emoji_picker"
android:src="@drawable/ic_insert_emoji" android:src="@drawable/ic_insert_emoji"
android:visibility="invisible"
app:layout_constraintBottom_toBottomOf="@id/attachmentButton" app:layout_constraintBottom_toBottomOf="@id/attachmentButton"
app:layout_constraintEnd_toEndOf="@id/composerEditTextOuterBorder" app:layout_constraintEnd_toEndOf="@id/composerEditTextOuterBorder"
app:layout_constraintStart_toEndOf="@id/composerEditText" app:layout_constraintStart_toEndOf="@id/composerEditText"
app:layout_constraintTop_toTopOf="@id/attachmentButton" app:layout_constraintTop_toTopOf="@id/attachmentButton"
app:layout_goneMarginEnd="8dp" app:layout_goneMarginEnd="8dp"
app:tint="?vctr_content_quaternary" app:tint="?vctr_content_quaternary"
tools:ignore="MissingPrefix" /> tools:ignore="MissingPrefix"
tools:visibility="visible" />
<ImageButton <ImageButton
android:id="@+id/sendButton" android:id="@+id/sendButton"

View File

@ -166,6 +166,7 @@
android:background="?android:attr/selectableItemBackground" android:background="?android:attr/selectableItemBackground"
android:contentDescription="@string/a11y_open_emoji_picker" android:contentDescription="@string/a11y_open_emoji_picker"
android:src="@drawable/ic_insert_emoji" android:src="@drawable/ic_insert_emoji"
android:visibility="invisible"
app:layout_constraintBottom_toBottomOf="@id/sendButton" app:layout_constraintBottom_toBottomOf="@id/sendButton"
app:layout_constraintEnd_toEndOf="@id/composerEditTextOuterBorder" app:layout_constraintEnd_toEndOf="@id/composerEditTextOuterBorder"
app:layout_constraintStart_toEndOf="@id/composerEditText" app:layout_constraintStart_toEndOf="@id/composerEditText"
@ -173,7 +174,8 @@
app:layout_goneMarginBottom="52dp" app:layout_goneMarginBottom="52dp"
app:layout_goneMarginEnd="8dp" app:layout_goneMarginEnd="8dp"
app:tint="?vctr_content_quaternary" app:tint="?vctr_content_quaternary"
tools:ignore="MissingPrefix" /> tools:ignore="MissingPrefix"
tools:visibility="visible" />
<ImageButton <ImageButton
android:id="@+id/sendButton" android:id="@+id/sendButton"