Mirrors/webamp
1
0
Fork 0
mirror of https://github.com/captbaritone/webamp.git synced 2026-07-25 19:13:54 +00:00
Code Issues Projects Releases Wiki Activity

Fix loading skins from context menu

Browse source 
I had forgotten to include MyFile in context.js. Additionally, for some reason
`self` was out of scope in the callback.
This commit is contained in:
Jordan Eldredge 2015-04-11 11:03:50 -07:00
parent cf12fd4fec
commit 624292e457
1 changed files with 7 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

13
js/context.js
View file

@ -1,4 +1,5 @@
define({
define(['file'], function(MyFile) {
return {
init: function(winamp) {
this.winamp = winamp;
@ -17,7 +18,7 @@ define({
var skinSelectNodes = document.getElementsByClassName('skin-select');
for(var i = 0; i < skinSelectNodes.length; i++) {
skinSelectNodes[i].onclick = this._loadSkin;
skinSelectNodes[i].onclick = this._loadSkin.bind(this);
}
document.getElementById('context-play-file').onclick = function(event) {
@ -33,10 +34,10 @@ define({
};
},
_loadSkin: function() {
_loadSkin: function(event) {
var skinFile = new MyFile();
skinFile.setUrl(this.dataset.skinUrl);
self.winamp.setSkin(skinFile);
skinFile.setUrl(event.target.dataset.skinUrl);
this.winamp.setSkin(skinFile);
}
};
});