@pradip1995/segment-product-card 0.3.23 → 0.3.24

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pradip1995/segment-product-card",
-  "version": "0.3.23",
+  "version": "0.3.24",
   "license": "MIT",
   "publishConfig": {
     "access": "public"
@@ -17,15 +17,19 @@
     "./product-scroll": "./src/product-scroll.tsx",
     "./product-rail-variants": "./src/product-rail-variants.tsx"
   },
+  "scripts": {
+    "typecheck": "tsc --noEmit",
+    "lint": "tsc --noEmit"
+  },
   "peerDependencies": {
     "@pradip1995/commerce-core": "^4.0.0",
     "@pradip1995/plugin-sdk": "^0.2.0",
-    "@pradip1995/segment-animation-login-popup": "*",
     "next": ">=15",
     "react": ">=19",
     "react-dom": ">=19"
   },
   "dependencies": {
+    "@pradip1995/segment-login-popup": "^0.1.3",
     "@pradip1995/segment-primitives": "^0.4.2",
     "@pradip1995/segment-tokens": "^0.3.7"
   },
@@ -36,11 +40,6 @@
     "@types/react-dom": "^19",
     "next": ">=15",
     "react": "19.0.3",
-    "typescript": "^5.7.2",
-    "@pradip1995/segment-animation-login-popup": "0.1.1"
-  },
-  "scripts": {
-    "typecheck": "tsc --noEmit",
-    "lint": "tsc --noEmit"
+    "typescript": "^5.7.2"
   }
-}
+}

package/src/product-card-actions.tsx

@@ -10,7 +10,7 @@ import {
 } from "@pradip1995/commerce-core/client/actions/wishlist"
 import type { HttpTypes } from "@medusajs/types"
 import { isNextRedirect } from "@pradip1995/segment-primitives/is-next-redirect"
-import LoginRequiredModal from "@pradip1995/segment-animation-login-popup/required"
+import LoginRequiredModal from "@pradip1995/segment-login-popup/required"
 import QuickAddPanel from "./quick-add-panel"
 
 type Props = {

package/src/resolve-product-images.ts

@@ -3,24 +3,18 @@ import { resolveMediaUrl } from "@pradip1995/segment-primitives/resolve-media-ur
 
 export function resolveProductCardImages(product: HttpTypes.StoreProduct) {
   const gallery =
-    (product.images as { url?: string | null }[] | undefined)
-      ?.map((image: { url?: string | null }) => resolveMediaUrl(image.url) || image.url?.trim())
-      .filter((url: string | undefined): url is string => Boolean(url)) ?? []
+    product.images
+      ?.map((image) => resolveMediaUrl(image.url))
+      .filter((url): url is string => !!url) ?? []
 
-  const variantImages = ((product.variants as { thumbnail?: string | null }[] | undefined) ?? [])
-    .map((variant: { thumbnail?: string | null }) => {
-      const raw = variant.thumbnail
-      return resolveMediaUrl(raw) || raw?.trim()
-    })
-    .filter((url: string | undefined): url is string => Boolean(url))
+  const variantImages = (product.variants ?? [])
+    .map((variant) => resolveMediaUrl((variant as { thumbnail?: string | null }).thumbnail))
+    .filter((url): url is string => !!url)
 
-  const rawThumb = product.thumbnail?.trim() || ""
-  const thumbnail = resolveMediaUrl(rawThumb) || rawThumb
-
-  const images = Array.from(new Set([thumbnail, ...gallery, ...variantImages].filter((url: string): url is string => Boolean(url))))
-  
+  const images = [...gallery, ...variantImages]
+  const thumbnail = resolveMediaUrl(product.thumbnail) || ""
   const primary = thumbnail || images[0] || ""
-  const hover = images.find((url: string) => Boolean(url) && url !== primary) || ""
+  const hover = [...new Set(images)].find((url) => url && url !== primary) || ""
 
   return { primary, hover }
 }

package/src/segment.tsx

@@ -5,10 +5,8 @@ import type { HttpTypes } from "@medusajs/types"
 import BeautyProductCard from "./beauty-product-card"
 import DefaultProductCard from "./default-product-card"
 import JewelryProductCard from "./jewelry-product-card"
-import MotionProductCard from "./motion-product-card"
-import ShopProductCard from "./shop-product-card"
 
-export type ProductCardVariant = "default" | "beauty" | "jewelry" | "motion" | "shop"
+export type ProductCardVariant = "default" | "beauty" | "jewelry"
 
 type Props = {
   product: HttpTypes.StoreProduct
@@ -21,21 +19,12 @@ type Props = {
 }
 
 function resolveProductCardVariant(variant?: ProductCardVariant): ProductCardVariant {
-  if (
-    variant === "beauty" ||
-    variant === "default" ||
-    variant === "jewelry" ||
-    variant === "motion" ||
-    variant === "shop"
-  )
-    return variant
+  if (variant === "beauty" || variant === "default" || variant === "jewelry") return variant
   return "default"
 }
 
 export default function ProductCard({ variant, ...props }: Props) {
   const layout = resolveProductCardVariant(variant)
-  if (layout === "shop") return <ShopProductCard {...props} />
-  if (layout === "motion") return <MotionProductCard {...props} />
   if (layout === "beauty") return <BeautyProductCard {...props} />
   if (layout === "jewelry") return <JewelryProductCard {...props} />
   return <DefaultProductCard {...props} />