From 986471dcbad8eb39fa08f785eb69bb27f9dd3dfb Mon Sep 17 00:00:00 2001
From: Fabien Pinckaers <fp@tinyerp.com>
Date: Wed, 22 Jan 2014 20:23:22 +0100
Subject: [PATCH 1/7] [IMP] design of sign in/up

bzr revid: fp@tinyerp.com-20140122192322-nvtcood5ll0outn9
---
 addons/auth_signup/views/auth_signup_login.xml | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/addons/auth_signup/views/auth_signup_login.xml b/addons/auth_signup/views/auth_signup_login.xml
index 9f5d88eb33a..2b526c38af5 100644
--- a/addons/auth_signup/views/auth_signup_login.xml
+++ b/addons/auth_signup/views/auth_signup_login.xml
@@ -5,12 +5,12 @@
     <data>
         <template id="auth_signup.login" inherit_id="web.login" name="Login (overloaded by auth_signup)">
             <xpath expr="//button[@type='submit']" position="before">
-                <a t-if="signup" t-attf-href="?mode=signup{{ '&amp;debug' if debug else '' }}" class="btn btn-default pull-left">Sign up</a>
-                <a t-if="reset" t-attf-href="?mode=reset{{ '&amp;debug' if debug else '' }}" class="btn btn-default pull-left">Reset Password</a>
+                <a t-if="signup" t-attf-href="?mode=signup{{ '&amp;debug' if debug else '' }}" class="btn btn-link pull-right">Sign up</a>
+                <a t-if="reset" t-attf-href="?mode=reset{{ '&amp;debug' if debug else '' }}" class="btn btn-link pull-right">Reset Password</a>
             </xpath>
         </template>
 
-        <template id="auth_signup.signup" name="Sinup">
+        <template id="auth_signup.signup" name="Sign up">
             <t t-call="web.login_layout">
                 <t t-set="head">
                     <t t-foreach="css" t-as="css_file">
@@ -62,14 +62,13 @@
                         <t t-esc="message"/>
                     </p>
 
-                    <hr/>
                     <input type="hidden" name="redirect" t-att-value="redirect"/>
                     <input type="hidden" name="mode" t-att-value="mode"/>
                     <input type="hidden" name="token" t-att-value="token"/>
                     <div class="clearfix oe_login_buttons">
-                        <a href="/web/login" class="btn btn-default pull-left">Back to Login</a>
-                        <button type="submit" class="btn btn-primary pull-right">
-                            <t t-if="mode == 'signup'">Signup</t>
+                        <a href="/web/login" class="btn btn-link pull-right">Back to Login</a>
+                        <button type="submit" class="btn btn-primary pull-left">
+                            <t t-if="mode == 'signup'">Sign up</t>
                             <t t-if="mode == 'reset'">Reset password</t>
                         </button>
                     </div>

From efcbaa4bd0a4baa1fc00be22f449e47e86278169 Mon Sep 17 00:00:00 2001
From: Antony Lesuisse <al@openerp.com>
Date: Wed, 22 Jan 2014 20:30:59 +0100
Subject: [PATCH 2/7] [FIX] typo

bzr revid: al@openerp.com-20140122193059-rh8rkju5nn8c09kl
---
 addons/auth_signup/auth_signup_data.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/addons/auth_signup/auth_signup_data.xml b/addons/auth_signup/auth_signup_data.xml
index c14a79b417f..f470811d39e 100644
--- a/addons/auth_signup/auth_signup_data.xml
+++ b/addons/auth_signup/auth_signup_data.xml
@@ -10,7 +10,7 @@
             <field name="password"></field>
             <field name="active" eval="False"/>
             <!-- Avoid auto-including this user in any default group, just like a typical portal member -->
-            <field name="groups_id" eval="[(4,ref('base.group_portal')]"/>
+            <field name="groups_id" eval="[(4,ref('base.group_portal'))]"/>
             <!-- allow signuped users to have a alias -->
             <field name="alias_name">_usertemplate</field>
         </record>

From df3a1486f4299d10f8f440ca98808cb0bcd8234f Mon Sep 17 00:00:00 2001
From: Fabien Pinckaers <fp@tinyerp.com>
Date: Wed, 22 Jan 2014 20:35:23 +0100
Subject: [PATCH 3/7] [IMP] remove under construction

bzr revid: fp@tinyerp.com-20140122193523-ni1uzv38plutq41m
---
 addons/auth_signup/views/auth_signup_login.xml |  2 +-
 addons/website/static/src/css/website.css      | 17 -----------------
 addons/website/static/src/css/website.sass     | 15 ---------------
 3 files changed, 1 insertion(+), 33 deletions(-)

diff --git a/addons/auth_signup/views/auth_signup_login.xml b/addons/auth_signup/views/auth_signup_login.xml
index 2b526c38af5..adbb9c2b41e 100644
--- a/addons/auth_signup/views/auth_signup_login.xml
+++ b/addons/auth_signup/views/auth_signup_login.xml
@@ -3,7 +3,7 @@
 -->
 <openerp>
     <data>
-        <template id="auth_signup.login" inherit_id="web.login" name="Login (overloaded by auth_signup)">
+        <template id="auth_signup.login" inherit_id="web.login" name="Sign up - Reset Password">
             <xpath expr="//button[@type='submit']" position="before">
                 <a t-if="signup" t-attf-href="?mode=signup{{ '&amp;debug' if debug else '' }}" class="btn btn-link pull-right">Sign up</a>
                 <a t-if="reset" t-attf-href="?mode=reset{{ '&amp;debug' if debug else '' }}" class="btn btn-link pull-right">Reset Password</a>
diff --git a/addons/website/static/src/css/website.css b/addons/website/static/src/css/website.css
index 170b6b5ff87..b6d41511e5f 100644
--- a/addons/website/static/src/css/website.css
+++ b/addons/website/static/src/css/website.css
@@ -162,27 +162,10 @@ footer {
   background-image: url("/website/static/src/img/drag_here.png");
 }
 
-.oe_structure.oe_empty:empty, [data-oe-type=html]:empty, .oe_structure.oe_empty > .oe_drop_zone.oe_insert:only-child, [data-oe-type=html] > .oe_drop_zone.oe_insert:only-child {
-  background-image: url("/website/static/src/img/under_construction.png");
-  background-repeat: no-repeat;
-  background-position: center;
-  height: 220px !important;
-}
-
 .oe_structure.oe_empty > .oe_drop_zone.oe_insert:only-child, [data-oe-type=html] > .oe_drop_zone.oe_insert:only-child {
   position: static;
 }
 
-.oe_structure.oe_empty:empty:before, [data-oe-type=html]:empty:before, .oe_structure.oe_empty > .oe_drop_zone.oe_insert:only-child:before, [data-oe-type=html] > .oe_drop_zone.oe_insert:only-child:before {
-  content: "Page under construction";
-  text-align: center;
-  display: block;
-  padding-top: 160px;
-  padding-bottom: 30px;
-  color: grey;
-  font-size: 24px;
-}
-
 .oe_structure.oe_editable.oe_empty:empty:before, .oe_editable[data-oe-type=html]:empty:before, .oe_structure.oe_editable.oe_empty > .oe_drop_zone.oe_insert:only-child:before, [data-oe-type=html] > .oe_drop_zone.oe_insert:only-child:before {
   content: "Drag Building Blocks Here";
 }
diff --git a/addons/website/static/src/css/website.sass b/addons/website/static/src/css/website.sass
index 0709047bb02..08fa3609635 100644
--- a/addons/website/static/src/css/website.sass
+++ b/addons/website/static/src/css/website.sass
@@ -122,24 +122,9 @@ footer
 .oe_structure.oe_editable.oe_empty:empty, .oe_editable[data-oe-type=html]:empty, .oe_structure.oe_editable.oe_empty > .oe_drop_zone.oe_insert:only-child, [data-oe-type=html] > .oe_drop_zone.oe_insert:only-child
     background-image: url('/website/static/src/img/drag_here.png')
 
-.oe_structure.oe_empty:empty, [data-oe-type=html]:empty, .oe_structure.oe_empty > .oe_drop_zone.oe_insert:only-child, [data-oe-type=html] > .oe_drop_zone.oe_insert:only-child
-    background-image: url('/website/static/src/img/under_construction.png')
-    background-repeat: no-repeat
-    background-position: center
-    height: 220px !important
-
 .oe_structure.oe_empty > .oe_drop_zone.oe_insert:only-child, [data-oe-type=html] > .oe_drop_zone.oe_insert:only-child
     position: static
 
-.oe_structure.oe_empty:empty:before, [data-oe-type=html]:empty:before, .oe_structure.oe_empty > .oe_drop_zone.oe_insert:only-child:before, [data-oe-type=html] > .oe_drop_zone.oe_insert:only-child:before
-    content: 'Page under construction'
-    text-align: center
-    display: block
-    padding-top: 160px
-    padding-bottom: 30px
-    color: grey
-    font-size: 24px
-
 .oe_structure.oe_editable.oe_empty:empty:before, .oe_editable[data-oe-type=html]:empty:before, .oe_structure.oe_editable.oe_empty > .oe_drop_zone.oe_insert:only-child:before, [data-oe-type=html] > .oe_drop_zone.oe_insert:only-child:before
     content: 'Drag Building Blocks Here'
 

From ef5a50fa98cad1c30a07f3cf8f113d7487e77bd6 Mon Sep 17 00:00:00 2001
From: Fabien Pinckaers <fp@tinyerp.com>
Date: Wed, 22 Jan 2014 20:38:18 +0100
Subject: [PATCH 4/7] [IMP] wording

bzr revid: fp@tinyerp.com-20140122193818-vxn20qaibfvgn1u3
---
 addons/auth_signup/views/auth_signup_login.xml | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/addons/auth_signup/views/auth_signup_login.xml b/addons/auth_signup/views/auth_signup_login.xml
index adbb9c2b41e..9d3d712e58a 100644
--- a/addons/auth_signup/views/auth_signup_login.xml
+++ b/addons/auth_signup/views/auth_signup_login.xml
@@ -31,14 +31,14 @@
 
                 <form class="oe_signup_form" role="form" t-attf-action="/web/login{{ '?debug' if debug else '' }}" method="post">
                     <div class="form-group field-name" t-if="not reset_without_token">
-                        <label for="name" class="control-label">Name</label>
-                        <input type="text" name="name" t-att-value="name" id="name" class="form-control" placeholder="Enter your name"
+                        <label for="name" class="control-label">Your Name</label>
+                        <input type="text" name="name" t-att-value="name" id="name" class="form-control" placeholder="e.g. John Doe"
                             required="required" autofocus="autofocus" t-att-disabled="'disabled' if mode == 'reset' and token else None"/>
                     </div>
 
                     <div class="form-group field-login">
-                        <label for="login" class="control-label">Username (email)</label>
-                        <input type="text" name="login" t-att-value="login" id="login" class="form-control" placeholder="Enter login"
+                        <label for="login" class="control-label">Your Email</label>
+                        <input type="text" name="login" t-att-value="login" id="login" class="form-control"
                             t-att-autofocus="'autofocus' if reset_without_token else None"
                             required="required" t-att-disabled="'disabled' if mode == 'reset' and token else None"/>
                         <input type="hidden" name="login" t-att-value="login" t-if="mode == 'reset' and token"/>
@@ -46,13 +46,13 @@
 
                     <div class="form-group field-password" t-if="not reset_without_token">
                         <label for="password" class="control-label">Password</label>
-                        <input type="password" name="password" id="password" class="form-control" placeholder="Password"
+                        <input type="password" name="password" id="password" class="form-control"
                             required="required" t-att-autofocus="'autofocus' if mode == 'reset' and token else None"/>
                     </div>
 
                     <div class="form-group field-confirm_password" t-if="not reset_without_token">
                         <label for="confirm_password" class="control-label">Confirm Password</label>
-                        <input type="password" name="confirm_password" id="confirm_password" class="form-control" placeholder="Confirm your password" required="required"/>
+                        <input type="password" name="confirm_password" id="confirm_password" class="form-control" required="required"/>
                     </div>
 
                     <p class="alert alert-danger" t-if="error">

From b32fbce49c157c5b4427f397918a0aa9ab1911db Mon Sep 17 00:00:00 2001
From: Fabien Pinckaers <fp@tinyerp.com>
Date: Wed, 22 Jan 2014 20:44:16 +0100
Subject: [PATCH 5/7] [IMP] empty pages

bzr revid: fp@tinyerp.com-20140122194416-veyfgwg4xmjk0w6v
---
 addons/website/static/src/css/website.css  | 7 +++++++
 addons/website/static/src/css/website.sass | 6 ++++++
 2 files changed, 13 insertions(+)

diff --git a/addons/website/static/src/css/website.css b/addons/website/static/src/css/website.css
index b6d41511e5f..ba04fed56f0 100644
--- a/addons/website/static/src/css/website.css
+++ b/addons/website/static/src/css/website.css
@@ -170,6 +170,13 @@ footer {
   content: "Drag Building Blocks Here";
 }
 
+.oe_structure.oe_empty:empty:before, [data-oe-type=html]:empty:before, .oe_structure.oe_empty > .oe_drop_zone.oe_insert:only-child:before, [data-oe-type=html] > .oe_drop_zone.oe_insert:only-child:before {
+  content: " ";
+  display: block;
+  padding-top: 160px;
+  padding-bottom: 30px;
+}
+
 .css_editable_display {
   display: none;
 }
diff --git a/addons/website/static/src/css/website.sass b/addons/website/static/src/css/website.sass
index 08fa3609635..fe85e5bdd4c 100644
--- a/addons/website/static/src/css/website.sass
+++ b/addons/website/static/src/css/website.sass
@@ -128,6 +128,12 @@ footer
 .oe_structure.oe_editable.oe_empty:empty:before, .oe_editable[data-oe-type=html]:empty:before, .oe_structure.oe_editable.oe_empty > .oe_drop_zone.oe_insert:only-child:before, [data-oe-type=html] > .oe_drop_zone.oe_insert:only-child:before
     content: 'Drag Building Blocks Here'
 
+.oe_structure.oe_empty:empty:before, [data-oe-type=html]:empty:before, .oe_structure.oe_empty > .oe_drop_zone.oe_insert:only-child:before, [data-oe-type=html] > .oe_drop_zone.oe_insert:only-child:before
+    content: ' '
+    display: block
+    padding-top: 160px
+    padding-bottom: 30px
+
 .css_editable_display
     display: none
 

From 4b429a033bc33f1738251d02296e2e3437e9d3a9 Mon Sep 17 00:00:00 2001
From: Christophe Matthieu <chm@openerp.com>
Date: Thu, 23 Jan 2014 09:14:11 +0100
Subject: [PATCH 6/7] [IMP] website_sale: extend timeout for shoptest tour

bzr revid: chm@openerp.com-20140123081411-9u5h4720504jdbvf
---
 .../tests/ui_suite/website_sale-sale_process-test.js            | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/addons/website_sale/tests/ui_suite/website_sale-sale_process-test.js b/addons/website_sale/tests/ui_suite/website_sale-sale_process-test.js
index 5554ea75271..5e7c7aa436f 100644
--- a/addons/website_sale/tests/ui_suite/website_sale-sale_process-test.js
+++ b/addons/website_sale/tests/ui_suite/website_sale-sale_process-test.js
@@ -14,5 +14,5 @@ testRunner.run(function websiteSaleTest (page, timeout) {
         page.evaluate(function () {
             window.openerp.website.TestConsole.test('shoptest').run(true);
         });
-    }, timeout/5);
+    }, timeout);
 });
\ No newline at end of file

From ffba2091e78d82692c982430b95a3638952d4367 Mon Sep 17 00:00:00 2001
From: Christophe Matthieu <chm@openerp.com>
Date: Thu, 23 Jan 2014 09:17:21 +0100
Subject: [PATCH 7/7] [IMP] website: ui_test_runer: display report before 'wait
 for' message

bzr revid: chm@openerp.com-20140123081721-gi9y1y5mfneaufy6
---
 addons/website/tests/ui_suite/ui_test_runner.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/addons/website/tests/ui_suite/ui_test_runner.js b/addons/website/tests/ui_suite/ui_test_runner.js
index d3e3132f603..c64cebee7e1 100644
--- a/addons/website/tests/ui_suite/ui_test_runner.js
+++ b/addons/website/tests/ui_suite/ui_test_runner.js
@@ -9,10 +9,10 @@ function waitFor (ready, callback, timeout, timeoutMessageCallback) {
             if(!condition) {
                 var message = timeoutMessageCallback ? timeoutMessageCallback() : "Timeout after "+timeout+" ms";
                 console.log('{ "event": "error", "message": "'+message+'" }');
-                console.log("Waiting for...\n"+ready);
                 if (window.localStorage.getItem("test-report")) {
                     console.log(JSON.parse(website.localStorage.getItem("test-report")));
                 }
+                console.log("Waiting for...\n"+ready);
                 phantom.exit(1);
             } else {
                 clearInterval(interval);