From 0a5d261fc422dc237f7e59f25aa742948eb960d5 Mon Sep 17 00:00:00 2001
From: Valera Rozuvan <valera@edx.org>
Date: Tue, 13 Aug 2013 12:40:01 +0300
Subject: [PATCH] For YouTube videos tabbing from Speeds to Volume closes
 Speeds dialog.

An old TODO item was done. It turns out a simple case of calling
the method to bind handlers after the Spees dialog was re-rendered.
---
 .../xmodule/js/src/video/08_video_speed_control.js        | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/common/lib/xmodule/xmodule/js/src/video/08_video_speed_control.js b/common/lib/xmodule/xmodule/js/src/video/08_video_speed_control.js
index 740a1aa63d..e8cb1e15ed 100644
--- a/common/lib/xmodule/xmodule/js/src/video/08_video_speed_control.js
+++ b/common/lib/xmodule/xmodule/js/src/video/08_video_speed_control.js
@@ -151,7 +151,7 @@ function () {
         $.each(this.videoSpeedControl.speeds, function(index, speed) {
             var link, listItem;
 
-            link = '<a href="#">' + speed + 'x</a>';
+            link = '<a class="speed_link" href="#">' + speed + 'x</a>';
 
             listItem = $('<li data-speed="' + speed + '">' + link + '</li>');
 
@@ -162,11 +162,7 @@ function () {
             _this.videoSpeedControl.videoSpeedsEl.prepend(listItem);
         });
 
-        this.videoSpeedControl.videoSpeedsEl.find('a')
-            .on('click', this.videoSpeedControl.changeVideoSpeed);
-
-        // TODO: After the control was re-rendered, we should attach 'focus'
-        // and 'blur' events once more.
+        _bindHandlers(this);
     }
 
 });