pixiv の javascript のコメントが面白い
お絵描きサイト、pixiv！
何気なく、pixiv の javascript のソースを眺めてみると、いろんなコメントがありました。
ニョーン
... 7686: // マウスオーバーでおすすめタグをニョーンと出す 7687: var timer = null; ...
マジ天使
... 452: selectCover: function(c) { 453: // [IE マジ天使] 送れないので遷移させる 454: if ($.browser.msie) { ...
pixiv.group.js
http://source.pixiv.net/source/js/pixiv.group.js
なげえｗ
... 248: if (handler) { 249: // TODO 引数再考。なげえ 250: handler = handler.call(self, { ...
既に target なのでは
... 272: args.unshift({ 273: element : target, // TODO targetでいいのでは 274: elements: elements, ...
抜け穴
... 353: // ちょっと無理がありそう。抜け穴たくさんあるし、使い勝手が変わりすぎる 354: // data string | jQuery object ...
(´Д`；)
... 436: w.alert(data && data.error + (data.hint ? '\n\n' + data.hint : '') || status + ': ' + (error || 'error')); 437: self.trigger('click.reposition-cover', '.action-reposition-cover'); // 微妙。。 438: }); ...
マジ天使
... 452: selectCover: function(c) { 453: // [IE マジ天使] 送れないので遷移させる 454: if ($.browser.msie) { ...
効果味
... 459: // 同じ画像を再度選択した際にも change イベントが発火するように、一度 value を空にする（この際は change イベントは発生しない） 460: // 現状では選択した瞬間に送信するので特に効果味ない 461: c.element.findBy('input[type="file"]', 'form').val('').click(); ...
難しい
... 564: templates: { 565: // 間違って true になる値を入れてしまうとデバッグが難しい 566: 'enable-invite': null ...
嫌
... 757: .toggleClass('active') 758: .findBy('.filecontainer', 'form') // これを毎回やるのが嫌。コンテキストごとにオブジェクト分けたい 759: .toggle() ...
疑問
... 1252: ['focus', '.ui-auto-select', 'auto-select'], 1253: ['click', '.ui-selectbox li', 'selectbox-select', {cancel: false}] // デフォルトオンはよくないかも 1254: ], ...
pixiv.js
http://source.pixiv.net/source/js/pixiv.js
便利、、
... 2056: // TODO newした方が圧倒的に便利、、 2057: pixiv.modal = { ...
いいんじゃないかな？
... 2191: function uiLoadContextShareButton(e, data) { // TODO replace的なメソッドがあればいい？ 2192: $('#template-ui-context-share-button') ...
頑張って
... 2194: .appendTo(this) 2195: .filter('.share-button-twitter').each(pixiv.ui.shareButton.twitter).end() // TODO 最適化出来そう 2196: .filter('.share-button-facebook').each(pixiv.ui.shareButton.facebook); ...
･･･。
... 4124: // - デフォルト: async 4125: // \# 第三引数は（ xhr オブジェクト）は模倣しない（出来ない…）。 4126: d.resolve(data, status || 'async'); ...
1px
4481: if (!this.container) return; 4482: // TODO 1px隙間が出来る 4483: item.animate({
開かん
5664: // [IE][Fx] 開かん。 modal.open で一括して非同期化する方がいいかも 5665: w.setTimeout(function() {
実行･･･？
... 6482: //====================================== 6483: // 評価されたら実効 6484: //====================================== ... 6525: //====================================== 6526: // 質の評価されたら実効 6527: //====================================== ...
とりあえず
... 7628: var recent_tags = []; 7629: // TODO: pixiv.storage.localStorage の使い方が分かりにくいのでとりあえず window.localStorage を使う 7630: if (window.localStorage && window.JSON) { ...
ニョーン
... 7686: // マウスオーバーでおすすめタグをニョーンと出す 7687: var timer = null; ...
分かりにくい
... 7711: //====================================== 7712: // TODO 名前が分かりにくい 7713: window.add_form = function(id, id_minor) { ...
pixiv 以外のサイトの js とか css
はてな
http://d.st-hatena.com/statics/js/diary_utils.js
... 3: // 似たようなクラスが既にありそう 4: FormVerifier = new Ten.Class({ ... 85: // 分離したのでこれはいらない、と 86: // TextInputDescriptionOption = new Ten.Class({
ニコニコ
http://res.nimg.jp/js/watch/channel/channel.js
... 14: no_connect : 0,//api叩かないモード ... 55: /***デバッグ用の便利ボタンを挿入***/ ... 92: //入れ物をつっこむ 93: if(typeof ch_prof_inserted=="undefined"){//もう入れてたらやらない ...
Ameba
http://stat100.ameba.jp/blog/js/blog_top/textbox_focus.js
... 45: //アンカー生かし 46: if(_helpOverFlg ==true) return; 47: 48: 49: //消す 50: $(e.target.parentNode).children(".inputDetail").css("display","none"); 51: 52: 53: //色 54: $(e.target).removeClass("textInputFocus"); ...
シンプルなコメントですね。
島根県
緊急事態に備えているようです
http://www.pref.shimane.lg.jp/stylesheets/top.css
... 170: /* 171: * The style for emergency section that replaces 172: * photo when emergency info exsist 173: */ ...
vim
http://www.pref.shimane.lg.jp/javascripts/common.js
...
408: // vim: set sw=4 ts=4 noexpandtab :
