Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: video fullscreen must not assume resizer exists (#22935)

Browse Source 
Otherwise it fails to go fullscreen before playback
This commit is contained in:
Adam Butterworth
2020-01-24 14:59:49 -05:00
committed by GitHub
parent 811d45c1ae
commit 48d20e338b
1 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
common/lib/xmodule/xmodule/js/src/video/04_video_full_screen.js
View File

@@ -188,8 +188,9 @@
.addClass('fa-arrows-alt');
$(closedCaptionsEl).css({top: '70%', left: '5%'});
this.resizer.delta.reset().setMode('width');
if (this.resizer) {
this.resizer.delta.reset().setMode('width');
}
this.el.trigger('fullscreen', [this.isFullScreen]);
}
@@ -210,8 +211,9 @@
.addClass('fa-compress');
$(closedCaptionsEl).css({top: '70%', left: '5%'});
this.resizer.delta.substract(this.videoFullScreen.updateControlsHeight(), 'height').setMode('both');
if (this.resizer) {
this.resizer.delta.substract(this.videoFullScreen.updateControlsHeight(), 'height').setMode('both');
}
this.el.trigger('fullscreen', [this.isFullScreen]);
}