Skip to content

Commit

Permalink
fix(FEC-11126): moved useShakaTextTrackDisplay from playback.options.…
Browse files Browse the repository at this point in the history
…playback.options.html5.dash to text (#432)
  • Loading branch information
RoyBregman authored Apr 12, 2021
1 parent fd08311 commit bd6022e
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 6 deletions.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@
},
"dependencies": {
"@babel/polyfill": "^7.0.0",
"@playkit-js/playkit-js": "0.69.0",
"@playkit-js/playkit-js": "canary",
"@playkit-js/playkit-js-dash": "1.23.0",
"@playkit-js/playkit-js-hls": "1.24.0",
"@playkit-js/playkit-js-ui": "0.65.0",
Expand Down
3 changes: 2 additions & 1 deletion src/common/utils/setup-helpers.js
Original file line number Diff line number Diff line change
Expand Up @@ -336,7 +336,7 @@ function checkNativeHlsSupport(options: KPOptionsObject): void {
* @returns {void}
*/
function checkNativeTextTracksSupport(options: KPOptionsObject): void {
if ((Env.isMacOS && Env.isSafari) || Env.isIOS) {
if ((Env.isMacOS && Env.isSafari) || Env.isIOS || (options.text && options.text.useShakaTextTrackDisplay)) {
const useNativeTextTrack = Utils.Object.getPropertyPath(options, 'text.useNativeTextTrack');
if (typeof useNativeTextTrack !== 'boolean') {
Utils.Object.mergeDeep(options, {
Expand Down Expand Up @@ -564,6 +564,7 @@ function supportLegacyOptions(options: Object): PartialKPOptionsObject {
['ui.components.fullscreen.inBrowserFullscreenForIOS', 'playback.inBrowserFullscreen'],
['playback.enableCEA708Captions', 'text.enableCEA708Captions'],
['playback.useNativeTextTrack', 'text.useNativeTextTrack'],
['playback.options.html5.dash.useShakaTextTrackDisplay', 'text.useShakaTextTrackDisplay'],
['playback.captionsTextTrack1Label', 'text.captionsTextTrack1Label'],
['playback.captionsTextTrack1LanguageCode', 'text.captionsTextTrack1LanguageCode'],
['playback.captionsTextTrack2Label', 'text.captionsTextTrack2Label'],
Expand Down
8 changes: 4 additions & 4 deletions yarn.lock
Original file line number Diff line number Diff line change
Expand Up @@ -1150,10 +1150,10 @@
react-redux "^7.2.0"
redux "^4.0.5"

"@playkit-js/playkit-js@0.69.0":
version "0.69.0"
resolved "https://registry.yarnpkg.com/@playkit-js/playkit-js/-/playkit-js-0.69.0.tgz#21d53583350bc3804b8e2da7cd91ce9eb9507f9a"
integrity sha512-sWq/hZ3D2PGALOT2GTwJgKuKY1Kf5O9sjYjACeSissBN4VtPso/DzSOdlb3WDDMP3IF578zseHgQHn9mA9ABjg==
"@playkit-js/playkit-js@canary":
version "0.69.1-canary.26cfe24"
resolved "https://registry.yarnpkg.com/@playkit-js/playkit-js/-/playkit-js-0.69.1-canary.26cfe24.tgz#9ad7dcfd7e6f3696e6d1d5d7b5ede679f9945e6e"
integrity sha512-w/FWKYcPq4IVjWyX+6/7npA486G4Fdo3gnyqE6JAhMfY7EghFyLY6bYLwnalePqUPaSXUjsZdKSgY0/5T/JAKQ==
dependencies:
js-logger "^1.6.0"
ua-parser-js "^0.7.21"
Expand Down

0 comments on commit bd6022e

Please sign in to comment.