dolbyio-samples / rts-app-react-publisher-viewer

A Real Time Streaming App example for dolby.io
MIT License
15 stars 9 forks source link

[Viewer App] Audio/Video/Playback control states should be maintain separately for every projected source #311

Closed vishalkharge closed 1 year ago

vishalkharge commented 1 year ago

States of the audio/video/playback control are not maintained by the Viewer App separately for every source.

- [ ] Issue 3: When the viewer stops the playback of the projected video source, it does not maintain the state of the playback control when projected the same video source again.

STEPS:

  1. Open the publisher app

  2. Add multiple sources such as a. Camera Source b. Stream local file mp4 with audio c. Screen share with youtube video d. Screen share with window

  3. Go live

  4. Open the viewer app

  5. On the viewer app, mutes/unmute the speaker of specific projected video source

  6. Select other source from the list to project it as main source

  7. It will mute/unmute the speaker of projected source

  8. On the viewer app, turns off/on the video of specific projected video source

  9. Select other source from the list to project it as main source

  10. It will turns off/on the video of projected source

  11. On the viewer app, stop the playback of specific projected video source say a screen share youtube video

  12. Select other source from the list to project it as main source

  13. During the projection it will play the audio track of the previous source [screen share youtube video]

  14. Again select the previous source [screen share youtube video] from the list, the state of the playback is playing instead of stop

Branch: int-issue-145-multisources OS: MAC Browser: Chrome

vincentsong commented 1 year ago

@jackshen we need to maintain the individual source's isVideoEnabled, isAudioEnabled and isPaused states in videwer app, while we also need to expose displayVideo, muted and paused properties in ViewerVideoView. Probabley we also need to main the volume for each source as well.

vincentsong commented 1 year ago

Might solve issue #314

vishalkharge commented 1 year ago

Verification Status: PASSED

Branch: int-issue-145-multisources Environment: MAC/Chrome Date: Feb 22, 2023

Notes: