fix: issue with id not being passed through, type definition

just214 · Mar 12, 2022 · c101517 · c101517
1 parent 8fab20b
commit c101517
Showing 1 changed file with 12 additions and 8 deletions.
diff --git a/src/react-hook-youtube.tsx b/src/react-hook-youtube.tsx
@@ -23,13 +23,16 @@ export function useYoutubePlayer(options: YT.PlayerOptions): {
   // COMPONENT
   const YoutubePlayer =
     React.useMemo((): React.FunctionComponent<YoutubePlayerProps> => {
-      return (props) => (
-        <div
-          className={props.className || ""}
-          style={props.style}
-          id={props.id || `youtube-player-${options?.videoId}`}
-        />
-      );
+      return (props) => {
+        const playerId = `youtube-player-${props?.id || options?.videoId}`;
+        return (
+          <div
+            className={props.className || ""}
+            style={props.style}
+            id={playerId}
+          />
+        );
+      };
     }, [options.videoId]);
 
   /**
@@ -239,7 +242,8 @@ export function useYoutubePlayer(options: YT.PlayerOptions): {
   };
 
   function loadVideo() {
-    new YT.Player(`youtube-player-${options.videoId}`, {
+    if (!window?.YT) return;
+    new window.YT.Player(`youtube-player-${options.videoId}`, {
       videoId: options.videoId,
       playerVars: options.playerVars,
       height: options.height || 390,