codeninjasllc/discourse
1
0
Fork 0
mirror of https://github.com/codeninjasllc/discourse.git synced 2024-11-27 17:46:05 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix: Set title for multiple youtube video player with the same ID (don't rely on id to set title)

Browse source 
When an identical youtube video is embedded multiple times on the same page. 
it will only be successful setting title to 1st element in the dom matching a unique id which is based on the actual youtube video id.
This commit is contained in:
lidlanca 2014-10-14 19:50:42 -04:00
parent cd8b896272
commit 17febe4914
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
plugins/lazyYT/assets/javascripts/lazyYT.js
View file

@ -33,9 +33,9 @@
}) })
.html('<p id="lazyYT-title-' + id + '" class="lazyYT-title"></p><div class="lazyYT-button"></div>') .html('<p id="lazyYT-title-' + id + '" class="lazyYT-title"></p><div class="lazyYT-button"></div>')
.addClass('lazyYT-image-loaded'); .addClass('lazyYT-image-loaded');
$el_title = $el.find("p.lazyYT-title"); //get reference to the current container title element
$.getJSON('https://gdata.youtube.com/feeds/api/videos/' + id + '?v=2&alt=json', function (data) { $.getJSON('https://gdata.youtube.com/feeds/api/videos/' + id + '?v=2&alt=json', function (data) {
$('#lazyYT-title-' + id).text(data.entry.title.$t); $el_title.text(data.entry.title.$t);
}); });
$el.on('click', function (e) { $el.on('click', function (e) {