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

Fixing code as per suggestions on PR 771.

Browse Source
This commit is contained in:
Valera Rozuvan
2013-08-27 10:19:55 +03:00
parent e788d6ce37
commit ba954ff164
1 changed files with 5 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
common/lib/xmodule/xmodule/js/src/video/09_video_caption.js
View File

@@ -28,11 +28,9 @@ function () {
// Depending on whether captions file could be loaded, the following
// function invocation can succeed or fail. If it fails, we do not
// go on with binding handlers to events.
if (!state.videoCaption.renderElements()) {
return;
if (state.videoCaption.renderElements()) {
state.videoCaption.bindHandlers();
}
state.videoCaption.bindHandlers();
};
// ***************************************************************
@@ -102,9 +100,6 @@ function () {
this.videoCaption.subtitlesEl = this.el.find('ol.subtitles');
this.videoCaption.hideSubtitlesEl = this.el.find('a.hide-subtitles');
this.el.find('.video-wrapper').after(this.videoCaption.subtitlesEl);
this.el.find('.video-controls .secondary-controls').append(this.videoCaption.hideSubtitlesEl);
// Fetch the captions file. If no file was specified, then we hide
// the "CC" button, and return.
if (!this.videoCaption.fetchCaption()) {
@@ -113,6 +108,9 @@ function () {
return false;
}
this.el.find('.video-wrapper').after(this.videoCaption.subtitlesEl);
this.el.find('.video-controls .secondary-controls').append(this.videoCaption.hideSubtitlesEl);
this.videoCaption.setSubtitlesHeight();
if (this.videoType === 'html5') {