From bfd1803c6f2407708cf1f10ca3f1234ab439f121 Mon Sep 17 00:00:00 2001
From: sebseb7 <sebgreenbus@gmail.com>
Date: Sun, 20 Jul 2025 01:16:44 +0200
Subject: [PATCH] refactor: remove isPrerender prop and update ProductImage
 component to unify fullscreen handling in PrerenderProduct and
 ProductDetailPage

---
 src/PrerenderProduct.js             |  6 +++-
 src/components/ProductDetailPage.js |  1 -
 src/components/ProductImage.js      | 49 ++---------------------------
 3 files changed, 7 insertions(+), 49 deletions(-)

diff --git a/src/PrerenderProduct.js b/src/PrerenderProduct.js
index 08dda0f..4b49c59 100644
--- a/src/PrerenderProduct.js
+++ b/src/PrerenderProduct.js
@@ -174,7 +174,11 @@ class PrerenderProduct extends React.Component {
               ProductImage,
               {
                 product: product,
-                isPrerender: true
+                socket: null,
+                socketB: null,
+                fullscreenOpen: false,
+                onOpenFullscreen: null,
+                onCloseFullscreen: null
               }
             ),
             // Product Details Section
diff --git a/src/components/ProductDetailPage.js b/src/components/ProductDetailPage.js
index 582f4aa..2cf4dc1 100644
--- a/src/components/ProductDetailPage.js
+++ b/src/components/ProductDetailPage.js
@@ -677,7 +677,6 @@ class ProductDetailPage extends Component {
             fullscreenOpen={this.state.imageDialogOpen}
             onOpenFullscreen={this.handleOpenDialog}
             onCloseFullscreen={this.handleCloseDialog}
-            isPrerender={false}
           />
 
           {/* Product Details */}
diff --git a/src/components/ProductImage.js b/src/components/ProductImage.js
index c5d9a36..ea8af83 100644
--- a/src/components/ProductImage.js
+++ b/src/components/ProductImage.js
@@ -9,18 +9,9 @@ const ProductImage = ({
   socketB, 
   fullscreenOpen, 
   onOpenFullscreen, 
-  onCloseFullscreen,
-  isPrerender = false 
+  onCloseFullscreen
 }) => {
-  // For prerender, use static image path
-  const getImagePath = (pictureList) => {
-    if (!pictureList || !pictureList.trim()) {
-      return '/assets/images/nopicture.jpg';
-    }
-    return `/assets/images/prod${pictureList.split(',')[0].trim()}.jpg`;
-  };
-
-  // Container styling - same for both versions
+  // Container styling - unified for all versions
   const containerSx = {
     width: { xs: "100%", sm: "555px" },
     maxWidth: "100%",
@@ -32,42 +23,6 @@ const ProductImage = ({
     justifyContent: "center",
   };
 
-  if (isPrerender) {
-    // Prerender version - use CardMedia with static paths
-    return (
-      <Box sx={containerSx}>
-        {!product.pictureList && (
-          <CardMedia
-            component="img"
-            height="400"
-            image="/assets/images/nopicture.jpg"
-            alt={product.name}
-            sx={{ objectFit: "cover" }}
-          />
-        )}
-        {product.pictureList && (
-          <Box sx={{ position: 'relative', display: 'inline-block' }}>
-            <CardMedia
-              component="img"
-              height="400"
-              image={getImagePath(product.pictureList)}
-              alt={product.name}
-              sx={{ 
-                objectFit: 'contain',
-                cursor: 'pointer',
-                transition: 'transform 0.2s ease-in-out',
-                '&:hover': {
-                  transform: 'scale(1.02)'
-                }
-              }}
-            />
-          </Box>
-        )}
-      </Box>
-    );
-  }
-
-  // SPA version - use existing Images component
   return (
     <Box sx={containerSx}>
       {!product.pictureList && (