avalonmediasystem / avalon

Avalon Media System – Samvera Application
http://www.avalonmediasystem.org/
Apache License 2.0
93 stars 51 forks source link

[BUG] Add to Playlist, Create Timeline, and Create Thumbnail buttons sometimes do not get enabled #5885

Open elynema opened 2 weeks ago

elynema commented 2 weeks ago

Describe the bug Sometimes, the Add to Playlist, Create Timeline, and Create Thumbnail buttons never get enabled on page load. So far, this is seen on Android and iOS mobile devices, mostly after reloading the page numerous times.

Once this happens, reloading the page does not seem to re-enable the buttons. Starting playing the video and then reloading the page may re-enable the buttons.

To Reproduce Steps to reproduce the behavior, including the results:

  1. As an example where we have seen this happen, go to https://avalon-dev.dlib.indiana.edu/media_objects/fj236208t on iOS/Safari or Android Chrome.
  2. Reload the page repeatedly until you see that these buttons remain gray.

Expected behavior These buttons should become enabled once the player is ready. We could conceivably enable them immediately on page load in the mobile context, where we seem to be having issues, but if they are enabled too early before the player is ready, they will have bad data and not work properly.

Questions

Environment (please complete the following information):

Release: Is this issue associated with a specific release? [e. g. 7.6]

Additional notes Any additional information or notes that may be helpful in diagnosing or fixing the problem.

elynema commented 2 weeks ago

Dananji is not sure how we could fix this better than we already have. The buttons are supposed to be on an interval that checks every second to see if the player is ready and then enable themselves.

elynema commented 2 weeks ago

In testing on avalon-staging, I was not able to replicate this issue on iOS Safari (iPhone), so this does feel like a regression. https://avalon-staging.dlib.indiana.edu/media_objects/n009w228g

joncameron commented 1 week ago

I tried on iOS Safari and Chrome on Android and could reproduce after many refreshes.

From Google Analytics, I looked at the past 12 months for traffic on /playlists from device category "mobile". Only 10 users on mobile used a /playlist/*/edit route, and 9 users on mobile were recorded for /playlist/new.