Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

fix: do not seek to live on first manual seek when autoplaying a live stream #6062

Merged
merged 1 commit into from
Jun 20, 2019

Conversation

misteroneill
Copy link
Member

Description

This addresses an issue where autoplay players playing live streams would seek to the live edge after the first seek a user initiated (via UI or API).

Specific Changes proposed

Before listening for the play/timeupdate combination when starting the LiveTracker, check if the player has already started playback.

Requirements Checklist

  • Feature implemented / Bug fixed
  • If necessary, more likely in a feature request than a bug fix
    • Change has been verified in an actual browser (Chome, Firefox, IE)
    • Unit Tests updated or fixed
      editors=1000#0))
  • Reviewed by Two Core Contributors

@misteroneill misteroneill changed the base branch from master to 7.5.x June 20, 2019 15:44
Copy link
Member

@gkatsev gkatsev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works well and verified that tests fail without the change.

@gkatsev gkatsev merged commit 8777c89 into 7.5.x Jun 20, 2019
@gkatsev gkatsev deleted the fix/live-autoplay branch June 20, 2019 17:41
gkatsev pushed a commit that referenced this pull request Jun 20, 2019
…e stream (#6062)

Before listening for the play/timeupdate combination when starting the LiveTracker, check if the player has already started playback.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants