summaryrefslogtreecommitdiff
path: root/addons/mail/static/src/js/thread_widget.js
diff options
context:
space:
mode:
authorAlexandre Kühn <aku@odoo.com>2018-08-11 21:02:55 +0200
committerAlexandre Kühn <aku@odoo.com>2018-08-13 14:17:40 +0200
commit5f1d32f3fb89f40740b25491caafb9a5710d555a (patch)
treed94aab4d0a6fb36d60079391216b3dd4c89b23c7 /addons/mail/static/src/js/thread_widget.js
parent614f9f84d7da281b0feacc39a612c19193f3a4d8 (diff)
[IMP] mail, im_livechat: 'is typing...' indicator on mail channels
With this commit, most threads show an indicator when a member is typing something. e.g. "Mitchell Stephens is typing..." Supported threads: - DM - (public & private) channels - Livechat (Website & Backend) Unsupported threads: - mailboxes (e.g. 'Inbox') - document thread (= 'Chatter') - support channel (with `im_support` module) Some technical details: - This is a mix between 'start'/'stop' and 'regular notify' strategies. - On typing text in composer: notify 'start typing' to all members of thread. - On clearing text in composer: notify 'stop typing' to all members of thread. - A member can only notify once every 2.5 seconds to the server when he starts or stops typing something (throttled, buffered notify call). - On receiving a message from someone that is typing: determine that he is no longer typing. - On non-empty and unchanged composer text for 5.0 seconds: automatically notifies all members of thread that he is no longer typing anything. (This is like a 'soft / cooperative sender' timeout). - On non-receiving a typing notification of someone after 60 seconds: determine that he is not longer typing something (This is like a 'hard / receiver assumption' timeout). - When some types for more than 50 seconds straight: notify that he is still typing something (again and again every 50 seconds). - When two or more people are typing something (Multi-User channels), it displays at most 2 typing users (ordering rule: longer typers first): "Mitchell Stephens, Marc Brown and more are typing..." - Timing summary: - Usually 2.5 seconds uncertainty. - Up to 5.0 seconds uncertainty on typer inactivity. - Up to 60 seconds uncertainty on typer page reload / browser tab closed / etc. Future Improvements: - Detect page reload or browser tab close with loose of longpolling connection Task-ID 28188
Diffstat (limited to 'addons/mail/static/src/js/thread_widget.js')
-rw-r--r--addons/mail/static/src/js/thread_widget.js28
1 files changed, 28 insertions, 0 deletions
diff --git a/addons/mail/static/src/js/thread_widget.js b/addons/mail/static/src/js/thread_widget.js
index c2d642d5dbe..2abd539f518 100644
--- a/addons/mail/static/src/js/thread_widget.js
+++ b/addons/mail/static/src/js/thread_widget.js
@@ -173,6 +173,12 @@ var ThreadWidget = Widget.extend({
dateFormat: time.getLangDatetimeFormat(),
}));
+ // must be after mail.widget.Thread rendering, so that there is the
+ // DOM element for the 'is typing' notification bar
+ if (thread.hasTypingNotification()) {
+ this.renderTypingNotificationBar(thread);
+ }
+
_.each(messages, function (message) {
var $message = self.$('.o_thread_message[data-message-id="'+ message.getID() +'"]');
$message.find('.o_mail_timestamp').data('date', message.getDate());
@@ -234,6 +240,28 @@ var ThreadWidget = Widget.extend({
return done;
},
/**
+ * Render the 'is typing...' text on the typing notification bar of the
+ * thread. This is called when there is a change in the list of users
+ * typing something on this thread.
+ *
+ * @param {mail.model.AbstractThread} thread with ThreadTypingMixin
+ */
+ renderTypingNotificationBar: function (thread) {
+ if (this._currentThreadID === thread.getID()) {
+ var shouldScrollToBottomAfterRendering = this.isAtBottom();
+
+ // typing notification bar rendering
+ var $typingBar = this.$('.o_thread_typing_notification_bar');
+ var text = thread.getTypingMembersToText();
+ $typingBar.toggleClass('o_hidden', !text); // hide if no text, because of padding
+ $typingBar.text(text);
+
+ if (shouldScrollToBottomAfterRendering) {
+ this.scrollToBottom();
+ }
+ }
+ },
+ /**
* Scroll to the bottom of the thread
*/
scrollToBottom: function () {