[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); }

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

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