MOBILE-4082 video: Fix subtitles wider than video

main
Dani Palou 2022-05-24 11:02:30 +02:00
parent 782d94f6c0
commit 995b4756c0
2 changed files with 7 additions and 5 deletions

View File

@ -29,7 +29,6 @@ import { CoreFilepool, CoreFilepoolFileActions, CoreFilepoolFileEventData } from
import { CoreSites } from '@services/sites'; import { CoreSites } from '@services/sites';
import { CoreUrlUtils } from '@services/utils/url'; import { CoreUrlUtils } from '@services/utils/url';
import { CoreUtils } from '@services/utils/utils'; import { CoreUtils } from '@services/utils/utils';
import { Platform } from '@singletons';
import { CoreLogger } from '@singletons/logger'; import { CoreLogger } from '@singletons/logger';
import { CoreError } from '@classes/errors/error'; import { CoreError } from '@classes/errors/error';
import { CoreSite } from '@classes/site'; import { CoreSite } from '@classes/site';
@ -304,12 +303,10 @@ export class CoreExternalContentDirective implements AfterViewInit, OnChanges, O
return; return;
} }
const line = Platform.is('tablet') || CoreApp.isAndroid() ? 90 : 80;
// Position all subtitles to a percentage of video height. // Position all subtitles to a percentage of video height.
// eslint-disable-next-line @typescript-eslint/no-explicit-any Array.from(track.cues).forEach((cue: VTTCue) => {
Array.from(track.cues).forEach((cue: any) => {
cue.snapToLines = false; cue.snapToLines = false;
cue.line = line; cue.line = 90;
cue.size = 100; // This solves some Android issue. cue.size = 100; // This solves some Android issue.
}); });
// Delete listener. // Delete listener.

View File

@ -1725,3 +1725,8 @@ ion-header.no-title {
bottom: 0; bottom: 0;
} }
} }
// Fix subtitles wider than video.
video::-webkit-media-text-track-display {
white-space: normal !important;
}