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

Addressed PR comments

Browse Source
This commit is contained in:
jmclaus
2013-09-24 12:12:10 +02:00
parent feb95eb869
commit 3a5d7d8638
2 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
common/lib/xmodule/xmodule/js/spec/video/video_speed_control_spec.js
View File

@@ -140,20 +140,22 @@
});
describe('make sure the speed control gets the focus afterwards', function() {
beforeEach(function() {
var anchor;
beforeEach(function() {
initialize();
anchor= $('.speeds > a').first();
videoSpeedControl.setSpeed(1.0);
spyOnEvent(state.videoSpeedControl.mainAnchor, 'focus');
spyOnEvent(anchor, 'focus');
});
it('when the speed is the same', function() {
$('li[data-speed="1.0"] a').click();
expect('focus').toHaveBeenTriggeredOn(state.videoSpeedControl.mainAnchor);
expect('focus').toHaveBeenTriggeredOn(anchor);
});
it('when the speed is not the same', function() {
$('li[data-speed="0.75"] a').click();
expect('focus').toHaveBeenTriggeredOn(state.videoSpeedControl.mainAnchor);
expect('focus').toHaveBeenTriggeredOn(anchor);
});
});
});

2
common/lib/xmodule/xmodule/js/src/video/08_video_speed_control.js
View File

@@ -62,8 +62,6 @@ function () {
state.videoSpeedControl.el = state.el.find('div.speeds');
state.videoSpeedControl.mainAnchor = state.videoSpeedControl.el.children('a');
state.videoSpeedControl.videoSpeedsEl = state.videoSpeedControl.el
.find('.video_speeds');