2012/03/03

[jQuery][YouTube]tubularについて

YouTubeの動画を背景にできるtubularというプラグインがPHPSPOTのYoutube動画をページ背景にするjQueryプラグイン「tubular」で紹介されていました。

実際に、jquery.tubular.jsを読んでみて少し突っ込みたくなったので、何点か。

jQueryで、bodyタグにytapiplayerというidのdivタグを挿入しているのだが、これは、指定できた方が便利なのでは??

jQuery('body').prepend('<div id="yt-container" style="overflow: hidden; position: fixed; z-index: 1;"><div id="ytapiplayer">You need Flash player 8+ and JavaScript enabled to view this video.</div></div><div id="video-cover" style="position: fixed; width: 100%; height: 100%; z-index: 2;"></div>');
もっと欲を言えば、idがyt-containerのdivタグも指定できたほうがいいのでは??

理由は、単純に衝突をさけるため。

外部変数として、
var videoWidth = 853;
var videoRatio = 16/9; // either 4/3 or 16/9
があったが、グローバル変数になってしまうため、
jQuery.fn.tubular = function(videoId,wrapperId) {
 var videoWidth = 853;
 var videoRatio = 16/9; // either 4/3 or 16/9

}
とした方が、変数の衝突をさけることができるのになー。

resizePlayer関数もグローバル宣言をせずに、
jQuery.fn.tubular = function(videoId,wrapperId) {

function resizePlayer(){
//省略
}

t = setTimeout(function(){
resizePlayer();
},1000);

}
の方がスマートだと思うのだが。。。

まー他にも突っ込みたいところがいっぱいあったので、ちょっと改造しようかなー。。。

0 コメント:

コメントを投稿