Apply suggestions from code review

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This commit is contained in:
Deco Vander 2025-07-05 18:51:48 -04:00 committed by GitHub
parent d20ce03264
commit 01b7186052
3 changed files with 6 additions and 5 deletions

View file

@ -167,7 +167,8 @@
</div> </div>
</footer> </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="utils.js"></script>
<script src="admin.js"></script> <script src="admin.js"></script>
</body> </body>

View file

@ -123,7 +123,7 @@
</footer> </footer>
<!-- Load shared theme utility --> <!-- Load shared theme utility -->
<script src="/theme-utils.js"></script> <script src="theme-utils.js"></script>
<script> <script>
// Initialize theme when page loads // Initialize theme when page loads
document.addEventListener('DOMContentLoaded', initializeTheme); document.addEventListener('DOMContentLoaded', initializeTheme);

View file

@ -123,12 +123,12 @@
} }
// Theme Toggle Mixin (consolidates duplicated theme toggle styles) // 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); @include button($bg-color: transparent);
border: 2px solid var(--border-color); border: 2px solid var(--border-color);
border-radius: $border-radius-full; border-radius: $border-radius-full;
width: 40px; width: $width;
height: 40px; height: $height;
@include flex-center; @include flex-center;
transition: all 0.3s ease; transition: all 0.3s ease;
box-shadow: 0 2px 4px var(--shadow); box-shadow: 0 2px 4px var(--shadow);