If I don’t want YouTube to start playing after clicking on the cover, what would be changed in the code?
This:
function play() {
player.playVideo();
}
return {
addPlayer,
play
};
}());
Would get changed to this:
}
});
}
return {
addPlayer
};
}());
This:
videoPlayer.play();
Would be removed from here:
function coverClickHandler(evt) {
const wrapper = evt.currentTarget.nextElementSibling;
show(wrapper);
}
Is this right?
That seems about right.
Redefinition of ‘player’ from line 20.
const player = event.target;
In Here
function onPlayerStateChange(event) {
const player = event.target;
const shufflePlaylist = true;
https://jsfiddle.net/jewhfdkt/1/
Removing the const from player fixes that.
It doesn’t like the guard condition. It’s totally valid, but it will prefer this other way:
function show(el) {
if (el) {
el.classList.remove("hide");
}
}
function hide(el) {
if (el) {
el.classList.add("hide");
}
}
How would this piece be converted from .forEach to for Loop instead?
I have one code that uses all .forEach, and another that uses for Loop.
For learning purposes.
function initCovers(coverOpts) {
coverOpts.forEach(function (opts) {
init(opts);
});
}
Loop codes are best to be avoided, because the index markers become confusing at best and can be completely messed up at worst.
forEach has been the standard for well over a decade. Don’t go back.
As I understand it, playVideo(); only gets used on codes where a cover is being used, right?
function play() {
player.playVideo();
}
return {
addPlayer,
play
};
}());
I’m confused about something.
Is function addPlayer(video)
only being used for when there is only a single player being used.
and
function addVideo(video, settings) {
gets used for when multiple videos are being used on a page?
Do I have that right?
That makes sense. When there is no cover a person can just play the video themself.
They can both be used in the opposite situations too, but the addVideo function is more capable.
As the addVideo function name seems to be too generic, I would have addPlayer(video, settings) as the function instead, as the settings are specific to the youtube video player.
21 posts were split to a new topic: Why is playVideo() not in the code?