From 9406bbad978def8e364a1574642cbc6a4d1e9a74 Mon Sep 17 00:00:00 2001
From: W13R <9070224-W13R@users.noreply.gitlab.com>
Date: Sat, 14 May 2022 11:28:32 +0200
Subject: [PATCH 1/7] Revised some UI elements

---
 application/app/templates/baseLayout.html |  2 -
 application/app/templates/userPanel.html  | 55 ++++++++++++-----------
 static/css/deposit.css                    |  8 ----
 static/css/login.css                      |  1 -
 static/css/main.css                       | 49 +++++++++-----------
 static/css/order.css                      |  8 ----
 6 files changed, 51 insertions(+), 72 deletions(-)

diff --git a/application/app/templates/baseLayout.html b/application/app/templates/baseLayout.html
index 1ef2823..ced9bb8 100644
--- a/application/app/templates/baseLayout.html
+++ b/application/app/templates/baseLayout.html
@@ -22,9 +22,7 @@
 
         {% if user.is_authenticated %}
 
-        <div class="userPanel">
             {% include "userPanel.html" %}
-        </div>
 
         {% endif %}
 
diff --git a/application/app/templates/userPanel.html b/application/app/templates/userPanel.html
index 61c0d9e..2ec16c2 100644
--- a/application/app/templates/userPanel.html
+++ b/application/app/templates/userPanel.html
@@ -1,31 +1,36 @@
 {% load i18n %}
 
-<div class="dropDownMenu" id="dropDownMenu">
-    <button class="dropDownButton" id="dropDownMenuButton">
-        <div>
-            {% if user.first_name != "" %}
-                {% translate "User" %}: {{ user.first_name }} {{ user.last_name }} ({{ user.username }})
-            {% else %}
-                {% translate "User" %}: {{ user.username }}
-            {% endif %}
-            &nbsp;-&nbsp;
-            {% if user.balance < 0.01 %}
-                <span class="userBalanceWarn">{% translate "Balance" %}: {{ user.balance }}{{ currency_suffix }}</span>
-            {% else %}
-                <span>{% translate "Balance" %}: {{ user.balance }}{{ currency_suffix }}</span>
-            {% endif %}
-        </div>
-    </button>
-    <div class="dropDownList">
-        <a class="button dropDownChoice" id="navBarBtnHome" href="/">Home</a>
-        <a class="button dropDownChoice" id="navBarBtnHistory" href="/history">{% translate "History" %}</a>
-        <a class="button dropDownChoice" id="navBarBtnStatistics" href="/statistics">{% translate "Statistics" %}</a>
-        <a class="button dropDownChoice" id="navBarBtnDeposit" href="/deposit">{% translate "Deposit" %}</a>
-        {% if user.is_superuser %}
-            <a class="button dropDownChoice" href="/admin/">Admin Panel</a>
+<div class="userPanel">
+    <div class="userInfo">
+        {% if user.first_name != "" %}
+            {% translate "User" %}: {{ user.first_name }} {{ user.last_name }} ({{ user.username }})
         {% else %}
-        <a class="button dropDownChoice" href="/accounts/password_change/">{% translate "Change Password" %}</a>
+            {% translate "User" %}: {{ user.username }}
         {% endif %}
-        <a class="button dropDownChoice" href="/accounts/logout">{% translate "Logout" %}</a>
+        &nbsp;-&nbsp;
+        {% if user.balance < 0.01 %}
+            <span class="userBalanceWarn">{% translate "Balance" %}: {{ user.balance }}{{ currency_suffix }}</span>
+        {% else %}
+            <span>{% translate "Balance" %}: {{ user.balance }}{{ currency_suffix }}</span>
+        {% endif %}
+    </div>
+    <div class="horizontalButtonList">
+        <a class="button" id="navBarBtnHome" href="/">Home</a>
+        <a class="button" id="navBarBtnDeposit" href="/deposit">{% translate "Deposit" %}</a>
+        <div class="dropDownMenu" id="dropDownMenu">
+            <button class="dropDownButton" id="dropDownMenuButton">
+                <div>{% translate "Account" %}</div>
+            </button>
+            <div class="dropDownList">
+                <a class="button dropDownChoice" id="navBarBtnHistory" href="/history">{% translate "History" %}</a>
+                <a class="button dropDownChoice" id="navBarBtnStatistics" href="/statistics">{% translate "Statistics" %}</a>
+                {% if user.is_superuser %}
+                    <a class="button dropDownChoice" href="/admin/">Admin Panel</a>
+                {% else %}
+                    <a class="button dropDownChoice" href="/accounts/password_change/">{% translate "Change Password" %}</a>
+                {% endif %}
+                <a class="button dropDownChoice" href="/accounts/logout">{% translate "Logout" %}</a>
+            </div>
+        </div>
     </div>
 </div>
