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(android)!: rework video tracks management #3778

Merged

Conversation

freeboub
Copy link
Collaborator

@freeboub freeboub commented May 15, 2024

Summary

Review and test video tracks selection by index

Motivation

Allow to select video tracks properly
This Pr is linked to #3324

Changes

  • Breaking change: trackId more from int to string
  • add index in video tracks object to match audio and subtitle tracks management
  • Fix typescript type (it was missing in onLoad)
  • Fix selection with ABR content (can be tested in the sample)
  • minor typo fixes
  • doc update
  • Sample: fix remaining types error
  • Sample: implement Video track selection
  • Sample: split video/audio/text Tracks components

Test plan

Can be tested with the sample:
play an asset with multiple video tracks and try to change playing video track

freeboub and others added 13 commits May 5, 2024 21:06
…ideo

# Conflicts:
#	android/src/main/java/com/brentvatne/exoplayer/ReactExoplayerViewManager.java
#	src/Video.tsx
…fix/avoidVideoResizingFlickering

# Conflicts:
#	.github/ISSUE_TEMPLATE/bug-report.yml
…ideo into fix/reworkVideoTracksManagement

# Conflicts:
#	android/src/main/java/com/brentvatne/exoplayer/ReactExoplayerView.java
@freeboub freeboub marked this pull request as draft May 15, 2024 19:46
Copy link
Collaborator

@KrzysztofMoch KrzysztofMoch left a comment

Choose a reason for hiding this comment

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

LGTM, let's merge!
PS: sorry for long time of checking it ^^`

@KrzysztofMoch KrzysztofMoch changed the title Fix/rework video tracks management fix(android)!: rework video tracks management May 22, 2024
@freeboub freeboub merged commit cad5c46 into TheWidlarzGroup:master May 22, 2024
6 checks passed
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.

None yet

2 participants