From 04d7c55fc65407adbf7fee1bd2b0867c7ce99ccb Mon Sep 17 00:00:00 2001
From: Jef van Schendel <jefvanschendel@gmail.com>
Date: Thu, 26 May 2011 01:24:13 +0200
Subject: [PATCH 1/7] Changed logout/account text

---
 mediagoblin/templates/mediagoblin/base.html | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/mediagoblin/templates/mediagoblin/base.html b/mediagoblin/templates/mediagoblin/base.html
index a0e0730d..9894f27a 100644
--- a/mediagoblin/templates/mediagoblin/base.html
+++ b/mediagoblin/templates/mediagoblin/base.html
@@ -34,9 +34,8 @@
               {% endblock %}{% block mediagoblin_header_title %}MediaGoblin Home{% endblock %}
         <div class="mediagoblin_header_right">
           {% if request.user %}
-                Welcome {{ request.user['username'] }}! --
-                <a href="{{ request.urlgen('mediagoblin.auth.logout') }}">
-                  Logout</a>
+                {{ request.user['username'] }}'s account 
+                (<a href="{{ request.urlgen('mediagoblin.auth.logout') }}">logout</a>)
           {% else %}
                 <a href="{{ request.urlgen('mediagoblin.auth.login') }}">
                   Login</a>

From 491b9109f368f06aa4032cba14518638b6b7cb5c Mon Sep 17 00:00:00 2001
From: Jef van Schendel <jefvanschendel@gmail.com>
Date: Sat, 28 May 2011 18:32:43 +0200
Subject: [PATCH 2/7] Styled forms

---
 mediagoblin/static/css/base.css               | 25 +++++++++++++++++++
 .../templates/mediagoblin/auth/register.html  |  2 +-
 2 files changed, 26 insertions(+), 1 deletion(-)

diff --git a/mediagoblin/static/css/base.css b/mediagoblin/static/css/base.css
index c7d3d4ad..48331554 100644
--- a/mediagoblin/static/css/base.css
+++ b/mediagoblin/static/css/base.css
@@ -84,3 +84,28 @@ label {
     left:0px;
     margin-top:-20px;
 }
+
+/* forms */
+
+.form_box {
+    width:300px;
+    margin-left:auto;
+    margin-right:auto;
+    background-color:#393939;
+    padding:50px 83px 83px;
+    border-top:5px solid #d49086;
+    font-size:18px;
+}
+
+.form_field_input input {
+    width:300px;
+    font-size:18px;
+}
+
+.form_field_box {
+    margin-bottom:20px;
+}
+
+.form_field_label {
+    margin-bottom:4px;
+}
diff --git a/mediagoblin/templates/mediagoblin/auth/register.html b/mediagoblin/templates/mediagoblin/auth/register.html
index 31c3d23e..2ebe99ec 100644
--- a/mediagoblin/templates/mediagoblin/auth/register.html
+++ b/mediagoblin/templates/mediagoblin/auth/register.html
@@ -20,7 +20,7 @@
 {% import "/mediagoblin/utils/wtforms.html" as wtforms_util %}
 
 {% block mediagoblin_content %}
-  <h2>Create an account!</h2>
+  <h1>Create an account!</h1>
 
   <form action="{{ request.urlgen('mediagoblin.auth.register') }}"
         method="POST" enctype="multipart/form-data">

From 9ef7c0f7376ef18e9703722b6730021c95bbdb33 Mon Sep 17 00:00:00 2001
From: Jef van Schendel <jefvanschendel@gmail.com>
Date: Sat, 28 May 2011 20:43:57 +0200
Subject: [PATCH 3/7] Fixed form header

---
 mediagoblin/static/css/base.css                      | 8 +++++++-
 mediagoblin/templates/mediagoblin/auth/register.html | 2 +-
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/mediagoblin/static/css/base.css b/mediagoblin/static/css/base.css
index 48331554..26bf8bd9 100644
--- a/mediagoblin/static/css/base.css
+++ b/mediagoblin/static/css/base.css
@@ -23,6 +23,8 @@ h1 {
     margin-top:40px;
 }
 
