show video play button in placeholder depends only on show button toggle

wix-incubator · Aug 27, 2024 · a5672d5 · a5672d5
1 parent e386e66
commit a5672d5
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/packages/gallery/src/components/item/videos/videoItemWrapper.js b/packages/gallery/src/components/item/videos/videoItemWrapper.js
@@ -69,7 +69,6 @@ class VideoItemWrapper extends React.Component {
       'isCurrentHover',
       'customComponents',
     ]);
-
     return (
       <VideoItemPlaceholder
         {...props}
@@ -78,8 +77,7 @@ class VideoItemWrapper extends React.Component {
         isThumbnail={!!this.props.thumbnailHighlightId}
         id={this.props.idx}
         videoPlayButton={
-          showVideoPlayButton &&
-          !this.state.videoItemLoaded && (
+          showVideoPlayButton && (
             <VideoPlayButton
               pointerEvents={
                 !useTransparentPlayButtonAndForceLoadVideo(this.props)
@@ -111,11 +109,11 @@ class VideoItemWrapper extends React.Component {
     const videoPlaceholder = this.createVideoPlaceholder(showVideoPlayButton);
 
     const VideoItem = this.VideoItem;
-    const shouldRenderVideoItem =
+    const shouldRenderVideoPlaceholder =
       !this.mightPlayVideo() ||
       !this.state.videoItemLoaded ||
       this.props.isPrerenderMode;
-    if (shouldRenderVideoItem) {
+    if (shouldRenderVideoPlaceholder) {
       return (
         <div>
           {shouldCreateVideoPlaceholder(this.props.options) && videoPlaceholder}