From 3f3b349555158f99c4aa22f77b602294f3f3221c Mon Sep 17 00:00:00 2001
From: Kyle Fiedler <kfiedler@thoughtbot.com>
Date: Wed, 7 Mar 2012 15:25:41 -0500
Subject: [PATCH 1/2] Added form and changes button to input so that users
 could hit enter on language and location

---
 profile.html | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/profile.html b/profile.html
index a126ad1fe9..b82996cb47 100644
--- a/profile.html
+++ b/profile.html
@@ -29,8 +29,8 @@ $(function() {
 
            loc=false;
 
-           $("#location_sub").html('<input id="id_loc_text" type="text" name="loc_text" />'+
-                                   '<input type="button" id="change_loc_button" value="Change" />');
+           $("#location_sub").html('<form>'+'<input id="id_loc_text" type="text" name="loc_text" />'+
+             '<input type="submit" id="change_loc_button" value="Change" />'+'</form>');
            $("#change_loc_button").click(function() {
              $("#change_location").show();
 
@@ -50,8 +50,8 @@ $(function() {
 
          if(lang) {
            lang=false;
-           $("#language_sub").html('<input id="id_lang_text" type="text" name="lang_text" />'+
-                                   '<input type="button" id="change_lang_button" value="Change" />');
+           $("#language_sub").html('<form>'+'<input id="id_lang_text" type="text" name="lang_text" />'+
+             '<input type="submit" id="change_lang_button" value="Change" />'+'</form>');
            $("#change_lang_button").click(function() {
              $("#change_language").show();
               postJSON('/change_setting', {'language':$("#id_lang_text").attr("value")}, function(json) {

From 64a4afe8c0cefd8c9f3b8311d02231e1b4b48ba3 Mon Sep 17 00:00:00 2001
From: Kyle Fiedler <kfiedler@thoughtbot.com>
Date: Wed, 7 Mar 2012 21:28:20 -0500
Subject: [PATCH 2/2] Added some new styles for edit button and changed some of
 the wording

---
 profile.html       | 11 +++++------
 sass/_profile.scss | 28 +++++++++++++++++++++++++++-
 2 files changed, 32 insertions(+), 7 deletions(-)

diff --git a/profile.html b/profile.html
index b82996cb47..5a753b914f 100644
--- a/profile.html
+++ b/profile.html
@@ -30,7 +30,8 @@ $(function() {
            loc=false;
 
            $("#location_sub").html('<form>'+'<input id="id_loc_text" type="text" name="loc_text" />'+
-             '<input type="submit" id="change_loc_button" value="Change" />'+'</form>');
+             '<input type="submit" id="change_loc_button" value="Save" />'+'</form>');
+
            $("#change_loc_button").click(function() {
              $("#change_location").show();
 
@@ -51,7 +52,7 @@ $(function() {
          if(lang) {
            lang=false;
            $("#language_sub").html('<form>'+'<input id="id_lang_text" type="text" name="lang_text" />'+
-             '<input type="submit" id="change_lang_button" value="Change" />'+'</form>');
+             '<input type="submit" id="change_lang_button" value="Save" />'+'</form>');
            $("#change_lang_button").click(function() {
              $("#change_language").show();
               postJSON('/change_setting', {'language':$("#id_lang_text").attr("value")}, function(json) {
@@ -140,15 +141,13 @@ $(function() {
         <li>Forum name: <strong>${username}</strong></li>
         <li>E-mail: <strong>${email}</strong></li>
         <li>
-          Location: <div id="location_sub">${location}</div><div id="description"></div> <a href="#" id="change_location">Change</a>
+          Location: <div id="location_sub">${location}</div><div id="description"></div> <a href="#" id="change_location">Edit</a>
         </li>
         <li>
-          Language: <div id="language_sub">${language}</div> <a href="#" id="change_language">Change</a>
+          Language: <div id="language_sub">${language}</div> <a href="#" id="change_language">Edit</a>
         </li>
       </ul>
 
-      <!-- <div><a class="modal" href="#change_password_pop">Reset password</a></div> -->
-
       <div id="change_password_pop">
         <h2>Password change</h2>
         <p>We'll e-mail a password reset link to ${email}.</p>
diff --git a/sass/_profile.scss b/sass/_profile.scss
index f696116ab8..3182d61b36 100644
--- a/sass/_profile.scss
+++ b/sass/_profile.scss
@@ -26,13 +26,26 @@ div.profile-wrapper {
         @include box-shadow(0 1px 0 #eee);
         padding: 7px lh();
         border-bottom: 1px solid #d3d3d3;
+        position: relative;
 
         div#location_sub, div#language_sub {
           font-weight: bold;
           @include inline-block();
 
+          form {
+            width: 100%;
+          }
+
           input {
-            margin: lh(.5) 0;
+
+            &[type="text"] {
+              margin: lh(.5) 0;
+              width: 100%;
+              @include box-sizing(border-box);
+            }
+
+            &[type="input"]{
+            }
           }
 
           &:empty {
@@ -43,6 +56,19 @@ div.profile-wrapper {
         div#description {
           font-size: 12px;
         }
+
+        a#change_language, a#change_location {
+          position: absolute;
+          top: 9px;
+          right: lh(.5);
+          text-transform: uppercase;
+          font-size: 12px;
+          color: #999;
+
+          &:hover {
+            color: #555;
+          }
+        }
       }
     }