Merge pull request #954 from jgable/fixChangePasswordEnter

Fix pressing enter key on user settings

core/client/tpl/settings/user-profile.hbs

@@ -19,7 +19,7 @@
 
             <figure class="user-image">
                 <a id="user-image" class="img" {{#if image}}style="background-image: url({{image}});"{{/if}} href="#"><span class="hidden">{{name}}'s Picture</span></a>
-                <button class="edit-user-image js-modal-image">Edit Picture</button>
+                <button type="button" class="edit-user-image js-modal-image">Edit Picture</button>
             </figure>
 
             <div class="form-group">
@@ -78,7 +78,7 @@
                 <input type="password" id="user-new-password-verification" />
             </div>
             <div class="form-group">
-                <button class="button-delete button-change-password">Change Password</button>
+                <button type="button" class="button-delete button-change-password">Change Password</button>
             </div>
 
         </fieldset>

core/client/views/settings.js

@@ -234,6 +234,8 @@
 
     // ### User profile
     Settings.user = Settings.Pane.extend({
+        templateName: 'settings/user-profile',
+
         id: 'user',
 
         options: {
@@ -244,7 +246,8 @@
             'click .button-save': 'saveUser',
             'click .button-change-password': 'changePassword',
             'click .js-modal-cover': 'showCover',
-            'click .js-modal-image': 'showImage'
+            'click .js-modal-image': 'showImage',
+            'keyup .user-profile': 'handleEnterKeyOnForm'
         },
         showCover: function (e) {
             e.preventDefault();
@@ -281,6 +284,31 @@
             }));
         },
 
+        handleEnterKeyOnForm: function (ev) {
+            // Don't worry about it unless it's an enter key
+            if (ev.which !== 13) {
+                return;
+            }
+
+            var $target = $(ev.target);
+
+            if ($target.is("textarea")) {
+                // Allow enter key on user bio text area.
+                return;
+            }
+
+            if ($target.is('input[type=password]')) {
+                // Change password if on a password input
+                return this.changePassword(ev);
+            }
+
+            // Simulate clicking save otherwise
+            ev.preventDefault();
+
+            this.saveUser(ev);
+
+            return false;
+        },
 
         saveUser: function () {
             var userName = this.$('#user-name').val(),
@@ -369,8 +397,6 @@
             }
         },
 
-        templateName: 'settings/user-profile',
-
         afterRender: function () {
             var self = this;
             Countable.live(document.getElementById('user-bio'), function (counter) {