From 82c1f386852b571d1e804be63596bd6d3a168b6b Mon Sep 17 00:00:00 2001
From: kuekhaoyang <kuekhaoyang@Gmail.com>
Date: Mon, 24 Nov 2025 17:39:19 +0800
Subject: [PATCH] refactor: Remove conditional image priority loading, setting
 all movie and video card images to eager.

---
 components/home/MovieCard.tsx   | 6 ++----
 components/home/MovieGrid.tsx   | 3 +--
 components/search/VideoCard.tsx | 7 ++-----
 components/search/VideoGrid.tsx | 3 +--
 4 files changed, 6 insertions(+), 13 deletions(-)

diff --git a/components/home/MovieCard.tsx b/components/home/MovieCard.tsx
index b923d53..f53cb8b 100644
--- a/components/home/MovieCard.tsx
+++ b/components/home/MovieCard.tsx
@@ -20,10 +20,9 @@ interface DoubanMovie {
 interface MovieCardProps {
   movie: DoubanMovie;
   onMovieClick: (movie: DoubanMovie) => void;
-  priority?: boolean;
 }
 
-export const MovieCard = memo(function MovieCard({ movie, onMovieClick, priority = false }: MovieCardProps) {
+export const MovieCard = memo(function MovieCard({ movie, onMovieClick }: MovieCardProps) {
   return (
     <Link
       href={`/?q=${encodeURIComponent(movie.title)}`}
@@ -45,8 +44,7 @@ export const MovieCard = memo(function MovieCard({ movie, onMovieClick, priority
             fill
             className="object-cover transition-transform duration-300 group-hover:scale-105 rounded-[var(--radius-2xl)]"
             sizes="(max-width: 640px) 50vw, (max-width: 768px) 33vw, (max-width: 1024px) 25vw, 20vw"
-            loading={priority ? "eager" : "lazy"}
-            priority={priority}
+            loading="eager"
             unoptimized
             referrerPolicy="no-referrer"
             onError={(e) => {
diff --git a/components/home/MovieGrid.tsx b/components/home/MovieGrid.tsx
index 1423b89..51dd1a7 100644
--- a/components/home/MovieGrid.tsx
+++ b/components/home/MovieGrid.tsx
@@ -37,12 +37,11 @@ export function MovieGrid({
   return (
     <>
       <div className="grid grid-cols-2 sm:grid-cols-3 md:grid-cols-4 lg:grid-cols-5 gap-4 md:gap-6">
-        {movies.map((movie, index) => (
+        {movies.map((movie) => (
           <MovieCard
             key={movie.id}
             movie={movie}
             onMovieClick={onMovieClick}
-            priority={index < 10}
           />
         ))}
       </div>
diff --git a/components/search/VideoCard.tsx b/components/search/VideoCard.tsx
index c077054..dba1cab 100644
--- a/components/search/VideoCard.tsx
+++ b/components/search/VideoCard.tsx
@@ -16,7 +16,6 @@ interface VideoCardProps {
     cardId: string;
     isActive: boolean;
     onCardClick: (e: React.MouseEvent, cardId: string, videoUrl: string) => void;
-    priority?: boolean;
 }
 
 export const VideoCard = memo<VideoCardProps>(({
@@ -24,8 +23,7 @@ export const VideoCard = memo<VideoCardProps>(({
     videoUrl,
     cardId,
     isActive,
-    onCardClick,
-    priority = false
+    onCardClick
 }) => {
     return (
         <div
@@ -61,8 +59,7 @@ export const VideoCard = memo<VideoCardProps>(({
                                 fill
                                 className="object-cover rounded-[var(--radius-2xl)]"
                                 sizes="(max-width: 640px) 33vw, (max-width: 1024px) 20vw, 16vw"
-                                loading={priority ? "eager" : "lazy"}
-                                priority={priority}
+                                loading="eager"
                                 unoptimized
                                 referrerPolicy="no-referrer"
                                 onError={(e) => {
diff --git a/components/search/VideoGrid.tsx b/components/search/VideoGrid.tsx
index 49a053f..3df511e 100644
--- a/components/search/VideoGrid.tsx
+++ b/components/search/VideoGrid.tsx
@@ -89,7 +89,7 @@ export const VideoGrid = memo(function VideoGrid({ videos, className = '' }: Vid
           contentVisibility: 'auto',
         }}
       >
-        {visibleItems.map(({ video, videoUrl, cardId }, index) => {
+        {visibleItems.map(({ video, videoUrl, cardId }) => {
           const isActive = activeCardId === cardId;
 
           return (
@@ -100,7 +100,6 @@ export const VideoGrid = memo(function VideoGrid({ videos, className = '' }: Vid
               cardId={cardId}
               isActive={isActive}
               onCardClick={handleCardClick}
-              priority={index < 10}
             />
           );
         })}