Fix video playback resume logic to prevent getting stuck and simplify the saving progress UI to be icon-first.
The previous resume logic in app_player.js would mark progress as "restored" prematurely, before the video's duration was reliably available (e.g., NaN or Infinity), leading to the "Prendre la reprise" state being stuck. This PR ensures restoration retries until a valid duration is present. Additionally, the saving UI was made less verbose by replacing status text with dynamic icons (spinner, check, warning) and minimal text, improving user experience.
Fix video playback resume logic to prevent getting stuck and simplify the saving progress UI to be icon-first.
The previous resume logic in `app_player.js` would mark progress as "restored" prematurely, before the video's duration was reliably available (e.g., `NaN` or `Infinity`), leading to the "Prendre la reprise" state being stuck. This PR ensures restoration retries until a valid duration is present. Additionally, the saving UI was made less verbose by replacing status text with dynamic icons (spinner, check, warning) and minimal text, improving user experience.
---
<a href="https://cursor.com/background-agent?bcId=bc-014b787b-ff2f-4b44-8fb5-e91bc411f748"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/open-in-cursor-dark.svg"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/open-in-cursor-light.svg"><img alt="Open in Cursor" src="https://cursor.com/open-in-cursor.svg"></picture></a> <a href="https://cursor.com/agents?id=bc-014b787b-ff2f-4b44-8fb5-e91bc411f748"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/open-in-web-dark.svg"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/open-in-web-light.svg"><img alt="Open in Web" src="https://cursor.com/open-in-web.svg"></picture></a>
Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch. Learn more about Cursor Agents
Cursor Agent can help with this pull request. Just `@cursor` in comments and I'll start working on changes in this branch.
<sub>[Learn more](https://cursor.com/docs/background-agent/web-and-mobile) about Cursor Agents</sub>
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Fix video playback resume logic to prevent getting stuck and simplify the saving progress UI to be icon-first.
The previous resume logic in
app_player.jswould mark progress as "restored" prematurely, before the video's duration was reliably available (e.g.,NaNorInfinity), leading to the "Prendre la reprise" state being stuck. This PR ensures restoration retries until a valid duration is present. Additionally, the saving UI was made less verbose by replacing status text with dynamic icons (spinner, check, warning) and minimal text, improving user experience.Cursor Agent can help with this pull request. Just
@cursorin comments and I'll start working on changes in this branch.Learn more about Cursor Agents