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 missing details if subtitle changed synchronously after loaded and before onLoaded callback #6424

Merged
merged 1 commit into from
May 22, 2024

Conversation

iamboorrito
Copy link
Collaborator

This PR will...

Fix issue where changing subtitle before onLoaded is called drops the details from the track

Why is this Pull Request needed?

Resolves #6402

Are there any points in the code the reviewer needs to double check?

Resolves issues:

#6402

Checklist

  • changes have been done against master branch, and PR does not conflict
  • new unit / functional tests have been added (whenever applicable)
  • API or design changes are documented in API.md

@iamboorrito iamboorrito requested a review from robwalch May 16, 2024 23:52
@robwalch robwalch modified the milestones: 1.6.0, 1.5.9 May 17, 2024
@robwalch robwalch merged commit a16912b into master May 22, 2024
16 checks passed
@robwalch robwalch deleted the u/iamboorrito/bugfix/subtitleLoadSwitch branch May 22, 2024 19:33
robwalch pushed a commit that referenced this pull request May 22, 2024
…d before onLoaded callback (#6424)

(cherry picked from commit a16912b)
robwalch added a commit that referenced this pull request May 31, 2024
* patch/v1.5.x:
  Update github actions
  chore(deps): update tjenkinson/gh-action-auto-merge-dependency-updates action to v1.4.2 (#6333)
  Emit FRAG_PARSING_ERROR on unsupported M2TS codec (#6447)
  Fix JS AES fallback when browserCrypto.subtle returns undefined (rather than null) (#6446)
  Fix 608 caption TextTrack Cue seek/discontinuity timing regression introduced in v1.5.0 with #5557
  Fix 608 parser handling of redundant control codes Fixes #6427
  Fix backtracking when an appended segment has no buffered timerange (#6434)
  Fix missing details if subtitle changed synchronously after loaded and before onLoaded callback (#6424)
  Bugfix: Handle irregular white-space in segment URIs (#6396)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Disabling inflight subtitle playlist prevents segments from ever loading
2 participants