操作
バグ #633
完了
140文字を超えるメッセージについて、141文字目以降に含まれるURLが認識されない。
プラグイン名:
mui
ブランチ:
クラッシュする:
説明
明けましておめでとうございます。
今年もよろしくお願いします。
暇つぶしに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では現状あり得ない状況なのですが、次期バージョンのマルチサービスの時に引っかかりそうなので、修正ご検討頂きたいなと。
toshi_a 初音 さんが約10年前に更新
- ステータス を パッチ適用待ち から レビュー待ち に変更
- 担当者 を toshi_a 初音 から Satoshi Okuno に変更
- 対象バージョン を 0.2 から 0.2.2 に変更
確かに、Twitterだけだと問題起こりませんが、これはおかしいですね。message[:text]でもいいかなと思ったのですが、頂いた方法を採用することにしました。
bug/633 ブランチに適用してみました。 Gdk::MiraclePainter#main_message は結構呼び出しコスト高そうなので、一時変数を作成して呼び出し回数を削減してみました (2014497e)。
操作