tDiary

On 17/03/2011 17:06, KADO Masanori wrote:
> comment_enter/leave_proc が欲しいです。
> プラグインでHTMLを挿入する場所を追加するためです。
> （ex. ツッコミの下にzenbackを表示したいなど）

お気持ちわかります。:)
ちなみに、どれくらい遅くなりますか？

# この skel/diary.rhtml での同一引数のTime::at呼びまくりも、ちょっともっ
たいない気がしないでもない。

かずひこ

2011年3月18日1:24 Kazuhiko <kazuhiko@...>:
> On 17/03/2011 17:06, KADO Masanori wrote:
>> comment_enter/leave_proc が欲しいです。
>> プラグインでHTMLを挿入する場所を追加するためです。
>> （ex. ツッコミの下にzenbackを表示したいなど）
>
> お気持ちわかります。:)
> ちなみに、どれくらい遅くなりますか？

abした限りでは特に変化ありませんでした。

> # この skel/diary.rhtml での同一引数のTime::at呼びまくりも、ちょっともっ
> たいない気がしないでもない。

あ、そうですね。このファイルで6箇所ありますね。

-- 
kdmsnr - KADO, Masanori

柴田です。

> comment_enter/leave_proc が欲しいです。
> プラグインでHTMLを挿入する場所を追加するためです。
> （ex. ツッコミの下にzenbackを表示したいなど）

サイトによってはコメントが凄いことになるような気もしますけど、
だいたい1つか2つなので私は導入に賛成です。

たださんどうでしょう?

-- 
SHIBATA Hiroshi shibata.hiroshi@...
http://www.hsbt.org/

ただただしです。

なかなか反応できなくてすみません。

2011年3月18日1:06 KADO Masanori <kdmsnr@...>:
> comment_enter/leave_proc が欲しいです。
> プラグインでHTMLを挿入する場所を追加するためです。
> （ex. ツッコミの下にzenbackを表示したいなど）
>
> 関連チケット：
> https://github.com/tdiary/tdiary-contrib/issues/6
> https://github.com/tdiary/tdiary-core/issues/17
>
> 以下、パッチです。挿入場所がココで大丈夫かどうかが不安です。

もっとも単純な実装としてはこれでも良いとは思いますが、ちょっと懸念が。

まず、このパッチだとenterとleaveの間にツッコミ、ツッコミフォームととリ
ンク元が入るようになっていますが、これでいいのか。ツッコミとリンク元
の間に何かを入れたいという要求も出てきそうな気がします。

現在、すでにツッコミフォームとリンク元はプラグインに逃がしてあるので、
これらもcomment_leave_proc内で処理するようにしてskel/diary.rhtmlからは
削るというのもアリかなと思います。

そうなるとツッコミそのものだけを本体側で埋め込むのもおかしな話なので、
これもプラグインに追いだそうとするとちょっと大掛かりな話になってしま
う。でもそうするとenter/leaveを分けるまでもなくcomment_procひとつで
済むという利点が出てきます。

というわけで、最終的にcomment_proc(もしくはより妥当な名前があると良
い)ひとつで済むような展望をもちつつ実装したくないですかね?

-- 
ただただし - ツッコミは、短く鋭く愛を込めて♪
http://tdtds.jp/
@tdtds

kdmsnrです。

2011年3月21日16:47 TADA Tadashi <t@...>:
> というわけで、最終的にcomment_proc(もしくはより妥当な名前があると良
> い)ひとつで済むような展望をもちつつ実装したくないですかね?

procをひとつにすると、ツッコミとリンク元の間に何かを挿入したい、
というのが難しくなりそうな気がしました。（メソッドの評価順？）

ここでは、単純に comment_enter/leave_proc と referer_enter/leave_proc の
4箇所を用意しておけば、たいていの要求は満たせそうです。

表示順がうまくできればひとつにまとめられそうですが、
add_xxx_proc の引数に優先度を渡すとかですかねえ。

-- 
kdmsnr - KADO, Masanori

ただただしです。

2011年3月21日18:22 KADO Masanori <kdmsnr@...>:
> procをひとつにすると、ツッコミとリンク元の間に何かを挿入したい、
> というのが難しくなりそうな気がしました。（メソッドの評価順？）

評価順はすべてのプラグインにまつわる課題ですからねぇ。

> ここでは、単純に comment_enter/leave_proc と referer_enter/leave_proc の
> 4箇所を用意しておけば、たいていの要求は満たせそうです。
>
> 表示順がうまくできればひとつにまとめられそうですが、
> add_xxx_proc の引数に優先度を渡すとかですかねえ。

評価順については別途議論するとして、いま要件があるのはcomment_leave
だけなのだとしたら、それだけ実装しておくのはOKだと思います(のちに
メソッド名を変えるかも知れないけど)。

その場合、ツッコミフォームのあとにするのか、リンク元のあとにする
のかは決めないといけないけれど。

-- 
ただただし - ツッコミは、短く鋭く愛を込めて♪
http://tdtds.jp/
@tdtds

kayakayaです。

2011年3月21日22:35 TADA Tadashi <t@...>:
> ただただしです。
>
> 2011年3月21日18:22 KADO Masanori <kdmsnr@...>:

>> ここでは、単純に comment_enter/leave_proc と referer_enter/leave_proc の
>> 4箇所を用意しておけば、たいていの要求は満たせそうです。

> 評価順については別途議論するとして、いま要件があるのはcomment_leave
> だけなのだとしたら、それだけ実装しておくのはOKだと思います(のちに
> メソッド名を変えるかも知れないけど)。
>
> その場合、ツッコミフォームのあとにするのか、リンク元のあとにする
> のかは決めないといけないけれど。

今回のcomment_enter/leave_procの要望は、
zenbackをツッコミ欄の後ろに表示したいという需要からでした。
このプラグイン修正「だけ」の期待で言えば、ツッコミフォームの後になります。

プラグイン側の利用者としては、
将来的に、comment_enter/leave_proc と
referer_enter/leave_procの"4つ"が実装されるならば、
なおさら、comment_enter/leave_procは、リンク元ではなく、
ツッコミフォームの前後として実装して良いと思います。
今後、referer_enter/leave_procが実装されれば、より便利になるわけで。

# ちなみに、現在の自分の日記はjQueryで
# zenbackの無理矢理ツッコミフォームの後ろ、リンク元の前に表示させています。
# ただ、zenback.rbが(アルファベット順では)最後に呼び出される振る舞いに依存してるため、
# コミットするレベルではなく、本体側で対応してほしいなぁという思いがあります。

ではでは。

-- 
KAYA Satoshi
http://kayakaya.net/

kdmsnrです。

2011年3月21日22:35 TADA Tadashi <t@...>:
> 評価順については別途議論するとして、いま要件があるのはcomment_leave
> だけなのだとしたら、それだけ実装しておくのはOKだと思います(のちに
> メソッド名を変えるかも知れないけど)。

comment_leave_procだけ変更したものをpull requestしておきました。
https://github.com/tdiary/tdiary-core/pull/18

> その場合、ツッコミフォームのあとにするのか、リンク元のあとにする
> のかは決めないといけないけれど。

kayakyaさんの意見を反映して、ツッコミフォームのあとにしてみました。

-- 
kdmsnr - KADO, Masanori