diff --git a/static/css/deposit.css b/static/css/deposit.css
index 8cb4e1c..3b4ff50 100644
--- a/static/css/deposit.css
+++ b/static/css/deposit.css
@@ -1,11 +1,3 @@
 #depositAmount {
     width: 10rem;
-}
-main {
-    margin-top: 0;
-}
-@media only screen and (max-width: 700px) {
-    main {
-        margin-top: -15vh;
-    }
 }
\ No newline at end of file
diff --git a/static/css/login.css b/static/css/login.css
index 22f8b43..0b18a9f 100644
--- a/static/css/login.css
+++ b/static/css/login.css
@@ -54,7 +54,6 @@ main > h1 {
     flex-direction: column;
     justify-content: start;
     align-items: center;
-    margin-top: 10vh;
 }
 .passwordOverlay > form {
     min-width: unset;
diff --git a/static/css/main.css b/static/css/main.css
index 5c5300c..298dd35 100644
--- a/static/css/main.css
+++ b/static/css/main.css
@@ -58,20 +58,23 @@ main {
     align-items: center;
     flex-grow: 1;
     width: 100%;
-    margin-top: calc(-14rem + 2vh);
+    margin-top: 2rem;
 }
 .userPanel {
-    width: 100%;
     display: flex;
     flex-direction: row;
     justify-content: center;
-    align-items: flex-start;
-    min-width: fit-content;
+    align-items: center;
     margin-top: 1rem;
-    pointer-events: none;
+    font-size: 1rem;
+    width: 90%;
 }
-.userPanel > div {
-    margin: 0 1rem;
+.userInfo span {
+    font-size: 1.1rem;
+}
+.userPanel > .horizontalButtonList {
+    margin-left: auto;
+    margin-right: 0;
 }
 .userBalanceWarn {
     color: var(--color-error);
@@ -118,25 +121,19 @@ main > h1 {
     box-shadow: none;
     text-align: center;
     justify-content: center;
-    pointer-events: all;
 }
 .dropDownButton, .dropDownChoice {
     font-size: 1rem;
 }
-.dropDownButton > div::after {
-    content: '\25BC';
-    display: inline-block;
-    transition: transform 100ms;
-    padding: 0 .3rem;
-}
 .dropDownList {
+    position: absolute;
     display: flex;
     flex-direction: column;
     pointer-events: none;
     border-radius: var(--glass-corner-radius) !important;
     backdrop-filter: var(--glass-blur);
     z-index: 200;
-    margin-top: .5rem;
+    margin-top: 3.2rem;
     opacity: 0%;
     transition: opacity 100ms;
 }
@@ -166,12 +163,6 @@ main > h1 {
     visibility: visible;
     pointer-events: visible;
 }
-.dropDownVisible > .dropDownButton > div::after {
-    transform: rotate(180deg);
-}
-.userPanel .dropDownButton, .userPanel .dropDownChoice {
-    font-size: 1.1rem;
-}
 /* FOOTER */
 .footer {
     z-index: 990;
@@ -215,8 +206,8 @@ tr:nth-child(2n+2) {
 }
 /*
 Rounded corners on table cells apparently don't work with
-Firefox (91), so Firefox users won't have rounded corners
-on tables. Won't fix that by myself.
+Firefox, so Firefox users won't have rounded corners
+on tables. Can't fix that by myself.
 */
 table tr:first-child th:first-child {
     border-top-left-radius: var(--glass-corner-radius);
@@ -285,7 +276,6 @@ form .button, form button {
     align-items: center;
     justify-content: center;
     font-family: var(--font-family);
-    font-size: .9rem;
     text-decoration: none;
     text-align: center !important;
     background: var(--glass-bg);
@@ -341,7 +331,9 @@ input[type="text"], input[type="password"], input[type="number"] {
     flex-direction: row;
     align-items: flex-end;
     justify-content: space-between;
-    width: 100%;
+}
+.horizontalButtonList > .button, .horizontalButtonList > button, .horizontalButtonList > div {
+    margin: 0 .5rem;
 }
 .errorText {
     margin-top: 1rem;
@@ -365,8 +357,9 @@ h1 {
         justify-content: start;
         align-items: center;
     }
-    .userPanel > div {
-        margin: 0;
-        margin-bottom: .5rem;
+    .userPanel > .horizontalButtonList {
+        margin-right: 0;
+        margin-left: 0;
+        margin-top: .5rem;
     }
 }
\ No newline at end of file
diff --git a/static/css/order.css b/static/css/order.css
index 9c15743..3d2685b 100644
--- a/static/css/order.css
+++ b/static/css/order.css
@@ -1,11 +1,3 @@
-main {
-    margin-top: 0;
-}
 form {
     width: 22rem;
-}
-@media only screen and (max-width: 700px) {
-    main {
-        margin-top: -15vh;
-    }
 }
\ No newline at end of file
-- 
2.43.0


From f8cde7136d6fd93d83e1586e6e11d461602a23de Mon Sep 17 00:00:00 2001
From: W13R <9070224-W13R@users.noreply.gitlab.com>
Date: Sun, 15 May 2022 11:24:04 +0200
Subject: [PATCH 2/7] Remove DjangoCSP dependency by writing own middleware for
 the CSP header

---
 application/app/middleware.py          | 11 +++++++++++
 application/drinks_manager/settings.py |  2 +-
 pip-dependencies.txt                   |  1 -
 3 files changed, 12 insertions(+), 2 deletions(-)
 create mode 100644 application/app/middleware.py

diff --git a/application/app/middleware.py b/application/app/middleware.py
new file mode 100644
index 0000000..43acf6c
--- /dev/null
+++ b/application/app/middleware.py
@@ -0,0 +1,11 @@
+
+# Define CSP middleware:
+
+def csp_middleware(get_response):
+
+    def middleware(request):
+        response = get_response(request)
+        response["content-security-policy"] = "default-src 'self'"
+        return response
+
+    return middleware
diff --git a/application/drinks_manager/settings.py b/application/drinks_manager/settings.py
index 2a6674d..6dc0a14 100644
--- a/application/drinks_manager/settings.py
+++ b/application/drinks_manager/settings.py
@@ -67,7 +67,7 @@ MIDDLEWARE = [
     'django.contrib.messages.middleware.MessageMiddleware',
     'django.middleware.clickjacking.XFrameOptionsMiddleware',
     "django_currentuser.middleware.ThreadLocalUserMiddleware",
-    "csp.middleware.CSPMiddleware"
+    "app.middleware.csp_middleware"
 ]
 
 ROOT_URLCONF = 'drinks_manager.urls'
diff --git a/pip-dependencies.txt b/pip-dependencies.txt
index 3107a0b..da50c9d 100644
--- a/pip-dependencies.txt
+++ b/pip-dependencies.txt
@@ -1,5 +1,4 @@
 django~=3.2.7
 django-currentuser==0.5.3
-django-csp==3.7
 psycopg2~=2.9.1
 uvicorn~=0.17.6
-- 
2.43.0


From aa0f2d67edeb4ee32122fa35c35bf629705f7dea Mon Sep 17 00:00:00 2001
From: W13R <9070224-W13R@users.noreply.gitlab.com>
Date: Sun, 15 May 2022 11:45:46 +0200
Subject: [PATCH 3/7] Add activate-devel-env.sh to lib

---
 lib/activate-devel-env.sh | 6 ++++++
 1 file changed, 6 insertions(+)
 create mode 100755 lib/activate-devel-env.sh

diff --git a/lib/activate-devel-env.sh b/lib/activate-devel-env.sh
new file mode 100755
index 0000000..3467721
--- /dev/null
+++ b/lib/activate-devel-env.sh
@@ -0,0 +1,6 @@
+#!/usr/bin/env bash
+
+source ./lib/env.sh
+source ./config/config.sh
+export DJANGO_DEBUG=true
+export PYTHONPATH="./packages"
-- 
2.43.0


From d792a7658c18204c527c2c793b877f8314df61f4 Mon Sep 17 00:00:00 2001
From: W13R <9070224-W13R@users.noreply.gitlab.com>
Date: Mon, 16 May 2022 10:51:22 +0200
Subject: [PATCH 4/7] Fixed a bug in the template for the order page caused by
 localization and removed the previously applied workaround for this issue

---
 application/app/templates/order.html | 3 ++-
 static/css/main.css                  | 2 +-
 static/js/order.js                   | 2 +-
 3 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/application/app/templates/order.html b/application/app/templates/order.html
index adbb8e3..3c99638 100644
--- a/application/app/templates/order.html
+++ b/application/app/templates/order.html
@@ -1,6 +1,7 @@
 {% extends "baseLayout.html" %}
 
 {% load i18n %}
+{% load l10n %}
 
 {% block title %}
 {% translate "Drinks - Order" %}
@@ -31,7 +32,7 @@
 
                 <div class="row">
                     <div class="column">{% translate "Price per Item" %} ({{ currency_suffix }}):</div>
-                    <div class="column" id="pricePerDrink" data-drink-price="{{ drink.price }}">{{ drink.price }}</div>
+                    <div class="column" id="pricePerDrink" data-drink-price="{% localize off %}{{ drink.price }}{% endlocalize %}">{{ drink.price }}</div>
                 </div>
 
                 {% if not drink.binary_availability %}
diff --git a/static/css/main.css b/static/css/main.css
index 298dd35..d8377b2 100644
--- a/static/css/main.css
+++ b/static/css/main.css
@@ -58,7 +58,7 @@ main {
     align-items: center;
     flex-grow: 1;
     width: 100%;
-    margin-top: 2rem;
+    margin-top: 5vh;
 }
 .userPanel {
     display: flex;
diff --git a/static/js/order.js b/static/js/order.js
index d35a025..d7f0f86 100644
--- a/static/js/order.js
+++ b/static/js/order.js
@@ -14,7 +14,7 @@ document.addEventListener("DOMContentLoaded", () => {
 
     // calculate & display sum
 
-    let order_price_per_drink = parseFloat(document.getElementById("pricePerDrink").dataset.drinkPrice.replace(",", "."));
+    let order_price_per_drink = parseFloat(document.getElementById("pricePerDrink").dataset.drinkPrice);
 
     function calculate_and_display_sum() {
 
-- 
2.43.0


From 5b72453dc3aff60803ce072ea54eb464c44d3c12 Mon Sep 17 00:00:00 2001
From: W13R <9070224-W13R@users.noreply.gitlab.com>
Date: Mon, 16 May 2022 11:00:24 +0200
Subject: [PATCH 5/7] Order drinks by product_name on the home page

---
 application/app/views.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/application/app/views.py b/application/app/views.py
index 372a630..7e153be 100644
--- a/application/app/views.py
+++ b/application/app/views.py
@@ -65,7 +65,7 @@ def login_page(request):
 @login_required
 def index(request):
     context = {
-        "available_drinks": Drink.objects.filter(available__gt=0).filter(deleted=False),
+        "available_drinks": Drink.objects.filter(available__gt=0).filter(deleted=False).order_by('product_name'),
     }
     return render(request, "index.html", context)
 
-- 
2.43.0


From ea644a6b7652c5c45185864ee87124d3016c41a8 Mon Sep 17 00:00:00 2001
From: W13R <9070224-W13R@users.noreply.gitlab.com>
Date: Mon, 16 May 2022 11:05:21 +0200
Subject: [PATCH 6/7] Removed obsolete configuration line in settings.py

---
 application/drinks_manager/settings.py | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/application/drinks_manager/settings.py b/application/drinks_manager/settings.py
index 6dc0a14..9523d14 100644
--- a/application/drinks_manager/settings.py
+++ b/application/drinks_manager/settings.py
@@ -36,12 +36,6 @@ ALLOWED_HOSTS = [
 ]
 
 
-### CSP Configuration ###
-
-
-CSP_DEFAULT_SRC = ("'self'", )
-
-
 ### ----------------- ###
 
 
-- 
2.43.0


From b7e96732676a2feb7c9cf1db44dae7189c266652 Mon Sep 17 00:00:00 2001
From: W13R <9070224-W13R@users.noreply.gitlab.com>
Date: Mon, 16 May 2022 13:17:47 +0200
Subject: [PATCH 7/7] Bump version to 5.0

---
 lib/env.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/env.sh b/lib/env.sh
index 43ec5e8..67baa3f 100644
--- a/lib/env.sh
+++ b/lib/env.sh
@@ -2,5 +2,5 @@
 
 export DJANGO_SK_ABS_FP="$(pwd)/config/secret_key.txt"
 export STATIC_FILES="$(pwd)/static/"
-export APP_VERSION="4.0"
+export APP_VERSION="5.0"
 export PYTHONPATH="$(pwd)/packages/"
-- 
2.43.0