Optimize code
This commit is contained in:
@ -6,8 +6,6 @@ const YouTubePlayer = ({
|
||||
autoplay = false,
|
||||
startTime = 45,
|
||||
onReady = () => {},
|
||||
onError = () => {},
|
||||
onPlayerReady = null,
|
||||
className = ''
|
||||
}) => {
|
||||
const iframeRef = useRef(null);
|
||||
@ -65,36 +63,20 @@ const YouTubePlayer = ({
|
||||
},
|
||||
events: {
|
||||
'onReady': (event) => {
|
||||
const player = event.target;
|
||||
console.log("YouTube player ready event fired");
|
||||
|
||||
playerRef.current = player;
|
||||
console.log("YouTube player ready");
|
||||
|
||||
try {
|
||||
const hasSeekTo = typeof player.seekTo === 'function';
|
||||
console.log(`Player has seekTo: ${hasSeekTo}`);
|
||||
|
||||
// Start playback if needed
|
||||
if (autoplay) {
|
||||
player.seekTo(startTime || 0);
|
||||
player.playVideo();
|
||||
}
|
||||
|
||||
setIsLoaded(true);
|
||||
onReady();
|
||||
|
||||
if (typeof onPlayerReady === 'function') {
|
||||
console.log("Providing player instance to parent");
|
||||
onPlayerReady(player);
|
||||
}
|
||||
} catch (err) {
|
||||
console.error("Error in player ready handler:", err);
|
||||
onError(err);
|
||||
// Start playback if needed
|
||||
if (autoplay) {
|
||||
event.target.seekTo(startTime || 0);
|
||||
event.target.playVideo();
|
||||
}
|
||||
|
||||
setIsLoaded(true);
|
||||
onReady();
|
||||
},
|
||||
'onError': (event) => {
|
||||
console.error("YouTube player error:", event);
|
||||
onError(event);
|
||||
// Remove error handling
|
||||
}
|
||||
}
|
||||
});
|
||||
@ -115,7 +97,7 @@ const YouTubePlayer = ({
|
||||
}
|
||||
}
|
||||
};
|
||||
}, [videoId, autoplay, onReady, onError, startTime, customStartTime]);
|
||||
}, [videoId, autoplay, onReady, startTime, customStartTime]);
|
||||
|
||||
useEffect(() => {
|
||||
if (playerRef.current && isLoaded && customStartTime !== startTime) {
|
||||
|
Reference in New Issue
Block a user