From ef95b70eb95e113f0295dd18c528e741e03c1629 Mon Sep 17 00:00:00 2001
From: Sky Johnson <skylearj@gmail.com>
Date: Mon, 25 Aug 2025 16:55:55 -0500
Subject: [PATCH] switch logout links to a single universal form

---
 assets/dk.css           |  5 +++++
 templates/layout.html   | 11 ++++++++++-
 templates/leftside.html |  6 +++++-
 templates/topnav.html   |  5 +----
 4 files changed, 21 insertions(+), 6 deletions(-)

diff --git a/assets/dk.css b/assets/dk.css
index 56391bb..7123f14 100644
--- a/assets/dk.css
+++ b/assets/dk.css
@@ -105,6 +105,7 @@ a {
 	color: #663300;
 	text-decoration: none;
 	font-weight: bold;
+	cursor: pointer;
 
 	&:hover {
 		color: #330000;
@@ -580,4 +581,8 @@ div.bottom-reply {
 
 hr {
 	margin: 1rem 0;
+}
+
+.hidden {
+	display: none;
 }
\ No newline at end of file
diff --git a/templates/layout.html b/templates/layout.html
index cd81eae..ca1c801 100644
--- a/templates/layout.html
+++ b/templates/layout.html
@@ -56,7 +56,16 @@
 	{yield "scripts"}
 
 	<script>
-		document.addEventListener('DOMContentLoaded', addSelectArrows)
+		document.addEventListener('DOMContentLoaded', () => {
+			addSelectArrows()
+
+			document.querySelectorAll('.logout-link').forEach(link => {
+				link.addEventListener('click', (e) => {
+					e.preventDefault()
+					document.getElementById('logout-form').submit()
+				})
+			})
+		})
 	</script>
 </body>
 </html>
diff --git a/templates/leftside.html b/templates/leftside.html
index 1e85102..6749414 100644
--- a/templates/leftside.html
+++ b/templates/leftside.html
@@ -49,10 +49,14 @@
 	<a href="/">Home</a>
 	<a href="/forum">Forum</a>
 	<a href="/change-password">Change Password</a>
-	<a href="#">Log Out</a>
+	<a class="logout-link">Log Out</a>
 	<a href="/help">Help</a>
 </section>
 
+<form id="logout-form" class="hidden" action="/logout" method="post">
+	{csrf}
+</form>
+
 <script>
 document.addEventListener('DOMContentLoaded', () => {
 	const teleportModal = new ConfirmModal({
diff --git a/templates/topnav.html b/templates/topnav.html
index 12b260b..5b3f093 100644
--- a/templates/topnav.html
+++ b/templates/topnav.html
@@ -1,8 +1,5 @@
 {if authenticated}
-<form action="/logout" method="post" class="logout">
-	{csrf}
-	<button class="img-button" type="submit"><img src="/assets/images/button_logout.gif" alt="Log Out" title="Log Out"></button>
-</form>
+<a class="logout-link"><img src="/assets/images/button_logout.gif" alt="Log Out" title="Log Out"></a>
 <a href="/help"><img src="/assets/images/button_help.gif" alt="Help" title="Help"></a>
 {else}
 <a href="/login"><img src="/assets/images/button_login.gif" alt="Log In" title="Log In"></a>