Project

General

Profile

Actions

提案 #1323

closed

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

Added by Izumi Tsutsui over 5 years ago. Updated over 5 years ago.

Status:
終了
Priority:
通常
Assignee:
Target version:
プラグイン名:
twitter

Description

解決しようとする課題

現状の 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が出ない

Files

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
Actions #1

Updated by cob odo over 5 years ago

-      }.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] }
Actions #2

Updated by Izumi Tsutsui over 5 years ago

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

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

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

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

Actions #3

Updated by toshi_a 初音 over 5 years ago

  • Status changed from 分類待ち to パッチ適用待ち
Actions #4

Updated by toshi_a 初音 over 5 years ago

  • Status changed from パッチ適用待ち to レビュー待ち
  • Assignee set to Izumi Tsutsui
  • Target version set to 3.9
  • ブランチ set to topic/1323-insert-newline-to-each-tweet-attachment-url

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

Actions #5

Updated by Izumi Tsutsui over 5 years ago

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

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

Actions #6

Updated by toshi_a 初音 over 5 years ago

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

merged

Actions

Also available in: Atom PDF