Improved Auth screen markup and validation checks

* Signup now focuses on 'name' on load
* Fixed fade in on auth forms to work with `display: table`
* The 'name' field is required on Sign up forms
* The length check on the Signup form is in order of inputs
* Added check for password length
* Changed the auth form class names to better represent individual pages
* Updated CasperJS tests

core/client/assets/sass/layouts/auth.scss

@@ -1,5 +1,5 @@
 /*
- * These styles control elements specific to the Ghost admin login screen.
+ * These styles control elements specific to the Ghost admin login / signup screens.
  *
  * Table of Contents:
  * 
@@ -14,6 +14,7 @@
    ============================================================================= */
 
 .ghost-login,
+.ghost-signup,
 .ghost-forgotten {
     color: $midgrey;
     background: $darkgrey;
@@ -28,7 +29,9 @@
 
 }//.ghost-login
 
-.login-box {
+.login-box,
+.signup-box,
+.forgotten-box {
     max-width: 530px;
     height: 90%;
     margin: 0 auto;

core/client/assets/sass/screen.scss

@@ -40,7 +40,7 @@
     @import "layouts/editor";
     /* The write/edit post screen. */
 
-    @import "layouts/login";
+    @import "layouts/auth";
     /* The login screen. */
 
     @import "layouts/errors";

core/client/router.js

@@ -19,7 +19,7 @@
         },
 
         signup: function () {
-            Ghost.currentView = new Ghost.Views.Signup({ el: '.js-login-box' });
+            Ghost.currentView = new Ghost.Views.Signup({ el: '.js-signup-box' });
         },
 
         login: function () {
@@ -27,7 +27,7 @@
         },
 
         forgotten: function () {
-            Ghost.currentView = new Ghost.Views.Forgotten({ el: '.js-login-box' });
+            Ghost.currentView = new Ghost.Views.Forgotten({ el: '.js-forgotten-box' });
         },
 
         blog: function () {

core/client/views/login.js

@@ -2,17 +2,14 @@
 (function () {
     "use strict";
 
+    Ghost.Views.Login = Ghost.View.extend({
 
-    Ghost.SimpleFormView = Ghost.View.extend({
         initialize: function () {
             this.render();
-            $(".js-login-box").fadeIn(500, function () {
+            $(".js-login-box").css({"opacity": 0}).animate({"opacity": 1}, 500, function () {
                 $("[name='email']").focus();
             });
-        }
+        },
-    });
-
-    Ghost.Views.Login = Ghost.SimpleFormView.extend({
 
         templateName: "login",
 
@@ -48,7 +45,14 @@
         }
     });
 
-    Ghost.Views.Signup = Ghost.SimpleFormView.extend({
+    Ghost.Views.Signup = Ghost.View.extend({
+
+        initialize: function () {
+            this.render();
+            $(".js-signup-box").css({"opacity": 0}).animate({"opacity": 1}, 500, function () {
+                $("[name='name']").focus();
+            });
+        },
 
         templateName: "signup",
 
@@ -62,6 +66,25 @@
                 email = this.$el.find('.email').val(),
                 password = this.$el.find('.password').val();
 
+            if (!name) {
+                Ghost.notifications.addItem({
+                    type: 'error',
+                    message: "Please enter a name",
+                    status: 'passive'
+                });
+            } else if (!email) {
+                Ghost.notifications.addItem({
+                    type: 'error',
+                    message: "Please enter an email",
+                    status: 'passive'
+                });
+            } else if (!password) {
+                Ghost.notifications.addItem({
+                    type: 'error',
+                    message: "Please enter a password",
+                    status: 'passive'
+                });
+            } else {
                 $.ajax({
                     url: '/ghost/signup/',
                     type: 'POST',
@@ -82,9 +105,17 @@
                     }
                 });
             }
+        }
     });
 
-    Ghost.Views.Forgotten = Ghost.SimpleFormView.extend({
+    Ghost.Views.Forgotten = Ghost.View.extend({
+
+        initialize: function () {
+            this.render();
+            $(".js-forgotten-box").css({"opacity": 0}).animate({"opacity": 1}, 500, function () {
+                $("[name='email']").focus();
+            });
+        },
 
         templateName: "forgotten",
 

core/server/controllers/admin.js

@@ -116,7 +116,7 @@ adminControllers = {
         }
     },
     'login': function (req, res) {
-        res.render('signup', {
+        res.render('login', {
             bodyClass: 'ghost-login',
             hideNavbar: true,
             adminNav: setSelected(adminNavbar, 'login')
@@ -161,7 +161,7 @@ adminControllers = {
     },
     'signup': function (req, res) {
         res.render('signup', {
-            bodyClass: 'ghost-login',
+            bodyClass: 'ghost-signup',
             hideNavbar: true,
             adminNav: setSelected(adminNavbar, 'login')
         });
@@ -189,7 +189,7 @@ adminControllers = {
     },
 
     'forgotten': function (req, res) {
-        res.render('signup', {
+        res.render('forgotten', {
             bodyClass: 'ghost-forgotten',
             hideNavbar: true,
             adminNav: setSelected(adminNavbar, 'login')

core/server/views/forgotten.hbs

@@ -0,0 +1,4 @@
+{{!< default}}
+<section class="forgotten-box js-forgotten-box">
+
+</section>

core/server/views/signup.hbs

@@ -1,4 +1,4 @@
 {{!< default}}
-<section class="login-box js-login-box">
+<section class="signup-box js-signup-box">
 
 </section>

core/test/functional/admin/01_login_test.js

@@ -17,14 +17,12 @@ casper.test.begin('Ensure a User is Registered', 2, function suite(test) {
 
     casper.start(url + 'ghost/signup/').viewport(1280, 1024);
 
-    casper.waitFor(function checkOpaque() {
+    casper.waitForOpaque(".signup-box",
-        return this.evaluate(function () {
+        function then() {
-            var loginBox = document.querySelector('.login-box');
+            this.fill("#signup", newUser, true);
-            return window.getComputedStyle(loginBox).getPropertyValue('display') === "table"
+        },
-                && window.getComputedStyle(loginBox).getPropertyValue('opacity') === "1";
+        function onTimeout() {
-        });
+            test.fail('Sign up form didn\'t fade in.');
-    }, function then() {
-        this.fill("#signup", user, true);
         });
 
     casper.waitForSelectorTextChange('.notification-error', function onSuccess() {
@@ -86,16 +84,11 @@ casper.test.begin("Can't spam it", 4, function suite(test) {
         test.assertTitle("", "Ghost admin has no title");
     }).viewport(1280, 1024);
 
-    casper.waitFor(function checkOpaque() {
+    casper.waitForOpaque(".login-box",
-        return this.evaluate(function () {
+        function then() {
-            var loginBox = document.querySelector('.login-box');
-
-            return window.getComputedStyle(loginBox).getPropertyValue('display') === "table"
-                && window.getComputedStyle(loginBox).getPropertyValue('opacity') === "1";
-        });
-    }, function then() {
             this.fill("#login", falseUser, true);
-    }, function onTimeout() {
+        },
+        function onTimeout() {
             test.fail('Sign in form didn\'t fade in.');
         });
 
@@ -127,13 +120,8 @@ casper.test.begin("Can login to Ghost", 4, function suite(test) {
         test.assertTitle("", "Ghost admin has no title");
     }).viewport(1280, 1024);
 
-    casper.waitFor(function checkOpaque() {
+    casper.waitForOpaque(".login-box",
-        return casper.evaluate(function () {
+        function then() {
-            var loginBox = document.querySelector('.login-box');
-            return window.getComputedStyle(loginBox).getPropertyValue('display') === "table"
-                && window.getComputedStyle(loginBox).getPropertyValue('opacity') === "1";
-        });
-    }, function then() {
             this.fill("#login", user, true);
         });
 

core/test/functional/base.js

@@ -27,6 +27,11 @@ var host = casper.cli.options.host || 'localhost',
     email = casper.cli.options.email || 'ghost@tryghost.org',
     password = casper.cli.options.password || 'Sl1m3rson',
     url = "http://" + host + (noPort ? '/' : ":" + port + "/"),
+    newUser = {
+        name: "Test User",
+        email: email,
+        password: password
+    },
     user = {
         email: email,
         password: password
@@ -51,6 +56,14 @@ casper.writeContentToCodeMirror = function (content) {
     return this;
 };
 
+casper.waitForOpaque = function (classname, then, timeout) {
+    casper.waitFor(function checkOpaque() {
+        return this.evaluate(function (element) {
+            var target = document.querySelector(element);
+            return window.getComputedStyle(target).getPropertyValue('opacity') === "1";
+        }, classname);
+    }, then, timeout);
+};
 
 // ## Debugging
 // output all errors to the console