プロジェクト

全般

プロフィール

提案 #1323

完了

ツイートの添付画像のURLを1枚ずつ改行して表示する

Izumi Tsutsui さんが5年以上前に追加. 5年以上前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
プラグイン名:
twitter

説明

解決しようとする課題

現状の mikutter の twitter プラグインのツイート添付画像の URL表示は
複数枚画像が添付されている場合も各画像のURL がスペース区切りで一行で表示されますが、
全部で何枚添付されているのかがわかりにくい、どのURLが何枚目かがわかりづらい、
という問題があると思っています。

パッチ

Worldon の場合は各画像の URLを 1枚1行で表示しているようなので、
twitter プラグインでも同じ表示ができないか、と思っていろいろ調べて
適当に書いてみました。
0001-Tweet-URL.patch

表示例

オリジナル (mikutter 3.8.6)

パッチを当てた後

問題としては、本文が1文字もなくて画像のみが添付されているツイートでも
画像URLの前に改行が入ってしまうところでしょうか。

mikutter-uwm-hommage 等のサードパーティープラグインへの影響は把握できていません。

蛇足

パッチの最後を
+      }.flat_map{|media| [newline, media, space] }

としてますが、最後の space を入れずに
+      }.flat_map{|media| [newline, media] }

とするとなぜか以下の状態になります。そういうもんなんでしょうか。
  • 添付画像が1枚の場合 t.co が展開されない
  • 添付画像が2枚以上の場合最後の画像のURLが出ない

ファイル

0001-Tweet-URL.patch (1.51 KB) 0001-Tweet-URL.patch Izumi Tsutsui, 2019-03-04 23:53
tsutsuii-1102148732017991682.png (15.2 KB) tsutsuii-1102148732017991682.png Izumi Tsutsui, 2019-03-05 00:00
tsutsuii-1068904358748680195.png (36.8 KB) tsutsuii-1068904358748680195.png Izumi Tsutsui, 2019-03-05 00:00
tsutsuii-1102148732017991682-1.png (15.4 KB) tsutsuii-1102148732017991682-1.png Izumi Tsutsui, 2019-03-05 00:01
tsutsuii-1068904358748680195-1.png (37.7 KB) tsutsuii-1068904358748680195-1.png Izumi Tsutsui, 2019-03-05 00:01
tsutsuii-1102148732017991682-2.png (13.8 KB) tsutsuii-1102148732017991682-2.png Izumi Tsutsui, 2019-03-05 00:05
tsutsuii-1068904358748680195-2.png (37.1 KB) tsutsuii-1068904358748680195-2.png Izumi Tsutsui, 2019-03-05 00:05
kurosirogurei-1098966023137624065.png (17.7 KB) kurosirogurei-1098966023137624065.png Izumi Tsutsui, 2019-03-05 00:10
0001-Tweet-URL-2.patch (1.55 KB) 0001-Tweet-URL-2.patch Izumi Tsutsui, 2019-03-05 23:11

cob odo さんが5年以上前に更新

-      }.flat_map{|media| [media, space] }
+      }.flat_map{|media| [newline, media, space] }
       result.pop

このコードのresult.popは、最後に付く余分なspaceを削除するためにあります。

-      }.flat_map{|media| [media, space] }
+      }.flat_map{|media| [newline, media] }
       result.pop

とする場合、直後のresult.popがあるままだと、最後のmediaが消えてしまいます。本文と添付画像リンクの間に改行が必要ない場合はresult.popresult.shiftに変更、改行が必要な場合はresult.popの行自体を消すのがいいんじゃないでしょうか。

個人的には後者が好みです。その場合、こうなると思います。

-      }.flat_map{|media| [media, space] }
-      result.pop
+      }.flat_map{|media| [newline, media] }

Izumi Tsutsui さんが5年以上前に更新

このコードのresult.popは、最後に付く余分なspaceを削除するためにあります。

解説ありがとうございます。納得しました。

個人的には後者が好みです。その場合、こうなると思います。

本文との間に改行があったほうがよいけれど、本文がからのときは改行はないほうがいいなあ、
などと思ったりしますが、そこまでこだわるほどでもないので後者でパッチを更新しました。
0001-Tweet-URL-2.patch

toshi_a 初音 さんが5年以上前に更新

  • ステータス分類待ち から パッチ適用待ち に変更

toshi_a 初音 さんが5年以上前に更新

  • ステータスパッチ適用待ち から レビュー待ち に変更
  • 担当者Izumi Tsutsui にセット
  • 対象バージョン3.9 にセット
  • ブランチtopic/1323-insert-newline-to-each-tweet-attachment-url にセット

良さそうですね。何も分からんけど 0001-Tweet-URL-2.patch をcommitしてみました。

Izumi Tsutsui さんが5年以上前に更新

  • ステータスレビュー待ち から マージ待ち に変更

topic/1323-insert-newline-to-each-tweet-attachment-url の
ブランチ元が 3.8.5 なのはなぜだろう、
と思いながら確認しました。問題ありません。

toshi_a 初音 さんが5年以上前に更新

  • ステータスマージ待ち から 終了 に変更

merged

他の形式にエクスポート: Atom PDF