Disallow submit of whitespace only labels
authorTim Düsterhus <duesterhus@woltlab.com>
Mon, 22 Apr 2013 14:40:32 +0000 (16:40 +0200)
committerTim Düsterhus <duesterhus@woltlab.com>
Mon, 22 Apr 2013 14:41:13 +0000 (16:41 +0200)
And improve uX of label update a bit

files/js/WCF.Conversation.js

index 351c0bba03d6554f9a1c9bafdda3f1da4d3ed7ea..964cf6f64f7e1d65d9d20aed1a5f71e2acdbdf07 100644 (file)
@@ -1141,7 +1141,7 @@ WCF.Conversation.Label.Manager = Class.extend({
         * Binds event listener for label management.
         */
        _bindListener: function() {
-               $('#labelName').keyup($.proxy(this._updateLabels, this));
+               $('#labelName').on('keyup keydown keypress', $.proxy(this._updateLabels, this));
                $('#addLabel').disable().click($.proxy(this._addLabel, this));
                $('#editLabel').disable();
                
@@ -1220,7 +1220,7 @@ WCF.Conversation.Label.Manager = Class.extend({
         * Updates label text within label management.
         */
        _updateLabels: function() {
-               var $value = $('#labelName').val();
+               var $value = $.trim($('#labelName').val());
                if ($value) {
                        $('#addLabel, #editLabel').enable();
                }