+28px
+
 a {
     color: #86D4B1;
 }
@@ -92,11 +94,15 @@ label {
     margin-left:auto;
     margin-right:auto;
     background-color:#393939;
-    padding:50px 83px 83px;
+    padding:0px 83px 83px;
     border-top:5px solid #d49086;
     font-size:18px;
 }
 
+.form_box h1 {
+    font-size:28px;
+}
+
 .form_field_input input {
     width:300px;
     font-size:18px;
diff --git a/mediagoblin/templates/mediagoblin/auth/register.html b/mediagoblin/templates/mediagoblin/auth/register.html
index 2ebe99ec..730d684d 100644
--- a/mediagoblin/templates/mediagoblin/auth/register.html
+++ b/mediagoblin/templates/mediagoblin/auth/register.html
@@ -20,11 +20,11 @@
 {% import "/mediagoblin/utils/wtforms.html" as wtforms_util %}
 
 {% block mediagoblin_content %}
-  <h1>Create an account!</h1>
 
   <form action="{{ request.urlgen('mediagoblin.auth.register') }}"
         method="POST" enctype="multipart/form-data">
     <div class="register_box form_box">
+      <h1>Create an account!</h1>
       {{ wtforms_util.render_divs(register_form) }}
       <div class="form_submit_buttons">
         <input type="submit" value="submit" class="button" />

From 716a0ac3f156459e9c6f6bcaab4769d2c64b4171 Mon Sep 17 00:00:00 2001
From: Jef van Schendel <jefvanschendel@gmail.com>
Date: Sat, 28 May 2011 21:02:34 +0200
Subject: [PATCH 4/7] Styled error messages

---
 mediagoblin/static/css/base.css               | 118 +++++++++---------
 .../templates/mediagoblin/utils/wtforms.html  |  12 +-
 2 files changed, 67 insertions(+), 63 deletions(-)

diff --git a/mediagoblin/static/css/base.css b/mediagoblin/static/css/base.css
index 26bf8bd9..618aaee0 100644
--- a/mediagoblin/static/css/base.css
+++ b/mediagoblin/static/css/base.css
@@ -1,9 +1,9 @@
 body {
-    background-color: #272727;
-    color: #f7f7f7;
-    font-family: sans;
-    padding:none;
-    margin:0px;
+  background-color: #272727;
+  color: #f7f7f7;
+  font-family: sans;
+  padding:none;
+  margin:0px;
 }
 
 /* Carter One font */
@@ -18,15 +18,13 @@ body {
 /* text styles */
 
 h1 {
-    font-family: 'Carter One', arial, serif;
-    margin-bottom: 20px;
-    margin-top:40px;
+  font-family: 'Carter One', arial, serif;
+  margin-bottom: 20px;
+  margin-top:40px;
 }
 
-28px
-
 a {
-    color: #86D4B1;
+  color: #86D4B1;
 }
 
 label {
@@ -36,82 +34,90 @@ label {
 /* website structure */
 
 .mediagoblin_header {
-    width:100%;
-    height:36px;
-    background-color:#393939;
-    padding-top:14px;
-    margin-bottom:40px;
+  width:100%;
+  height:36px;
+  background-color:#393939;
+  padding-top:14px;
+  margin-bottom:40px;
 }
 
 .icon {
-    vertical-align:middle;
-    margin-right:10px;
+  vertical-align:middle;
+  margin-right:10px;
 }
 
 .mediagoblin_container {
-    width: 960px;
-    margin-left: auto;
-    margin-right: auto;
+  width: 960px;
+  margin-left: auto;
+  margin-right: auto;
 }
 
 .mediagoblin_header_right {
-    float:right;
+  float:right;
 }
 
 .button {
-    font-family:'Carter One', arial, serif;
-    height:32px;
-    min-width:99px;
-    background-color:#86d4b1;
-    box-shadow:0px 0px 4px #000;
-    border-radius:5px;
-    border:none;
-    color:#272727;
-    margin:10px;
-    font-size:1em;
-    float:left;
-    display:block;
-    text-align:center;
-    padding-left:11px;
-    padding-right:11px;
+  font-family:'Carter One', arial, serif;
+  height:32px;
+  min-width:99px;
+  background-color:#86d4b1;
+  box-shadow:0px 0px 4px #000;
+  border-radius:5px;
+  border:none;
+  color:#272727;
+  margin:10px;
+  font-size:1em;
+  float:left;
+  display:block;
+  text-align:center;
+  padding-left:11px;
+  padding-right:11px;
 }
 
 /* common website elements */
 
 .dotted_line {
-    width:100%;
-    height:0px;
-    border-bottom: dotted 1px #5f5f5f;
-    position:absolute;
-    left:0px;
-    margin-top:-20px;
+  width:100%;
+  height:0px;
+  border-bottom: dotted 1px #5f5f5f;
+  position:absolute;
+  left:0px;
+  margin-top:-20px;
 }
 
 /* forms */
 
 .form_box {
-    width:300px;
-    margin-left:auto;
-    margin-right:auto;
-    background-color:#393939;
-    padding:0px 83px 83px;
-    border-top:5px solid #d49086;
-    font-size:18px;
+  width:300px;
+  margin-left:auto;
+  margin-right:auto;
+  background-color:#393939;
+  padding:0px 83px 83px;
+  border-top:5px solid #d49086;
+  font-size:18px;
 }
 
 .form_box h1 {
-    font-size:28px;
+  font-size:28px;
 }
 
 .form_field_input input {
-    width:300px;
-    font-size:18px;
+  width:300px;
+  font-size:18px;
 }
 
 .form_field_box {
-    margin-bottom:20px;
+  margin-bottom:24px;
 }
 
-.form_field_label {
-    margin-bottom:4px;
+.form_field_label,.form_field_input {
+  margin-bottom:4px;
+}
+
+.form_field_error {
+  background-color:#87453b;
+  border:none;
+  font-size:16px;
+  padding:9px;
+  margin-top:8px;
 }
diff --git a/mediagoblin/templates/mediagoblin/utils/wtforms.html b/mediagoblin/templates/mediagoblin/utils/wtforms.html
index 4a37ab33..9adf8e53 100644
--- a/mediagoblin/templates/mediagoblin/utils/wtforms.html
+++ b/mediagoblin/templates/mediagoblin/utils/wtforms.html
@@ -26,13 +26,11 @@
       {%- endif %}
       <div class="form_field_input">{{ field }}</div>
       {%- if field.errors -%}
-        <div class="form_field_errors">
-          <ul>
-            {% for error in field.errors %}
-              <li>{{ error }}</li>
-            {% endfor %}
-          </ul>
-        </div>
+        {% for error in field.errors %}
+          <div class="form_field_error">
+            {{ error }}
+          </div>
+        {% endfor %}
       {%- endif %}
     </div>
   {% endfor %}

From b0ff25c82cdf424a54b388bb084cfd6849683662 Mon Sep 17 00:00:00 2001
From: Jef van Schendel <jefvanschendel@gmail.com>
Date: Sat, 28 May 2011 21:10:54 +0200
Subject: [PATCH 5/7] Changed login forms to use divs instead of tables

---
 .../templates/mediagoblin/auth/login.html     | 30 ++++++++-----------
 1 file changed, 13 insertions(+), 17 deletions(-)

diff --git a/mediagoblin/templates/mediagoblin/auth/login.html b/mediagoblin/templates/mediagoblin/auth/login.html
index 02bfb91f..27a3e52a 100644
--- a/mediagoblin/templates/mediagoblin/auth/login.html
+++ b/mediagoblin/templates/mediagoblin/auth/login.html
@@ -20,25 +20,21 @@
 {% import "/mediagoblin/utils/wtforms.html" as wtforms_util %}
 
 {% block mediagoblin_content %}
-  <h1>Login:</h1>
 
   <form action="{{ request.urlgen('mediagoblin.auth.login') }}"
         method="POST" enctype="multipart/form-data">
-
-    {% if login_failed %}
-      <p><i>Login failed!</i></p>
-    {% endif %}
-
-    <table>
-      {{ wtforms_util.render_table(login_form) }}
-      <tr>
-        <td></td>
-        <td><input type="submit" value="submit" class="button"/></td>
-      </tr>
-    </table>
-
-    {% if next %}
-      <input type="hidden" name="next" value="{{ next }}" class="button" />
-    {% endif %}
+    <div class="login_box form_box">
+      <h1>Log in</h1>
+      {% if login_failed %}
+        <div class="form_field_error">Login failed!</div>
+      {% endif %}
+      {{ wtforms_util.render_divs(login_form) }}
+      <div class="form_submit_buttons">
+        <input type="submit" value="submit" class="button"/>
+      </div>
+      {% if next %}
+        <input type="hidden" name="next" value="{{ next }}" class="button" />
+      {% endif %}
+    </div>
   </form>
 {% endblock %}

From 7c2005b6e2fdc9665ffc5b443f6450f9d5b34b16 Mon Sep 17 00:00:00 2001
From: Jef van Schendel <jefvanschendel@gmail.com>
Date: Sat, 28 May 2011 21:15:15 +0200
Subject: [PATCH 6/7] Forgot a bottom-margin for the error div

---
 mediagoblin/static/css/base.css | 1 +
 1 file changed, 1 insertion(+)

diff --git a/mediagoblin/static/css/base.css b/mediagoblin/static/css/base.css
index 618aaee0..fbe84c9c 100644
--- a/mediagoblin/static/css/base.css
+++ b/mediagoblin/static/css/base.css
@@ -120,4 +120,5 @@ label {
   font-size:16px;
   padding:9px;
   margin-top:8px;
+  margin-bottom:8px;
 }

From b5cc7afea210cb1fc2e607f75d4922136ba4ecb5 Mon Sep 17 00:00:00 2001
From: Jef van Schendel <jefvanschendel@gmail.com>
Date: Sat, 28 May 2011 21:32:12 +0200
Subject: [PATCH 7/7] Fixed submit button style, added create-account notice to
 log in page

---
 mediagoblin/static/css/base.css                   | 8 ++++++--
 mediagoblin/templates/mediagoblin/auth/login.html | 1 +
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/mediagoblin/static/css/base.css b/mediagoblin/static/css/base.css
index fbe84c9c..672cc04c 100644
--- a/mediagoblin/static/css/base.css
+++ b/mediagoblin/static/css/base.css
@@ -23,6 +23,11 @@ h1 {
   margin-top:40px;
 }
 
+p {
+  font-family: sans;
+  font-size:16px;
+}
+
 a {
   color: #86D4B1;
 }
@@ -67,7 +72,6 @@ label {
   color:#272727;
   margin:10px;
   font-size:1em;
-  float:left;
   display:block;
   text-align:center;
   padding-left:11px;
@@ -92,7 +96,7 @@ label {
   margin-left:auto;
   margin-right:auto;
   background-color:#393939;
-  padding:0px 83px 83px;
+  padding:0px 83px 30px 83px;
   border-top:5px solid #d49086;
   font-size:18px;
 }
diff --git a/mediagoblin/templates/mediagoblin/auth/login.html b/mediagoblin/templates/mediagoblin/auth/login.html
index 27a3e52a..22a57b70 100644
--- a/mediagoblin/templates/mediagoblin/auth/login.html
+++ b/mediagoblin/templates/mediagoblin/auth/login.html
@@ -35,6 +35,7 @@
       {% if next %}
         <input type="hidden" name="next" value="{{ next }}" class="button" />
       {% endif %}
+      <p>Don't have an account yet? <a href="{{ request.urlgen('mediagoblin.auth.register') }}">Create one here!</a></p>
     </div>
   </form>
 {% endblock %}