[Merge] lp:~phablet-team/messaging-app/send-typing-notification into lp:messaging-app
Roberto Mier Escandón
roberto.escandon at canonical.com
Sat Jun 25 14:50:28 UTC 2016
Review: Needs Fixing
page text property is always empty while typing a text message. This prevents proper work of typing notifications
Diff comments:
> === modified file 'src/qml/Messages.qml'
> --- src/qml/Messages.qml 2016-06-16 11:19:36 +0000
> +++ src/qml/Messages.qml 2016-06-16 11:19:36 +0000
> @@ -1177,6 +1191,15 @@
> showContents: !selectionMode && !isSearching
> maxHeight: messages.height - keyboard.height - screenTop.y
> text: messages.text
> + onTextChanged: {
There is a problem in this method. text is always empty while writing a message. It is only fulfilled when pressed button to send the message. This is preventing typing notification being sent properly to the other side. Please check where is the failure
> + if (text == "") {
> + messages.chatEntry.setChatState(ChatEntry.ChannelChatStateActive)
> + selfTypingTimer.stop()
> + return
> + }
> + messages.chatEntry.setChatState(ChatEntry.ChannelChatStateComposing)
> + selfTypingTimer.start()
> + }
> canSend: participants.length > 0 || multiRecipient.recipientCount > 0 || multiRecipient.searchString !== ""
> oskEnabled: messages.oskEnabled
>
--
https://code.launchpad.net/~phablet-team/messaging-app/send-typing-notification/+merge/296609
Your team Ubuntu Phablet Team is subscribed to branch lp:~phablet-team/messaging-app/async_send_message.
More information about the Ubuntu-reviews
mailing list