-
Notifications
You must be signed in to change notification settings - Fork 51
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
feat(FEC-10766): create text config section and option for styling #387
Conversation
Issue: text track doesn't have a config for styling, there is only an API for styling. Solution: create a text container for the text track and add the options for styling the text track.
playback: { | ||
enableCEA708Captions: true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe leave this test to check backward compatibility and add a comment to mention that?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, but its one config for all tests... Nevermind
src/kaltura-player.js
Outdated
@@ -434,6 +434,10 @@ class KalturaPlayer extends FakeEventTarget { | |||
return this._localPlayer.textStyle; | |||
} | |||
|
|||
get textDisplaySetting(): Object { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
move next to its setter
Description of the Changes
Issue: text track doesn't have a config for styling, there is only an API for styling.
Solution: create a text container for the text track and add the options for styling the text track.
CheckLists