samvera-labs / ramp

Interactive, IIIF powered audio/video media player React components library. Styleguidist Docs: https://samvera-labs.github.io/ramp/
https://ramp.avalonmediasystem.org/
27 stars 4 forks source link

Media Player Has Gray Overlay When Moving Between Items #548

Closed joncameron closed 14 hours ago

joncameron commented 2 weeks ago

Description

If you start from a playable playlist item that immediately follows an inaccessible item and click previous canvas, the inaccessible item doesn't fully load. It displays with timer, but is grayed out looking and the previous and next buttons are disabled. To reproduce, load the playlist, click on playlist item #8 and click the previous canvas button. Note that after I've played with this playlist a bunch, the problem is not always reproducible unless I reload the playlist.

Related bugs

Example

https://avalon-dev.dlib.indiana.edu/playlists/5

Done Looks Like

Dananji commented 1 week ago

Created a new issue for the mobile devices: https://github.com/samvera-labs/ramp/issues/552

elynema commented 1 week ago

@Dananji I tested the 4 cases listed above that mentioned desktop in Safari and Chrome.

I ran into 2 issues that were intermittent, unfortunately. Perhaps you can see if you are able to reproduce them?

  1. [In Safari only] Select playlist item 17, and hit the next canvas button. Playlist item 18 loads, but can't actually play (just spins if you try to play it, and the poster doesn't load). This happens intermittently, not always, and maybe be unrelated to the prev/next actions?
  2. [In Safari only] Select playlist item 26, use next canvas buttons to advance through to item number 29. Use the previous canvas buttons to go backward. Sometimes when going back from playlist item 28 to playlist item 27, the playlist jumps to 27 and then back to 28. This happened to me multiple times, but every time I tried to record a video, it worked fine. [In Chrome only] When trying this in Chrome, I got the gray overlay when I clicked Previous from playlist item 28 to go back to item 27.
Dananji commented 1 week ago

@elynema I was able to reproduce issue 1 in Safari and the Chrome issue in 2 intermittently, but not the issue 2 in Safari.

As you mentioned both these are unrelated to prev/next button functionality.

These could be related to player events that are used for identifying the status to reveal the player as needed. We've had issues with player events not triggering in Safari but not Chrome before. Maybe create a new bug ticket for these?

elynema commented 1 week ago

@Dananji Is the Chrome issue for 2 related to this ticket since it is also a situation where we are getting the gray overlay when moving between items?

Dananji commented 1 week ago

Yes I think so. I'll do that work as a part of this ticket.

elynema commented 1 week ago

I've created Ramp #560 and #561 for Safari issue 1 and 2.

joncameron commented 14 hours ago

👍