From 1a5fd691bd3856db7d2cb2f9e99793b05d7ccfc0 Mon Sep 17 00:00:00 2001
From: Local Server <server@skyartshop.local>
Date: Wed, 14 Jan 2026 21:19:23 -0600
Subject: [PATCH] Phase 1: Fix script loading order and remove duplicates

- Remove duplicate cart.js loads from about.html and contact.html
- Standardize script loading order across all pages:
  1. api-cache.js (if needed)
  2. main.js
  3. shop-system.js
  4. Page-specific scripts
  5. UI enhancements last
- Remove cart.js from shop.html and product.html (shop-system handles it)
- Add api-cache.js to about.html and contact.html (they make API calls)
- Prevent double initialization conflicts
---
 website/public/about.html     | 15 ++++++++++-----
 website/public/blog.html      | 12 ++++++++----
 website/public/contact.html   | 15 ++++++++++-----
 website/public/portfolio.html | 12 ++++++++----
 website/public/product.html   |  7 ++++---
 website/public/shop.html      |  4 ++--
 6 files changed, 42 insertions(+), 23 deletions(-)

diff --git a/website/public/about.html b/website/public/about.html
index 4f59337..67fbce2 100644
--- a/website/public/about.html
+++ b/website/public/about.html
@@ -435,12 +435,19 @@
       </div>
     </footer>
 
+    <!-- Core utilities and state management -->
+    <script src="/assets/js/api-cache.js"></script>
+    <script src="/assets/js/main.js"></script>
+    
+    <!-- Shopping system (handles cart & wishlist) -->
+    <script src="/assets/js/shop-system.js"></script>
+    
+    <!-- Page interactions -->
     <script src="/assets/js/page-transitions.js?v=1766709739"></script>
     <script src="/assets/js/back-button-control.js?v=1766723554"></script>
-    <script src="/assets/js/main.js"></script>
     <script src="/assets/js/navigation.js"></script>
-    <script src="/assets/js/cart.js"></script>
-    <script src="/assets/js/shopping.js"></script>
+    
+    <!-- Page-specific content loading -->
     <script>
       // Load about page content from API
       async function loadAboutContent() {
@@ -598,7 +605,5 @@
         loadTeamMembers();
       });
     </script>
-    <script src="/assets/js/cart.js"></script>
-    <script src="/assets/js/shop-system.js"></script>
   </body>
 </html>
diff --git a/website/public/blog.html b/website/public/blog.html
index 34c06b3..2aa93e3 100644
--- a/website/public/blog.html
+++ b/website/public/blog.html
@@ -248,13 +248,17 @@
         </div>
       </div>
     </footer>
+    <!-- Core dependencies (load in order) -->
+    <script src="/assets/js/api-cache.js"></script>
+    <script src="/assets/js/main.js"></script>
+    <script src="/assets/js/shop-system.js"></script>
+    
+    <!-- UI enhancements -->
     <script src="/assets/js/page-transitions.js?v=1766709739"></script>
     <script src="/assets/js/back-button-control.js?v=1766723554"></script>
-    <script src="/assets/js/main.js"></script>
     <script src="/assets/js/navigation.js"></script>
-    <script src="/assets/js/shop-system.js"></script>
-    <script src="/assets/js/shopping.js"></script>
-    <script src="/assets/js/api-cache.js"></script>
+    
+    <!-- Page-specific functionality -->
     <script>
       // Load blog posts from API
       async function loadBlog() {
diff --git a/website/public/contact.html b/website/public/contact.html
index 5e1ada9..c4f3847 100644
--- a/website/public/contact.html
+++ b/website/public/contact.html
@@ -546,12 +546,19 @@
       </div>
     </footer>
 
+    <!-- Core utilities and state management -->
+    <script src="/assets/js/api-cache.js"></script>
+    <script src="/assets/js/main.js"></script>
+    
+    <!-- Shopping system (handles cart & wishlist) -->
+    <script src="/assets/js/shop-system.js"></script>
+    
+    <!-- Page interactions -->
     <script src="/assets/js/page-transitions.js?v=1766709739"></script>
     <script src="/assets/js/back-button-control.js?v=1766723554"></script>
-    <script src="/assets/js/main.js"></script>
     <script src="/assets/js/navigation.js"></script>
-    <script src="/assets/js/cart.js"></script>
-    <script src="/assets/js/shopping.js"></script>
+    
+    <!-- Page-specific functionality -->
     <script>
       // Handle contact form submission
       document
@@ -658,7 +665,5 @@
         loadContactInfo();
       }
     </script>
-    <script src="/assets/js/cart.js"></script>
-    <script src="/assets/js/shop-system.js"></script>
   </body>
 </html>
diff --git a/website/public/portfolio.html b/website/public/portfolio.html
index 07e008a..d54816d 100644
--- a/website/public/portfolio.html
+++ b/website/public/portfolio.html
@@ -319,13 +319,17 @@
         ></div>
       </div>
     </div>
+    <!-- Core dependencies (load in order) -->
+    <script src="/assets/js/api-cache.js"></script>
+    <script src="/assets/js/main.js"></script>
+    <script src="/assets/js/shop-system.js"></script>
+    
+    <!-- UI enhancements -->
     <script src="/assets/js/page-transitions.js?v=1766709739"></script>
     <script src="/assets/js/back-button-control.js?v=1766723554"></script>
-    <script src="/assets/js/main.js"></script>
     <script src="/assets/js/navigation.js"></script>
-    <script src="/assets/js/shop-system.js"></script>
-    <script src="/assets/js/shopping.js"></script>
-    <script src="/assets/js/api-cache.js"></script>
+    
+    <!-- Page-specific functionality -->
     <script>
       let portfolioProjects = [];
       
diff --git a/website/public/product.html b/website/public/product.html
index 526aa0d..d54eca1 100644
--- a/website/public/product.html
+++ b/website/public/product.html
@@ -195,14 +195,15 @@
 
     <div id="productDetail" style="display: none"></div>
 
+    <!-- Core Scripts (load in dependency order) -->
     <script src="/assets/js/api-cache.js"></script>
+    <script src="/assets/js/main.js"></script>
     <script src="/assets/js/shop-system.js"></script>
-    <script src="/assets/js/cart.js"></script>
+    <script src="/assets/js/api-client.js"></script>
+    <script src="/assets/js/notifications.js"></script>
     <script src="/assets/js/page-transitions.js?v=1766709739"></script>
     <script src="/assets/js/back-button-control.js?v=1766723554"></script>
     <script src="/assets/js/navigation.js?v=1766708114"></script>
-    <script src="/assets/js/api-client.js"></script>
-    <script src="/assets/js/notifications.js"></script>
     <script>
       // Function to change primary image
       function changePrimaryImage(imageUrl, index) {
diff --git a/website/public/shop.html b/website/public/shop.html
index c326bce..6e038bd 100644
--- a/website/public/shop.html
+++ b/website/public/shop.html
@@ -856,10 +856,10 @@
       </div>
     </footer>
 
-    <!-- Core Scripts -->
+    <!-- Core Scripts (load in dependency order) -->
     <script src="/assets/js/api-cache.js"></script>
+    <script src="/assets/js/main.js"></script>
     <script src="/assets/js/shop-system.js"></script>
-    <script src="/assets/js/cart.js"></script>
     <script src="/assets/js/api-client.js"></script>
     <script src="/assets/js/notifications.js"></script>
     <script src="/assets/js/page-transitions.js?v=1767228800"></script>