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

Poor UI Behavior with Roles #2307

Closed
bloomtom opened this issue Dec 18, 2019 · 3 comments
Closed

Poor UI Behavior with Roles #2307

bloomtom opened this issue Dec 18, 2019 · 3 comments
Assignees
Labels
component: UI The issue involves the Shaka Player UI status: archived Archived and locked; will not be updated type: bug Something isn't working correctly
Milestone

Comments

@bloomtom
Copy link
Contributor

bloomtom commented Dec 18, 2019

Have you read the FAQ and checked for duplicate open issues?
Yes

What version of Shaka Player are you using?
v2.5.6

Can you reproduce the issue with our latest release version?
Yes

Can you reproduce the issue with the latest code from master?
Yes

Are you using the demo app or your own custom app?
Both

If custom app, can you reproduce the issue using our demo app?
Yes

What browser and OS are you using?
Firefox 71 and Chrome 79

For embedded devices (smart TVs, etc.), what model and firmware version are you using?
N/A

What are the manifest and license server URIs?
A test manifest is available here.
Demo Link

What did you do?
Produce a DASH manifest containing multiple audio or subtitle tracks in different adaptation sets with the same language code but different roles.

What did you expect to happen?
Audio and subtitle tracks with the same language but different roles appear showing both the language and the role, and the user can select any of these roles.

What actually happened?
Audio roles are not visible. Only a single element for each language is visible. The user cannot switch between roles, and they probably don't even know more than one role exists.
Subtitle roles are visible as duplicates, but selecting one causes all of them to be checked in the UI, and only one is actually displayed. The user cannot select the role they want.

I can select these roles using selectTextLanguage and selectAudioLanguage, but the UI remains inconsistent.

Comparison of Shaka and the DASH reference player.
EryOZzs0

MPD attached as a txt

@theodab
Copy link
Contributor

theodab commented Dec 18, 2019

Yeah, this looks like a problem. Thanks for the report!

@theodab theodab added type: bug Something isn't working correctly component: UI The issue involves the Shaka Player UI and removed needs triage labels Dec 18, 2019
@shaka-bot shaka-bot added this to the v2.6 milestone Dec 18, 2019
@theodab
Copy link
Contributor

theodab commented Dec 20, 2019

Ok, I've changed the UI to show each role for a given language.
I also added a new UI configuration, trackLabelFormat, which determines how we make the labels for audio and text language selections. It defaults to shaka.ui.TrackLabelFormat.LANGUAGE, which just shows the language; you might want to set it to shaka.ui.TrackLabelFormat.ROLE, to show the role but not the language, since it looks like the roles in your content are designed to be displayed without their associated language.

If you think that we need to do any more work to get role selection up to snuff, feel free to re-open this issue and tell us. The person who is normally in charge of UI stuff is out right now, and I might have made some incorrect design decision here.

@theodab theodab self-assigned this Dec 20, 2019
joeyparrish pushed a commit that referenced this issue Jan 15, 2020
Closes #2307

Backported to v2.5.x

Change-Id: I13ce5d6185f27a4bfa7481eff4fef629520ff0ce
@joeyparrish
Copy link
Member

These changes have been cherry-picked for v2.5.8.

@shaka-project shaka-project locked and limited conversation to collaborators Feb 18, 2020
@shaka-bot shaka-bot added the status: archived Archived and locked; will not be updated label Apr 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
component: UI The issue involves the Shaka Player UI status: archived Archived and locked; will not be updated type: bug Something isn't working correctly
Projects
None yet
Development

No branches or pull requests

4 participants