Commit 0a5d261f by Valera Rozuvan

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.
parent 05539b1b
...@@ -151,7 +151,7 @@ function () { ...@@ -151,7 +151,7 @@ function () {
$.each(this.videoSpeedControl.speeds, function(index, speed) { $.each(this.videoSpeedControl.speeds, function(index, speed) {
var link, listItem; 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>'); listItem = $('<li data-speed="' + speed + '">' + link + '</li>');
...@@ -162,11 +162,7 @@ function () { ...@@ -162,11 +162,7 @@ function () {
_this.videoSpeedControl.videoSpeedsEl.prepend(listItem); _this.videoSpeedControl.videoSpeedsEl.prepend(listItem);
}); });
this.videoSpeedControl.videoSpeedsEl.find('a') _bindHandlers(this);
.on('click', this.videoSpeedControl.changeVideoSpeed);
// TODO: After the control was re-rendered, we should attach 'focus'
// and 'blur' events once more.
} }
}); });
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment