From 01b7186052503725c86ba524b55bd1ea744ef05e Mon Sep 17 00:00:00 2001
From: Deco Vander <deco@deco.sh>
Date: Sat, 5 Jul 2025 18:51:48 -0400
Subject: [PATCH] Apply suggestions from code review

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
---
 public/admin.html    | 3 ++-
 public/privacy.html  | 2 +-
 src/scss/mixins.scss | 6 +++---
 3 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/public/admin.html b/public/admin.html
index 07f58f8..c39c699 100644
--- a/public/admin.html
+++ b/public/admin.html
@@ -167,7 +167,8 @@
         </div>
     </footer>
 
-    <!-- Load shared theme utility --><script src="theme-utils.js"></script>
+    <!-- Load shared theme utility -->
+    <script src="theme-utils.js"></script>
     <script src="utils.js"></script>
     <script src="admin.js"></script>
 </body>
diff --git a/public/privacy.html b/public/privacy.html
index f85fc45..ea6476b 100644
--- a/public/privacy.html
+++ b/public/privacy.html
@@ -123,7 +123,7 @@
     </footer>
     
     <!-- Load shared theme utility -->
-    <script src="/theme-utils.js"></script>
+    <script src="theme-utils.js"></script>
     <script>
         // Initialize theme when page loads
         document.addEventListener('DOMContentLoaded', initializeTheme);
diff --git a/src/scss/mixins.scss b/src/scss/mixins.scss
index 328f5df..bb0d2c9 100644
--- a/src/scss/mixins.scss
+++ b/src/scss/mixins.scss
@@ -123,12 +123,12 @@
 }
 
 // Theme Toggle Mixin (consolidates duplicated theme toggle styles)
-@mixin theme-toggle-styles {
+@mixin theme-toggle-styles($width: 40px, $height: 40px) {
   @include button($bg-color: transparent);
   border: 2px solid var(--border-color);
   border-radius: $border-radius-full;
-  width: 40px;
-  height: 40px;
+  width: $width;
+  height: $height;
   @include flex-center;
   transition: all 0.3s ease;
   box-shadow: 0 2px 4px var(--shadow);