diff options
| author | Jules Laplace <julescarbon@gmail.com> | 2020-11-18 15:37:42 +0100 |
|---|---|---|
| committer | Jules Laplace <julescarbon@gmail.com> | 2020-11-18 15:37:42 +0100 |
| commit | d351d8dd915c5075983eaa5cac6f0d5a1b99a877 (patch) | |
| tree | 57a89011f201711a902870f7b9a23426d1d40413 /animism-align/frontend/app/views/viewer/player | |
| parent | 50411efc89fabb77c02b472446221e6bcd9ff621 (diff) | |
staggered, cancellable preloading of all section image assets, as soon as the section loads
Diffstat (limited to 'animism-align/frontend/app/views/viewer/player')
| -rw-r--r-- | animism-align/frontend/app/views/viewer/player/components.fullscreen/fullscreen.image.js | 20 | ||||
| -rw-r--r-- | animism-align/frontend/app/views/viewer/player/player.container.js | 13 |
2 files changed, 16 insertions, 17 deletions
diff --git a/animism-align/frontend/app/views/viewer/player/components.fullscreen/fullscreen.image.js b/animism-align/frontend/app/views/viewer/player/components.fullscreen/fullscreen.image.js index 60ba7c9..a05789b 100644 --- a/animism-align/frontend/app/views/viewer/player/components.fullscreen/fullscreen.image.js +++ b/animism-align/frontend/app/views/viewer/player/components.fullscreen/fullscreen.image.js @@ -1,29 +1,21 @@ import React from 'react' import { CURTAIN_COLOR_LOOKUP } from 'app/constants' +import { displayElementImageURL } from 'app/utils/annotation.utils' + export const FullscreenImage = ({ element, media, transitionDuration }) => { const color = element.color || CURTAIN_COLOR_LOOKUP.white - const item = media.lookup[element.settings.media_id] + const mediaItem = media.lookup[element.settings.media_id] const style = { backgroundColor: color.backgroundColor, color: color.textColor, transitionDuration, } - let url; - // console.log(element, item) - if (item.type === 'gallery') { - const index = parseInt(element.settings.frame_index) - const frame_id = item.settings.image_order[index] - const frame = item.settings.display_lookup[frame_id] - if (!frame) { - console.error("Slide not found:", element.settings.frame_index) - return <div /> - } - url = frame.url - } else { - url = item.settings.display.url + let url = displayElementImageURL(element, mediaItem) + if (!url) { + return <div /> } return ( diff --git a/animism-align/frontend/app/views/viewer/player/player.container.js b/animism-align/frontend/app/views/viewer/player/player.container.js index 87a325b..8602284 100644 --- a/animism-align/frontend/app/views/viewer/player/player.container.js +++ b/animism-align/frontend/app/views/viewer/player/player.container.js @@ -2,7 +2,8 @@ import React, { Component } from 'react' import { connect } from 'react-redux' import actions from 'app/actions' -import { floatEQ, floatInRange, clamp } from 'app/utils' +import { floatInRange, clamp } from 'app/utils' +import { preloadSectionImages } from 'app/utils/image.utils' import PlayerTranscript from './player.transcript' import PlayerFullscreen from './player.fullscreen' @@ -70,8 +71,13 @@ class PlayerContainer extends Component { } componentDidUpdate(prevProps) { - if (this.props.audio.play_ts === prevProps.audio.play_ts) return - this.handleTimeUpdate() + if (this.props.audio.play_ts !== prevProps.audio.play_ts) { + this.handleTimeUpdate() + } + if (this.props.currentSection !== prevProps.currentSection) { + this.preloader && this.preloader.cancel() + this.preloader = preloadSectionImages(this.props.currentSection, this.props.mediaLookup) + } } handleTimeUpdate() { @@ -114,6 +120,7 @@ class PlayerContainer extends Component { const mapStateToProps = state => ({ viewer: state.viewer, audio: state.audio, + mediaLookup: state.media.index.lookup, sections: state.viewer.sections, currentSection: state.viewer.currentSection, autoAdvance: state.viewer.autoAdvance, |
