Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

chore(demo): reset player state before loading new media #122

Merged
merged 1 commit into from
Dec 4, 2023

Conversation

amtins
Copy link
Member

@amtins amtins commented Nov 30, 2023

Description

Ensures that the player has been reset before loading new media. This prevents the old media being displayed for a fraction of the time before the new media is loaded. It also allows media without a poster to inherit the poster from the previous media.

Screencast.from.30.11.23.16.29.58.webm

Changes made

  • reset player
  • reset error screen
  • reset title bar

Copy link

github-actions bot commented Nov 30, 2023

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements 98.36% 480/488
🟢 Branches 93.42% 227/243
🟢 Functions 100% 134/134
🟢 Lines 99.14% 463/467

Test suite run success

142 tests passing in 9 suites.

Report generated by 🧪jest coverage report action from 9ce3357

@amtins amtins added this to the Demo milestone Nov 30, 2023
@amtins amtins force-pushed the chore/demo-reset-player-state branch from 61c7738 to 671789b Compare December 4, 2023 09:59
Ensures that the player has been reset before loading new media. This prevents
the old media being displayed for a fraction of the time before the new media
is loaded. It also allows media without a poster to inherit the poster from
the previous media.

- reset player
- reset error screen
- reset title bar
@amtins amtins force-pushed the chore/demo-reset-player-state branch from 671789b to 9ce3357 Compare December 4, 2023 10:03
@amtins amtins added this pull request to the merge queue Dec 4, 2023
Merged via the queue into main with commit 01ffc74 Dec 4, 2023
3 checks passed
@amtins amtins deleted the chore/demo-reset-player-state branch December 4, 2023 10:42
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants