Browse Source

Fix change in video to work correctly

pull/3975/head
Jack Lukic 9 years ago
parent
commit
8a1027d63a
1 changed files with 23 additions and 4 deletions
  1. 27
      src/definitions/modules/embed.js

27
src/definitions/modules/embed.js

@ -147,6 +147,12 @@ $.fn.embed = function(parameters) {
module.debug('Creating embed object', $embed);
},
changeEmbed: function(url) {
$embed
.html( module.generate.embed(url) )
;
},
createAndShow: function() {
module.createEmbed();
module.show();
@ -158,9 +164,19 @@ $.fn.embed = function(parameters) {
$module
.data(metadata.source, source)
.data(metadata.id, id)
.data(metadata.url, url)
;
module.create();
if(url) {
$module.data(metadata.url, url);
}
else {
$module.removeData(metadata.url);
}
if(module.has.embed()) {
module.changeEmbed();
}
else {
module.create();
}
},
// clears embed
@ -340,6 +356,9 @@ $.fn.embed = function(parameters) {
},
has: {
embed: function() {
return ($embed.length > 0);
},
placeholder: function() {
return settings.placeholder || $module.data(metadata.placeholder);
}
@ -613,7 +632,7 @@ $.fn.embed.settings = {
return {
autohide : !settings.brandedUI,
autoplay : settings.autoplay,
color : settings.colors || undefined,
color : settings.color || undefined,
hq : settings.hd,
jsapi : settings.api,
modestbranding : !settings.brandedUI
@ -631,7 +650,7 @@ $.fn.embed.settings = {
api : settings.api,
autoplay : settings.autoplay,
byline : settings.brandedUI,
color : settings.colors || undefined,
color : settings.color || undefined,
portrait : settings.brandedUI,
title : settings.brandedUI
};

Loading…
Cancel
Save