All other loaded videos pause when one video is playing #1896
Conversation
There was a problem hiding this comment.
This works really well in most cases, though if you put multiple of the same video on a page then it'll occasionally not pause other videos if their URL happens to match due to them being at the same timestamp in the video. Comparing the video ids instead of their URLs seems to get around this, since each one is given a unique id even if they have the same URL.
…5-pause-other-videos
…API player data on unmount
There was a problem hiding this comment.
Forgive me @walid-i but I added a few last minute tweaks to get this into dev/25. Based on what @jpeterson976 found I changed the code that attempts to pause video from using the video URL that I had originally suggested to the id property that exists on the youtube player object. From what I found that just simply increments each time a video is added to the page and should always be unique.
I also noticed that the loadedVideos array was growing as you changed pages and never got rid of the player objects, so I added some code to remove them from loadedVideos on unmount. To make that slightly easier I changed loadedVideos from an array to an object.
No description provided.