From 02907a768475a7043d673eb8be06f6ebc4b2de42 Mon Sep 17 00:00:00 2001
From: Halfdan Mouritzen <halfdan@robothangarskib.dk>
Date: Sat, 13 Jan 2024 22:14:23 +0100
Subject: [PATCH 1/6] Minimal CSS for tables

---
 src/project/static/css/style.css | 22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/src/project/static/css/style.css b/src/project/static/css/style.css
index 7a6f045..49bb56e 100644
--- a/src/project/static/css/style.css
+++ b/src/project/static/css/style.css
@@ -28,6 +28,7 @@ p, h1, h2, h3, h4, h5, h6 {
   --light : #fff;
   --light-dust : #f6f6f6;
   --dust : #f1f1f1;
+  --medium-dust : #dadada;
   --dark-dust : #bfbfbf;
   --fade : #878787;
   --twilight : #4a4a4a;
@@ -61,6 +62,8 @@ h1, h2, h3, h4, h5, h6 {
 
 a {
   font-weight : 500;
+  color : var(--splash);
+  text-decoration : none;
 }
 
 body {
@@ -249,6 +252,25 @@ button:hover {
   opacity : 1.0;
 }
 
+article table {
+  width : 100%;
+  border-spacing : 0;
+  margin : var(--space) 0;
+}
+
+article table thead th {
+  text-align : left;  
+}
+
+article table tbody tr:nth-child(odd) {
+  background : var(--medium-dust);  
+}
+
+article table thead th,
+article table tbody td {
+  padding : var(--half-space);  
+}
+
 form > div {
   margin : 0 0 var(--double-space);
 }
-- 
2.43.4


From 45467efa9ee467283b33584f039b7328f8cf1ca2 Mon Sep 17 00:00:00 2001
From: Halfdan Mouritzen <halfdan@robothangarskib.dk>
Date: Mon, 15 Jan 2024 13:16:00 +0100
Subject: [PATCH 2/6] Consistency in naming in login

---
 src/project/templates/account/login.html | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/project/templates/account/login.html b/src/project/templates/account/login.html
index cfcac05..523d32b 100644
--- a/src/project/templates/account/login.html
+++ b/src/project/templates/account/login.html
@@ -13,20 +13,20 @@
                     </div>
                 {% endfor %}
             {% endif %}
-            <h2>Log in</h2>
+            <h2>Login</h2>
 
             <form method="post" action="">
                 {% csrf_token %}
                 <div>
                     <label for="id_username"
                            class="visually-hidden">
-                        {% trans "E-mail" %}
+                        {% trans "Email" %}
                     </label>
                     <input type="text"
                            id="id_username"
                            name="login"
                            class="form-control mb-lg-2"
-                           placeholder="{% trans "E-mail" %}"
+                           placeholder="{% trans "Email" %}"
                            required
                            autofocus>
                 </div>
@@ -42,7 +42,7 @@
                            required>
                 </div>
                 <div>
-                    <button type="submit">{% trans "Sign in" %}</button>
+                    <button type="submit">{% trans "Login" %}</button>
                 </div>
             </form>
             <div>
@@ -55,7 +55,7 @@
         <div class="signup">
             <img src="https://data.coop/static/img/logo_da.svg" alt="data.coop logo">
             <div class="new_here">
-                <h2> Are you new here? </h2>
+                <h2>{% trans "Are you new here?" %}</h2>
                 <a class="button" href="{% url "account_signup" %}">{% trans "Become a member" %}</a>
             </div>
         </div>
-- 
2.43.4


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 3/6] 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>
 
-- 
2.43.4


From a0bbc3adc03ec7e960e713387dafab050e551a73 Mon Sep 17 00:00:00 2001
From: Halfdan Mouritzen <halfdan@robothangarskib.dk>
Date: Mon, 15 Jan 2024 19:21:08 +0100
Subject: [PATCH 4/6] Rearranging buttons

---
 src/project/static/css/style.css         |  6 ++++++
 src/project/templates/account/email.html | 16 ++++++++--------
 2 files changed, 14 insertions(+), 8 deletions(-)

diff --git a/src/project/static/css/style.css b/src/project/static/css/style.css
index 0adee7f..9436c33 100644
--- a/src/project/static/css/style.css
+++ b/src/project/static/css/style.css
@@ -267,6 +267,12 @@ button:hover {
   opacity : 1.0;
 }
 
+button:disabled {
+  opacity : 0.6;
+  background : var(--twilight);
+  cursor : default;
+}
+
 article table {
   width : 100%;
   border-spacing : 0;
diff --git a/src/project/templates/account/email.html b/src/project/templates/account/email.html
index a1aaf53..1dc31e9 100644
--- a/src/project/templates/account/email.html
+++ b/src/project/templates/account/email.html
@@ -16,6 +16,12 @@
             <form action="{% url 'account_email' %}" class="email_list" method="post">
                 {% csrf_token %}
                 <fieldset class="blockLabels">
+                    <div class="buttonHolder">
+                        <button class="btn small btn-success" type="submit" name="action_primary" style="float:right">+ Add Email</button>
+                        <button class="btn small btn-success" disabled 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>
                     <table class="table" id="user_email_table">
                         <thead>
                             <tr>
@@ -62,13 +68,6 @@
 
                         </tbody>
                     </table>
-
-                    <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>
-
                 </fieldset>
             </form>
         {% else %}
@@ -77,7 +76,7 @@
                 {% 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 %}
-
+        <!--
         <hr />
 
         <h3>{% trans "Add E-mail" %}</h3>
@@ -91,6 +90,7 @@
                 </button>
             </form>
         </div>
+        -->
     </div>
 
     <script type="text/javascript">
-- 
2.43.4


From 04e4cc0cc651f56a5c573f3d3f103446c99def37 Mon Sep 17 00:00:00 2001
From: Halfdan Mouritzen <halfdan@robothangarskib.dk>
Date: Thu, 18 Jan 2024 09:54:34 +0100
Subject: [PATCH 5/6] Email page : Disable/enable the "Make Primary" button
 according to which email is selected

---
 src/project/templates/account/email.html | 18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

diff --git a/src/project/templates/account/email.html b/src/project/templates/account/email.html
index 1dc31e9..5251e26 100644
--- a/src/project/templates/account/email.html
+++ b/src/project/templates/account/email.html
@@ -17,10 +17,10 @@
                 {% csrf_token %}
                 <fieldset class="blockLabels">
                     <div class="buttonHolder">
-                        <button class="btn small btn-success" type="submit" name="action_primary" style="float:right">+ Add Email</button>
-                        <button class="btn small btn-success" disabled 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>
+                        <button class="small" type="submit" name="action_primary" style="float:right">+ Add Email</button>
+                        <button class="small" disabled type="submit" id="action_primary" name="action_primary">Make Primary</button>
+                        <button class="small" type="submit" name="action_send">Re-send Verification</button>
+                        <button class="small" type="submit" name="action_remove">Remove</button>
                     </div>
                     <table class="table" id="user_email_table">
                         <thead>
@@ -45,6 +45,7 @@
                                                 {% if emailaddress.primary or user.emailaddress_set.count == 1 %}
                                                 checked="checked"
                                                 {% endif %}
+                                                class="{% if emailaddress.primary %}primary_email{% endif %}"
                                             />
                                         </td>
                                         <td>
@@ -104,6 +105,15 @@
                     }
                 });
             }
