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

Add change event to audiotracks and texttracks #2134

Merged
merged 1 commit into from
Sep 4, 2019

Conversation

avelad
Copy link
Member

@avelad avelad commented Sep 4, 2019

No description provided.

@ismena
Copy link
Contributor

ismena commented Sep 4, 2019

LGTM. Is there an associated bug or something?

@shaka-bot
Copy link
Collaborator

All tests passed!

@avelad
Copy link
Member Author

avelad commented Sep 4, 2019

@ismena I haven't reported any problem, but what it does is it fires the event when the tracks change. For example when they are changed from the native UI and you need the event from the app. I have also seen that sometimes with the Shaka UI the audio language or subtitle is not changed correctly because of this.

@ismena ismena merged commit 6d31b57 into shaka-project:master Sep 4, 2019
@ismena
Copy link
Contributor

ismena commented Sep 4, 2019

Aaaand merged!

@avelad avelad mentioned this pull request Oct 22, 2019
@joeyparrish joeyparrish added the type: bug Something isn't working correctly label Oct 31, 2019
@joeyparrish joeyparrish added this to the v2.6 milestone Oct 31, 2019
TheModMaker pushed a commit that referenced this pull request Nov 6, 2019
Change-Id: I453a06e9fc4b217dcd42f1db62c66f7bc4d7fd0e
@avelad avelad deleted the nativeSrcChangeEvent branch March 16, 2022 16:00
@github-actions github-actions bot added the status: archived Archived and locked; will not be updated label Jul 25, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: archived Archived and locked; will not be updated type: bug Something isn't working correctly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants