機能 #652
closed
DMをMiraclePainterで表示する
100%
Description
DMの中にURLやユーザー名が入っていてもリンクにならなくて不便なので、いっそのことMiraclePainterで処理してしまえば便利な気がしますよね。
Related issues
Updated by toshi_a 初音 over 9 years ago
- Target version set to 3.1.0
良いと思いますが、一つ懸念点があります。
DMはMessageオブジェクトをもたないことです。
DMはツイートとは違ったデータスキームを持っているため、これが問題になることがあると考えています。具体的には、IDの衝突などです。このあたりはどういう風に回避する予定ですか？
これに限らず、困っているということがあれば、こちらでもなにか考えてみます。
Updated by Osamu Koga over 9 years ago
そのへんはMessageオブジェクトで対応するにも限界があるので、新しくDM用のクラスを作ってしまおうと思っています。
おそらくかなりの部分でMessageと重なる箇所があるので、mix-inを作ったほうがコードがきれいになりそうですが、それについては実装してうまく動くか調べてからということで……。
懸念としては、RTやfavをMiraclePainterレベルで扱っているため、DMクラスには本来関係ないRTやfavに関するメソッドを実装しなければいけないという点がありますが、MiraclePainterを中心としたモデルで考えると仕方ないのかな……。
Updated by Osamu Koga over 7 years ago
とりあえずDM用のRetrieverとModelを作って、MiraclePainterで表示できるようにしました。
最低限あとやること：- PostBoxを生やす
- RTボタン押して死ぬのを直す
- 右クリックで死ぬのを直す
Updated by Osamu Koga over 7 years ago
- Status changed from 実装待ち to レビュー待ち
- Assignee set to toshi_a 初音
とりあえず一通り使えるようになりました。なんか一旦remoteのブランチを消して、rebaseしたやつをもう一度pushしたら右の表示が変になってしまったけど、気にしないでください。
Updated by toshi_a 初音 over 7 years ago
遅くなりました。
mikutterのミュート機能を利用して一人でもミュートしているとクラッシュすることがわかりました（テストアカウントをミュートしておいてよかった）。修正済みです。
ミュートは邪悪だ…と言う話は置いといて、これでちょっと利用してみて、developに取り込みます。事前にちょっと話しておいたおかげでマルチサービスとも相性良さそうです
Updated by Izumi Tsutsui over 3 years ago
- Related to バグ #474: DMタブで内容を表示する際に右端が切れる added