+
+            let radio_actions = document.getElementsByName('email');
+            if (radio_actions.length) {
+                for (radio of radio_actions) {
+                    radio.addEventListener("change", function (e) {
+                        document.getElementById('action_primary').disabled = e.target.classList.contains('primary_email')
+                    });
+                }
+            }
         })();
     </script>
 
-- 
2.43.4


From 8e7291e5bfc631459dc246ae1b89ed5bece85eba Mon Sep 17 00:00:00 2001
From: Halfdan Mouritzen <halfdan@robothangarskib.dk>
Date: Thu, 18 Jan 2024 10:28:59 +0100
Subject: [PATCH 6/6] CSS : Pagination

---
 src/project/static/css/style.css | 42 ++++++++++++++++++++++++++++++--
 1 file changed, 40 insertions(+), 2 deletions(-)

diff --git a/src/project/static/css/style.css b/src/project/static/css/style.css
index 9436c33..b10458e 100644
--- a/src/project/static/css/style.css
+++ b/src/project/static/css/style.css
@@ -441,10 +441,48 @@ span.time_remaining {
     display : flex;
     justify-content : center;
     list-style : none;
-    padding : 0;
+    padding : var(--half-space) 0;
     margin : 0;
 }
 
 .pagination > li {
-    margin : 0 var(--half-space);
+  margin : 0 var(--half-space);
+}
+
+.pagination > li:first-child {
+  margin-right : var(--double-space);
+}
+
+.pagination > li:last-child {
+  margin-left : var(--double-space);
+}
+
+.pagination .page-item {
+  border : 1px solid var(--fade);  
+  padding : var(--quarter-space) var(--half-space);
+  border-radius : var(--half-space);
+  background : var(--light-dust);
+  font-size : 0.78em;
+}
+
+.pagination .page-link {
+  padding : var(--half-space);
+  color : var(--twilight);
+}
+
+.pagination .page-item.active {
+  background : var(--twilight);
+}
+
+.pagination .page-item.active .page-link {
+  color : var(--light-dust);  
+  font-weight : bold;
+}
+
+.pagination .page-item.disabled {
+  opacity : 0.6;
+}
+
+.pagination .page-item.disabled .page-link {
+  cursor : default;
 }
-- 
2.43.4