Project

General

Profile

バグ #633

140文字を超えるメッセージについて、141文字目以降に含まれるURLが認識されない。

Added by Satoshi Okuno over 6 years ago. Updated over 6 years ago.

Status:
終了
Priority:
通常
Assignee:
Target version:
プラグイン名:
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では現状あり得ない状況なのですが、次期バージョンのマルチサービスの時に引っかかりそうなので、修正ご検討頂きたいなと。

#1

Updated by toshi_a 初音 over 6 years ago

  • Status changed from 新規 to パッチ適用待ち
  • Assignee set to toshi_a 初音

検証します。取り急ぎ把握しました。

#2

Updated by toshi_a 初音 over 6 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)。

#3

Updated by toshi_a 初音 over 6 years ago

  • Status changed from レビュー待ち to 終了

Also available in: Atom PDF