Skip to content

Fix repeating previous video in case the new one fail to load when recycling players

Martin Santangelo requested to merge js/fix-video-error-handling-recycling into master

Ticket(s) / Related Merge Requests

#4673 (closed)

Summary of Changes

The problem was caused by the recycling of the video players when the next video had a problem.

  • Display error overlay when the video fails to load and cancel the reproduction
  • Black background in the error overlay to hide the frame of the previous video in case of error (expo video doesn't provide a clear functionality)
  • Added a cancelable promise to the video init (preload) in case the video info fetch overlaps with the recycling of the component
  • Added a retry option in the error message

Testing Considerations

Check https://www.minds.com/andyleitermann/ channel There are some videos with errors that cause the problem

Regression Scope

(Can this change cause another regression issue? Which areas to focus on for regression testing?)

Platform Affected (web, mobile, etc)

both

Developer Testing Completed

Android device

Screenshots

imagen

Does this impact

  • Localization
  • Dark/light mode
  • Guest mode

Merge request reports