From 89d7a39b2f9b6ea9636e477d13e126637d9660c7 Mon Sep 17 00:00:00 2001
From: Lance Stout <lancestout@gmail.com>
Date: Wed, 18 Dec 2013 14:01:32 -0800
Subject: [PATCH] Make chat state updates less chatty

---
 clientapp/pages/chat.js      | 19 +++++++++----------
 clientapp/pages/groupchat.js | 18 ++++++++++--------
 public/x-manifest.cache      |  2 +-
 3 files changed, 20 insertions(+), 19 deletions(-)

diff --git a/clientapp/pages/chat.js b/clientapp/pages/chat.js
index a05ea4b..0eea49d 100644
--- a/clientapp/pages/chat.js
+++ b/clientapp/pages/chat.js
@@ -61,7 +61,6 @@ module.exports = BasePage.extend(chatHelpers).extend({
 
         this.renderAndBind();
 
-        this.typingTimer = null;
         this.$chatInput = this.$('.chatBox textarea');
         this.$chatBox = this.$('.chatBox');
         this.$messageList = this.$('.messages');
@@ -110,8 +109,6 @@ module.exports = BasePage.extend(chatHelpers).extend({
         this.resizeInput();
     },
     handleKeyDown: function (e) {
-        clearTimeout(this.typingTimer);
-
         if (e.which === 13 && !e.shiftKey) {
             this.sendChat();
             e.preventDefault();
@@ -128,8 +125,9 @@ module.exports = BasePage.extend(chatHelpers).extend({
             e.preventDefault();
             return false;
         } else if (!e.ctrlKey && !e.metaKey) {
-            if (!this.typing) {
+            if (!this.typing || this.paused) {
                 this.typing = true;
+                this.paused = false;
                 client.sendMessage({
                     to: this.model.lockedResource || this.model.jid,
                     chatState: 'composing'
@@ -139,25 +137,25 @@ module.exports = BasePage.extend(chatHelpers).extend({
     },
     handleKeyUp: function (e) {
         this.resizeInput();
-        clearTimeout(this.typingTimer);
-        this.typingTimer = setTimeout(this.pausedTyping.bind(this), 5000);
         if (this.typing && this.$chatInput.val().length === 0) {
             this.typing = false;
             client.sendMessage({
                 to: this.model.lockedResource || this.model.jid,
                 chatState: 'active'
             });
+        } else if (this.typing) {
+            this.pausedTyping();
         }
     },
-    pausedTyping: function () {
-        if (this.typing) {
-            this.typing = false;
+    pausedTyping: _.debounce(function () {
+        if (this.typing && !this.paused) {
+            this.paused = true;
             client.sendMessage({
                 to: this.model.lockedResource || this.model.jid,
                 chatState: 'paused'
             });
         }
-    },
+    }, 5000),
     sendChat: function () {
         var message;
         var val = this.$chatInput.val();
@@ -189,6 +187,7 @@ module.exports = BasePage.extend(chatHelpers).extend({
         }
         this.editMode = false;
         this.typing = false;
+        this.paused = false;
         this.$chatInput.removeClass('editing');
         this.$chatInput.val('');
     },
diff --git a/clientapp/pages/groupchat.js b/clientapp/pages/groupchat.js
index 593b037..d22eec8 100644
--- a/clientapp/pages/groupchat.js
+++ b/clientapp/pages/groupchat.js
@@ -54,7 +54,6 @@ module.exports = BasePage.extend(chatHelpers).extend({
         this.rendered = true;
 
         this.renderAndBind();
-        this.typingTimer = null;
         this.$chatInput = this.$('.chatBox textarea');
         this.$chatBox = this.$('.chatBox');
         this.$messageList = this.$('.messages');
@@ -96,7 +95,6 @@ module.exports = BasePage.extend(chatHelpers).extend({
         this.resizeInput();
     },
     handleKeyDown: function (e) {
-        clearTimeout(this.typingTimer);
         if (e.which === 13 && !e.shiftKey) {
             this.sendChat();
             e.preventDefault();
@@ -113,8 +111,9 @@ module.exports = BasePage.extend(chatHelpers).extend({
             e.preventDefault();
             return false;
         } else if (!e.ctrlKey && !e.metaKey) {
-            if (!this.typing) {
+            if (!this.typing || this.paused) {
                 this.typing = true;
+                this.paused = false;
                 client.sendMessage({
                     type: 'groupchat',
                     to: this.model.jid,
@@ -125,14 +124,16 @@ module.exports = BasePage.extend(chatHelpers).extend({
     },
     handleKeyUp: function (e) {
         this.resizeInput();
-        this.typingTimer = setTimeout(this.pausedTyping.bind(this), 3000);
         if (this.typing && this.$chatInput.val().length === 0) {
             this.typing = false;
+            this.paused = false;
             client.sendMessage({
                 type: 'groupchat',
                 to: this.model.jid,
                 chatState: 'active'
             });
+        } else if (this.typing) {
+            this.pausedTyping();
         }
     },
     resizeInput: function () {
@@ -158,16 +159,16 @@ module.exports = BasePage.extend(chatHelpers).extend({
             }
         }
     },
-    pausedTyping: function () {
-        if (this.typing) {
-            this.typing = false;
+    pausedTyping: _.debounce(function () {
+        if (this.typing && !this.paused) {
+            this.paused = true;
             client.sendMessage({
                 type: 'groupchat',
                 to: this.model.jid,
                 chatState: 'paused'
             });
         }
-    },
+    }, 5000),
     sendChat: function () {
         var message;
         var val = this.$chatInput.val();
@@ -197,6 +198,7 @@ module.exports = BasePage.extend(chatHelpers).extend({
         }
         this.editMode = false;
         this.typing = false;
+        this.paused = false;
         this.$chatInput.removeClass('editing');
         this.$chatInput.val('');
     },
diff --git a/public/x-manifest.cache b/public/x-manifest.cache
index 0f532fa..7f71117 100644
--- a/public/x-manifest.cache
+++ b/public/x-manifest.cache
@@ -1,5 +1,5 @@
 CACHE MANIFEST
-# 0.0.1 1387399967854
+# 0.0.1 1387404089194
 
 CACHE:
 /app.js