Actions
バグ #633
closed
140文字を超えるメッセージについて、141文字目以降に含まれるURLが認識されない。
プラグイン名:
mui
ブランチ:
クラッシュする:
Description
明けましておめでとうございます。
今年もよろしくお願いします。
暇つぶしにFacebookプラグインなどを作ってて、気になったとこです。
MiraclePainterのmain_pos_to_indexについて、バイト数からインデックスを求める処理で
140文字の切り詰められたmessage.to_sが基準として使われています。
そのため141文字目以降をクリックしてもインデックスが140となります。
結果として141文字目以降のURLが検索されず、クリックで開いたりURLがコピー出来なかったりと言う状況になりました。
TLに表示されてるテキストであるPango::Layout.textを基準にすると上手く行くようです。
https://github.com/moguno/mikutter-folk-song/commit/7ee8051aeac002b7e7512d1f4eef4a54d8d897c9
twitterでは現状あり得ない状況なのですが、次期バージョンのマルチサービスの時に引っかかりそうなので、修正ご検討頂きたいなと。
Updated by toshi_a 初音 about 12 years ago
- Status changed from 新規 to パッチ適用待ち
- Assignee set to toshi_a 初音
検証します。取り急ぎ把握しました。
Updated by toshi_a 初音 about 12 years ago
- Status changed from パッチ適用待ち to レビュー待ち
- Assignee changed from toshi_a 初音 to Satoshi Okuno
- Target version changed from 0.2 to 0.2.2
確かに、Twitterだけだと問題起こりませんが、これはおかしいですね。message[:text]でもいいかなと思ったのですが、頂いた方法を採用することにしました。
bug/633 ブランチに適用してみました。 Gdk::MiraclePainter#main_message は結構呼び出しコスト高そうなので、一時変数を作成して呼び出し回数を削減してみました (2014497e)。
Actions