From 60d0503dbf8306883cc051e4f499e9f99c3d3ba8 Mon Sep 17 00:00:00 2001
From: Halfdan Mouritzen <halfdan@robothangarskib.dk>
Date: Mon, 15 Jan 2024 18:28:11 +0100
Subject: [PATCH] Colours, spacing, sizes

---
 src/project/static/css/style.css         |  92 ++++++++++--
 src/project/templates/account/email.html | 178 ++++++++++-------------
 2 files changed, 155 insertions(+), 115 deletions(-)

diff --git a/src/project/static/css/style.css b/src/project/static/css/style.css
index 49bb56e..0adee7f 100644
--- a/src/project/static/css/style.css
+++ b/src/project/static/css/style.css
@@ -25,15 +25,17 @@ p, h1, h2, h3, h4, h5, h6 {
 /* Variables */
 :root {
   /* Colors */
-  --light : #fff;
-  --light-dust : #f6f6f6;
-  --dust : #f1f1f1;
+  --light : #ffffff;
+  --light-dust : #fefef9;
+  --dust : #f4f1ef;
   --medium-dust : #dadada;
   --dark-dust : #bfbfbf;
   --fade : #878787;
   --twilight : #4a4a4a;
   --dark : #2a2a2a;
+  --light-custard : #eee7d5;
   --custard : #f0dcac;
+  --dark-custard : #d4c7a9; 
   --water : #a8f3f4;
   --splash : #4b3aba;
 
@@ -53,6 +55,7 @@ p, h1, h2, h3, h4, h5, h6 {
 
 html, body {
   height : 100%;
+  font-size : 1.05em;
 }
 
 h1, h2, h3, h4, h5, h6 {
@@ -64,6 +67,14 @@ a {
   font-weight : 500;
   color : var(--splash);
   text-decoration : none;
+  cursor : pointer;
+}
+
+hr {
+  margin : var(--double-space) 0;
+  height : 0;
+  border : 0;
+  border-bottom : 1px solid var(--dark-custard);  
 }
 
 body {
@@ -102,7 +113,7 @@ header > a.logout:hover {
 }
 
 aside {
-  padding : var(--double-space) var(--outer-space);
+  padding : 0 var(--outer-space) var(--double-space) var(--outer-space);
   background : var(--light);
 }
 
@@ -141,7 +152,7 @@ aside > div > dl > dt {
 
 nav {
   display : block;
-  border-bottom : 1px solid var(--dark-dust);
+  border-bottom : 1px solid var(--dark-custard);
   background : var(--light);
 }
 
@@ -188,14 +199,14 @@ article {
   padding : var(--double-space) var(--outer-space);
 }
 
-article > div {
+article > div.content-view {
   background : var(--dust);
   padding : var(--double-space);
-  margin-bottom : var(--double-space);
+  margin-bottom : var(--space);
 }
 
 div.content-view > h2 {
-  margin : 0 0 var(--double-space) 0;
+  margin : 0 0 var(--space) 0;
 }
 
 div.services {
@@ -236,16 +247,20 @@ button {
   color : var(--light);
   background : var(--splash);
   padding : var(--space) var(--double-space);
-  border-radius : 3px;
-  opacity : 0.9;
+  border-radius : var(--quarter-space);
+  opacity : 0.85;
   cursor : pointer;
   text-align : center;
   border : 0;
   font-weight : 600;
-  letter-spacing : 0.03em;
   text-decoration : none;
 }
 
+button.small {
+  font-size : 0.78em;  
+  padding : var(--half-space) var(--space);
+}
+
 div.services > div > a:hover,
 a.button:hover,
 button:hover {
@@ -259,16 +274,47 @@ article table {
 }
 
 article table thead th {
-  text-align : left;  
+  background : var(--twilight);
+  color : var(--medium-dust);
+}
+
+article table thead th a {
+  color : var(--light);
+}
+
+article table thead th:first-child {
+  border-radius : var(--half-space) 0 0 0;
+}
+
+article table thead th:last-child {
+  border-radius : 0 var(--half-space) 0 0;
+}
+
+article table tbody {
+  background : var(--light-dust);
 }
 
 article table tbody tr:nth-child(odd) {
-  background : var(--medium-dust);  
+  background : var(--light-custard);  
 }
 
+article table tbody tr:nth-child(odd) td {
+  border-top : 1px solid var(--dark-custard);
+  border-bottom : 1px solid var(--dark-custard);
+}
+
+article table tbody tr:last-child td {
+  border-bottom : var(--half-space) solid var(--twilight);
+}
+ 
 article table thead th,
 article table tbody td {
-  padding : var(--half-space);  
+  padding : var(--space);  
+  text-align : left;
+}
+
+article table#user_email_table tbody tr td:first-child {
+  text-align : center;
 }
 
 form > div {
@@ -281,7 +327,8 @@ form > div >label {
 }
 
 form > div > input[type="text"],
-form > div > input[type="password"] {
+form > div > input[type="password"],
+input[type="email"] {
   border : 2px solid var(--twilight);
   border-radius : 6px;
   padding : 8px;
@@ -289,6 +336,19 @@ form > div > input[type="password"] {
   width : 100%;
 }
 
+form fieldset {
+  border : 0;
+  padding : 0;
+  margin : 0;  
+}
+
+form div.buttonHolder {
+}
+
+form div.buttonHolder button {
+  display : inline-block;
+}
+
 #login {
   height : 100%;
   display : flex;
@@ -380,5 +440,5 @@ span.time_remaining {
 }
 
 .pagination > li {
-    margin : 0 6px;
+    margin : 0 var(--half-space);
 }
diff --git a/src/project/templates/account/email.html b/src/project/templates/account/email.html
index 9010491..a1aaf53 100644
--- a/src/project/templates/account/email.html
+++ b/src/project/templates/account/email.html
@@ -2,114 +2,94 @@
 
 {% load i18n %}
 
-{% block head_title %}{% trans "E-mail Addresses" %}{% endblock %}
-
-
+{% block head_title %}{% trans "Email Addresses" %}{% endblock %}
 
 {% block content %}
+    <div class="content-view">
+        <h2>{% trans "Email Addresses" %}</h2>
+        <p>Her kan du tilføje og ændre emailadresser tilknyttet din konto.</p>
+        <p>{% trans 'The following email addresses are associated with your account:' %}</p>
 
-    <div class="row">
-        <div class="col-md-12">
+        <hr />
 
-            <div class="panel panel-default">
-                <div class="panel-heading">
-                    <h4>{% trans "E-mail Addresses" %}</h4>
-                </div>
-                <div class="panel-body">
+        {% if user.emailaddress_set.all %}
+            <form action="{% url 'account_email' %}" class="email_list" method="post">
+                {% csrf_token %}
+                <fieldset class="blockLabels">
+                    <table class="table" id="user_email_table">
+                        <thead>
+                            <tr>
+                                <th></th>
+                                <th>{% trans "Address" %}</th>
+                                <th>{% trans "Status" %}</th>
+                                <th>{% trans "Primary" %}</th>
+                            </tr>
+                        </thead>
+                        <tbody>
+                            {% for emailaddress in user.emailaddress_set.all %}
+                                <tr>
+                                    <label for="email_radio_{{ forloop.counter }}"
+                                           class="{% if emailaddress.primary %}primary_email{% endif %}">
+                                        <td>
+                                            <input
+                                                id="email_radio_{{ forloop.counter }}"
+                                                type="radio"
+                                                name="email"
+                                                value="{{ emailaddress.email }}"
+                                                {% if emailaddress.primary or user.emailaddress_set.count == 1 %}
+                                                checked="checked"
+                                                {% endif %}
+                                            />
+                                        </td>
+                                        <td>
+                                            {{ emailaddress.email }}
+                                        </td>
+                                        <td>
+                                            {% if emailaddress.verified %}
+                                                <span class="label label-success">{% trans "Verified" %}</span>
+                                            {% else %}
+                                                <span class="label label-danger">{% trans "Unverified" %}</span>
+                                            {% endif %}
+                                        </td>
+                                        <td>
+                                            {% if emailaddress.primary %}
+                                                <span class="label label-primary">{% trans "Primary" %}</span>
+                                            {% endif %}
+                                        </td>
+                                    </label>
+                                </tr>
+                            {% endfor %}
 
-                    {% if user.emailaddress_set.all %}
-                        <p>{% trans 'The following e-mail addresses are associated with your account:' %}</p>
-                        <form action="{% url 'account_email' %}" class="email_list"
-                              method="post">
-                            {% csrf_token %}
-                            <fieldset class="blockLabels">
+                        </tbody>
+                    </table>
 
-                                <table class="table">
-                                    <thead>
-                                    <tr>
-                                        <th></th>
-                                        <th>{% trans "Address" %}</th>
-                                        <th>{% trans "Status" %}</th>
-                                        <th>{% trans "Primary" %}</th>
-                                    </tr>
-                                    </thead>
-                                    <tbody>
+                    <div class="buttonHolder">
+                        <button class="btn small btn-success" type="submit" name="action_primary">Make Primary</button>
+                        <button class="btn small btn-primary" type="submit" name="action_send">Re-send Verification</button>
+                        <button class="btn small btn-danger" type="submit" name="action_remove">Remove</button>
+                    </div>
 
-                                    {% for emailaddress in user.emailaddress_set.all %}
-                                        <tr class="ctrlHolder">
-                                            <label for="email_radio_{{ forloop.counter }}"
-                                                   class="{% if emailaddress.primary %}primary_email{% endif %}">
-                                                <td>
-                                                    <input
-                                                            id="email_radio_{{ forloop.counter }}"
-                                                            type="radio"
-                                                            name="email"
-                                                            value="{{ emailaddress.email }}"
-                                                            {% if emailaddress.primary or user.emailaddress_set.count == 1 %}
-                                                            checked="checked"
-                                                            {% endif %}
-                                                    />
-                                                </td>
-                                                <td>
-                                                    {{ emailaddress.email }}
-                                                </td>
-                                                <td>
-                                                    {% if emailaddress.verified %}
-                                                        <span class="label label-success">Verified</span>
-                                                    {% else %}
-                                                        <span class="label label-danger">Unverified</span>
-                                                    {% endif %}
-                                                </td>
-                                                <td>
-                                                    {% if emailaddress.primary %}
-                                                        <span class="label label-primary">Primary</span>{% endif %}
-                                                </td>
-                                            </label>
-                                        </tr>
-                                    {% endfor %}
+                </fieldset>
+            </form>
+        {% else %}
+            <p>
+                <strong>{% trans 'Warning:' %}</strong>
+                {% trans "You currently do not have any e-mail address set up. You should really add an e-mail address so you can receive notifications, reset your password, etc." %}
+            </p>
+        {% endif %}
 
-                                    </tbody>
-                                </table>
-
-                                <div class="buttonHolder">
-                                    <button class="btn btn-success" type="submit"
-                                            name="action_primary">Make Primary
-                                    </button>
-                                    <button class="btn btn-primary" type="submit"
-                                            name="action_send">Re-send Verification
-                                    </button>
-                                    <button class="btn btn-danger" type="submit"
-                                            name="action_remove">Remove
-                                    </button>
-                                </div>
-
-                            </fieldset>
-                        </form>
-                    {% else %}
-                        <p>
-                            <strong>{% trans 'Warning:' %}</strong>
-                            {% trans "You currently do not have any e-mail address set up. You should really add an e-mail address so you can receive notifications, reset your password, etc." %}
-                        </p>
-                    {% endif %}
-                </div>
-            </div>
-
-            <div class="panel panel-default">
-                <div class="panel-heading">
-                    <h4>{% trans "Add E-mail" %}</h4>
-                </div>
-                <div class="panel-body">
-                    <form method="post" action="{% url 'account_email' %}"
-                          class="add_email">
-                        {% csrf_token %}
-                        {{ form.as_p }}
-                        <button name="action_add" class="btn btn-success" type="submit">
-                            {% trans "Add E-mail" %}
-                        </button>
-                    </form>
-                </div>
-            </div>
+        <hr />
 
+        <h3>{% trans "Add E-mail" %}</h3>
+        <div class="panel-body">
+            <form method="post" action="{% url 'account_email' %}"
+                  class="add_email">
+                {% csrf_token %}
+                {{ form.as_p }}
+                <button name="action_add" class="btn btn-success" type="submit">
+                    {% trans "Add E-mail" %}
+                </button>
+            </form>
         </div>
     </div>