Project

General

Profile

バグ #1109

ツイートの詳細タブでURLリンクが多数ある場合に表示が切れてしまう

Added by Izumi Tsutsui almost 2 years ago. Updated 4 months ago.

Status:
終了
Priority:
通常
Assignee:
Target version:
プラグイン名:
message_detail_view
ブランチ:
topic/1109-message-detail-view-description
クラッシュする:
No

Description

このツイート
https://twitter.com/PkgsrcChanges/status/939584643149516800
を検索もしくは https://github.com/osak/show_tweet 等のプラグインを使って表示すると
↓こんな感じで複数のリンクが貼られているのですが

これを右クリックメニューの「詳細」で表示すると

のように URL が途中で切れてしまうようです。

文字数制限もしくは extended_tweet の扱いに問題があるんでしょうか。


Files

tweet1.png (18 KB) tweet1.png Izumi Tsutsui, 2017-12-10 06:12
tweet2.png (19.9 KB) tweet2.png Izumi Tsutsui, 2017-12-10 06:12
0001-message_detail_view-description.patch (1.29 KB) 0001-message_detail_view-description.patch Shibafu Midorino, 2019-05-01 19:40

Related issues

Related to バグ #1358: 抽出条件「本文」がMessage Modelのto_sを使用している終了Actions

Associated revisions

Revision 8878133f (diff)
Added by Shibafu Midorino 4 months ago

message_detail_view: descriptionを本文として参照する refs #1109

History

#1

Updated by Shibafu Midorino 6 months ago

該当の機能を提供するプラグインである message_detail_view の実装を見ると、本文の表示のために Model の #to_s を参照していました。

Plugin::Twitter::Message#to_s は (おそらく互換性維持のため?) 本文を、140文字で収まるよう末尾を捨てて返す作りになっています。
そのため、URLの途中で切り捨てられた本文が表示されていたと考えられます。

今日では、本文情報のようなデータは description フィールドで扱うという規約が存在するため、ひとまずはそちらを参照するのが妥当だと考えます。

mikutter 3.8.7向けのパッチを添付しました。

#2

Updated by toshi_a 初音 4 months ago

  • Status changed from パッチ適用待ち to レビュー待ち
  • Assignee set to Izumi Tsutsui
  • Target version changed from 3.5 to 3.8
  • ブランチ set to topic/1109-message-detail-view-description

pushしました。確認してみてください。

#3

Updated by Izumi Tsutsui 4 months ago

  • Status changed from レビュー待ち to マージ待ち

master だとテストがややしんどいので develop に同様のパッチを当ててテストしました。
問題ありません。

#4

Updated by toshi_a 初音 4 months ago

  • Status changed from マージ待ち to 終了

merged.

#5

Updated by Izumi Tsutsui 4 months ago

  • Related to バグ #1358: 抽出条件「本文」がMessage Modelのto_sを使用している added

Also available in: Atom PDF