Does not work well with pages with multiple video players

[juliano-araujo]

It basically works on all players on all pages, but on pages with multiple video players it tracks only the first player that appears on the page.

[hnryjms]

Very fair. I'm mostly unsure how the UI would prompt the user which video to choose before starting the session. Code-wise, it would be this section that's picking the first video on the page.

https://github.com/streamtogether/stream-together-web-extension/blob/master/src/session/session.ts#L62

Using a document.querySelectorAll() would let you get an array of videos, and then send that up to the Browser extension popup, for the user to pick. I'm just not sure if we can display a thumbnail or something in there (and how we'd get that thumbnail), or just have a dropdown or something.

I'd imagine for better support here we'd need a few things:

• Session to use document.querySelectorAll(), start sending popup all video element data, and ask the user to pick a video if there are more than one elements on screen.
• Popup window to display some type of picker for multiple video elements, otherwise auto-pick the one video when available
• PeerJS code to synchronize an ID or other info about what video is chosen, so that joiners can have it automatically choose the right player to stay synchronized.

[hnryjms]

Aside: @wmww has PR #26 which adds support for no video being found. I'm wondering if it makes more sense to implement some type of querySelectorAll in that work, rather than two separate Event types being emitted (Video, NoVideo).