vimpr / vimperator-plugins

for Vimperator plugins
http://vimpr.github.com/
408 stars 123 forks source link

direct_bookmark.jsに現在のタブ以外のURLをブクマできる機能がほしい (ほぼ実装済み) #18

Closed tyru closed 12 years ago

tyru commented 12 years ago

自分用に欲しかったのでdirect_bookmark/sbm-url-optionブランチで実装してます。 ただ--urlで指定されたURLが現在のタブ以外だったらgetTitleByURL()という追加した関数でGETしてtitleを取得してるのですが、charsetを考慮してる訳ではないので日本語が文字化けします。 JSでエンコーディング変換するのは面倒な気がするので(そもそも自分はあまりJSを本気で使ったことないのでこういう場合のベストプラクティスを知らない)、何かいい方法あったら教えてほしいです。

あと--url:openみたいな補完をつけたいなーとか。 'complete'オプション見て自前で実装するしかないんでしょうか。 vimperator側で公開してる(内部用の関数しかない気がするけど)関数とかあればそれも教えてほしいです。

教えて君ですみません...

tyru commented 12 years ago

あ、すみません。説明不足。 はてなとdeliciousでしかテストしてません。 文字化けるのはdeliciousのみです。 そもそもはてなの場合、titleはAPIに送信しません。(はてな側で取ってくる?)

anekos commented 12 years ago

補完のソースは何があるんといいんでしょうね。 たしかオプションの補完は非同期なのができないんですよね。 Vimperator 本体をいじるべきなのかも…

tyru commented 12 years ago

遅くなってすみません。

非同期でなくてもいいので、自分としては:openの補完と同じものが使えればいいかなーと。 そのためのAPIがなかったらほしいですね。 あ、それだとバッファの補完の時に「1: ほげタイトル」みたいになってURLじゃないものまで挿入されてしまう可能性があるので、:set complete=slぐらいの補完ができるとうれしいですね。(自分はset complete=stlでtはいらないので)

anekos commented 12 years ago

非同期だから、不便というわけじゃなくて、非同期前提の補完は流用できませんねってことです。

anekos commented 12 years ago

もうひとつコマンド用意するってのはどうですかね。 sbmなんとか みたいな名前で。 それで一つめを URL にすると。

anekos commented 12 years ago

ちゃんとコード見て検討してるわけじゃないのでむずかしかったらすみません。はい。

anekos commented 12 years ago

:sbmother なるコマンドをためしに作ってみました。 (もっと良い名前とかにかえてくださいw)

https://github.com/vimpr/vimperator-plugins/tree/direct_bookmark/sbm-add-url-command

anekos commented 12 years ago

こちらの判断で勝手に、hsl にしてあります。 設定できるようにしても、いいかもですね。

tyru commented 12 years ago

遅くなってすみません。 動いてるっぽいです。 マージよろしくおねがいします。(Twitterでも言ったけど)

tyru commented 12 years ago

の前に古いコメントとか削除しました。 92c05a3d71b74a20f8fb080595e3121533a75d6a

tyru commented 12 years ago

マージの件、自分はこれでいいと思います。

anekos commented 12 years ago

マージしました

tyru commented 12 years ago

どうもです。