milas/jellyfish-web
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

subtitlesync toggle: make one-line conditions

Browse source
This commit is contained in:
redSpoutnik 2020-04-24 16:49:13 +02:00
parent be8a1baa27
commit 3a815709cf
1 changed files with 4 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

12
src/components/subtitlesync/subtitlesync.js
View file

@ -151,14 +151,10 @@ define(['playbackManager', 'layoutManager', 'text!./subtitlesync.template.html',
/* eslint-disable no-fallthrough */ /* eslint-disable no-fallthrough */
switch (action) { switch (action) {
case undefined: case undefined:
// if showing subtitle sync is enabled // if showing subtitle sync is enabled and if there is an external subtitle stream enabled
if (playbackManager.isShowingSubtitleOffsetEnabled(player) && if (playbackManager.isShowingSubtitleOffsetEnabled(player) && playbackManager.canHandleOffsetOnCurrentSubtitle(player)) {
// if there is an external subtitle stream enabled // if no subtitle offset is defined or element has focus (offset being defined)
playbackManager.canHandleOffsetOnCurrentSubtitle(player)) { if (!(playbackManager.getPlayerSubtitleOffset(player) || subtitleSyncTextField.hasFocus)) {
// if no subtitle offset is defined
if (!(playbackManager.getPlayerSubtitleOffset(player) ||
// or being defined (element has focus)
subtitleSyncTextField.hasFocus)) {
// set default offset to '0' = 50% // set default offset to '0' = 50%
subtitleSyncSlider.value = "50"; subtitleSyncSlider.value = "50";
subtitleSyncTextField.textContent = "0s"; subtitleSyncTextField.textContent = "0s";