chore: disable video preview autoplay when embedded in a super note […

…skip e2e]
standardnotes · Apr 21, 2024 · b5dbbae · b5dbbae
1 parent cdb380d
commit b5dbbae
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/packages/web/src/javascripts/Components/FilePreview/PreviewComponent.tsx b/packages/web/src/javascripts/Components/FilePreview/PreviewComponent.tsx
@@ -90,7 +90,14 @@ const PreviewComponent: FunctionComponent<Props> = ({
   }
 
   if (file.mimeType.startsWith('video/')) {
-    return <VideoPreview file={file} filesController={application.filesController} objectUrl={objectUrl} />
+    return (
+      <VideoPreview
+        file={file}
+        filesController={application.filesController}
+        objectUrl={objectUrl}
+        isEmbeddedInSuper={isEmbeddedInSuper}
+      />
+    )
   }
 
   if (file.mimeType.startsWith('audio/')) {

diff --git a/packages/web/src/javascripts/Components/FilePreview/VideoPreview.tsx b/packages/web/src/javascripts/Components/FilePreview/VideoPreview.tsx
@@ -9,6 +9,7 @@ type Props = {
   file: FileItem
   filesController: FilesController
   objectUrl: string
+  isEmbeddedInSuper: boolean
 }
 
 /**
@@ -17,7 +18,7 @@ type Props = {
  * when not using the <source/> tag.
  * We show an error message if neither works.
  */
-const VideoPreview = ({ file, filesController, objectUrl }: Props) => {
+const VideoPreview = ({ file, filesController, objectUrl, isEmbeddedInSuper }: Props) => {
   const [showError, setShowError] = useState(false)
   const [shouldTryFallback, setShouldTryFallback] = useState(false)
 
@@ -65,7 +66,7 @@ const VideoPreview = ({ file, filesController, objectUrl }: Props) => {
     <video
       className="h-full w-full"
       controls
-      autoPlay
+      autoPlay={!isEmbeddedInSuper}
       onError={() => {
         setShouldTryFallback(true)
       }}