Semro / syncwatch

Browser extension to watch videos together
MIT License
135 stars 19 forks source link

BUG: Doesn't allow to toggle between "screen" and "lecture" on Harvard CS50 video player #41

Closed sepsol closed 1 year ago

sepsol commented 1 year ago

Steps to reproduce:

  1. Go here: https://cs50.harvard.edu/x/2023/weeks/5/
  2. Play the video and try the Screen toggle at the top right of the player
  3. Now, create a room and share the page
  4. See how the Screen stops working
Semro commented 1 year ago

I don't think it's a bug. That's how the player works on this site. When User1 toggles the screen, then the video player fires "pause" and then dispatches "seek" events each 500ms, extension handles it and sends to other users in the room. image

sepsol commented 1 year ago

I think you're right, it works if I just open the player in full screen using the arrow at the top right hand side.