LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues 596 Pull requests 6 Projects Releases 238 Packages Wiki Activity

Disable chapters for mobile for now...

Browse source 
- Deemed not useful in mobile, unless you have an S-pen that can hover.
- The chapters button (that invokes the chapters popup) would make more sense in Mobile, but we need to deal with the limited controlbar space first (e.g. overflow menu system)
This commit is contained in:
infinite-persistence 2022-05-04 20:38:13 +08:00 committed by Thomas Zarebczan
parent f2495df615
commit 4b6e4db7cd
1 changed files with 7 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
ui/component/viewers/videoViewer/internal/chapters.jsx
View file

@ -1,4 +1,6 @@
// @flow // @flow
import { platform } from 'util/platform';
const REQUIRED_DELAY_FOR_IOS_MS = 10; const REQUIRED_DELAY_FOR_IOS_MS = 10;
const MIN_SECONDS_BETWEEN_CHAPTERS = 10; const MIN_SECONDS_BETWEEN_CHAPTERS = 10;
const MIN_CHAPTERS = 3; const MIN_CHAPTERS = 3;
@ -148,6 +150,11 @@ function load(player: any, timestampData: TimestampData, duration: number) {
export function parseAndLoad(player: any, claim: StreamClaim) { export function parseAndLoad(player: any, claim: StreamClaim) {
console.assert(claim, 'null claim'); console.assert(claim, 'null claim');
if (platform.isMobile()) {
return;
}
const tsData = parse(claim); const tsData = parse(claim);
const duration = claim?.value?.video?.duration || claim?.value?.audio?.duration; const duration = claim?.value?.video?.duration || claim?.value?.audio?.duration;