Rework logic to remove confusion; add comment about why modifier keys…

… are ignored.
kandanapp · Mar 7, 2013 · 61f9614 · 61f9614
1 parent b47d9b4
commit 61f9614
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/app/assets/javascripts/backbone/views/chatbox.js.coffee b/app/assets/javascripts/backbone/views/chatbox.js.coffee
@@ -11,7 +11,11 @@ class Kandan.Views.Chatbox extends Backbone.View
 
 
   postMessageOnEnter: (event)->
-    if event.keyCode == 13 && !(event.metaKey || event.shiftKey || event.altKey || event.ctrlKey)
+    # If a modifier key is used with enter, messages are not posted.
+    # This allows the chatbox textarea to behave predictably, inline
+    # with usual form semantics (ie. ctrl+enter etc. generates a new
+    # line).
+    if event.keyCode == 13 && !event.metaKey && !event.shiftKey && !event.altKey && !event.ctrlKey
       @postMessage(event)
       event.preventDefault()