From b708e2a7de2c849fdf2edf436c831c026da978d5 Mon Sep 17 00:00:00 2001
From: Shane Tomlinson <stomlinson@mozilla.com>
Date: Mon, 26 Sep 2011 16:57:06 +0100
Subject: [PATCH] Giving the "new email address" field a label.  Made the
 transition to show the new email address field smoother.

---
 .../static/dialog/controllers/pickemail_controller.js      | 7 ++++---
 browserid/static/dialog/views/pickemail.ejs                | 3 ++-
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/browserid/static/dialog/controllers/pickemail_controller.js b/browserid/static/dialog/controllers/pickemail_controller.js
index 3a287c4ff..d5382c5d5 100644
--- a/browserid/static/dialog/controllers/pickemail_controller.js
+++ b/browserid/static/dialog/controllers/pickemail_controller.js
@@ -70,9 +70,10 @@
 
     "#useDifferentEmail click": function() {
       //this.close("pickemail:addemail");
-      $("#signInButton").hide();
-      $("#differentEmail").slideDown(300);
-      $("#newEmail").focus();
+      $("#signInButton,#useDifferentEmail").fadeOut(250, function() {
+        $("#differentEmail").fadeIn(250);
+        $("#newEmail").focus();
+      });
     }
 
   });
diff --git a/browserid/static/dialog/views/pickemail.ejs b/browserid/static/dialog/views/pickemail.ejs
index 448014170..85539946b 100644
--- a/browserid/static/dialog/views/pickemail.ejs
+++ b/browserid/static/dialog/views/pickemail.ejs
@@ -29,7 +29,8 @@
                   <div class="add cf">
                       <a id="useDifferentEmail" href="#">Use a different email</a>
                       <div id="differentEmail" class="cf">
-                          <input id="newEmail" type="email"/>
+                          <label for="newEmail">New email address</label>
+                          <input id="newEmail" name="newEmail" type="email"/>
                           <button id="addDifferentEmail">Add</button>
                       </div>
                   </div>
-- 
GitLab