Project

General

Profile

機能 #806

SubPartsQuoteのナウいメッセージ表示を、サードパーティのプラグインからも使いたい

Added by Satoshi Okuno over 3 years ago. Updated over 1 year ago.

Status:
終了
Priority:
通常
Target version:
Start date:
2016-01-03
Due date:
% Done:

0%

プラグイン名:

Description

あけましておめでとうございます。

mikutter3.3.0から所謂「ナウい引用」がシステムプラグインに昇格しましたが、
このナウいメッセージ表示をサードパーティのプラグインからも使えるようにしたいです。

なんでこんなことを言っているかと言うと、

 ・従来型のリプライをナウく表示するプラグイン(mikutter-trendy-reply)を書いたが、ほぼSubPartsQuoteのコピペになったのでソースを共有したい。
 ・TLを見ていると、私以外にもニーズがありそう。
  (この前、「翻訳プラグインを作って翻訳結果をサブパーツに表示したいけど、作り方がようわからん」と言うつぶやきを見ました。)

具体的には、SubPartQuoteのサブパーツ描画部分をSubPartsMessageBaseと言うクラスに切り出しました。
そしてSubPartsQuote(とかサードパーティのプラグイン)はSubPartsMessageBaseを継承して、必要なメソッドを実装すれば良い様にしました。

3.3.1に対するパッチを作りましたので、お時間あるときにご検討ください。


Files


Related issues

Related to 機能 #809: in_reply_to_statusの表示を、quoted tweetのようないい感じの見た目にする終了2016-01-11

Actions

Associated revisions

Revision b70ccf9a (diff)
Added by toshi_a 初音 over 3 years ago

Gdk::SubPartsMessageBaseに表示するテキストを3行で切る refs #806

Revision b27e2585 (diff)
Added by toshi_a 初音 over 3 years ago

Gdk::SubPartsMessageBaseのヘッダ部分を削除する手段 refs #806

Revision d371c0a0 (diff)
Added by toshi_a 初音 over 3 years ago

replyviewer: リプライ先表示からヘッダを削除できる refs #806

History

#1

Updated by toshi_a 初音 over 3 years ago

いいですね。実はリプライも同じような形にしたいと考えていて、その過程でこういうことは必要だと思ってました。

ただ、現状でそれをやるとリプライ、コメント付きリツイート、サードパーティで追加されたものが無秩序に並んでしまうので、見分けが着くようにする方法を考えているところです。

#2

Updated by Satoshi Okuno over 3 years ago

大筋賛同ありがとうございます。

見分けがつく方法として、例えば枠の色を変えると言うのはどうでしょうか。
色を返す抽象メソッドを用意するか、filter_message_quoted_message_colorのようなフィルタにする。

#3

Updated by toshi_a 初音 over 3 years ago

ああ、リプライを示すアイコンでも表示しようかと思ってましたが、色も変えてみようかな。
これ3.3からやったほうが良いですかねぇ。こういった変更は3.4からにしようかと思うんですが

#4

Updated by Satoshi Okuno over 3 years ago

急ぐ話でも無いですし、3.4以降ででOKかと。
手伝えることあればぜひ教えてください。

#5

Updated by toshi_a 初音 over 3 years ago

  • Related to 機能 #809: in_reply_to_statusの表示を、quoted tweetのようないい感じの見た目にする added
#6

Updated by toshi_a 初音 over 3 years ago

#809 のほうでこんな感じの機能追加をしたんですが(origin/topic/809-better-surface-of-in-reply-to-status)、なんかサードパーティで実装したいことに関してmikutter側でやってほしいことってあります?

#7

Updated by Satoshi Okuno over 3 years ago

返信遅れました。

今日まで常用してきて、問題なく動作しています。
追加での要望はありません。

Redmineを触っていて。例えば「このサイトのIssueへのリンクがあれば件名をナウいメッセージで表示する」とかにも使えそうです。

#8

Updated by toshi_a 初音 over 3 years ago

  • Status changed from 新規 to レビュー待ち
  • Assignee changed from toshi_a 初音 to Satoshi Okuno
  • Target version changed from 3.3 to 3.4

全て実装終わりました。あの後からは、枠のスタイル、フォントの指定などができるようになりました。詳しくはCairo::SubPartsMessageBaseをご覧ください。

#9

Updated by Satoshi Okuno over 3 years ago

ごめんな。気づいてあげられなくて。
(すんません。今気づきました。)

週末見させていただきたく。
また投稿します。

#10

Updated by Satoshi Okuno over 3 years ago

またまた遅くなり失礼しました。

Gdk::SubPartsQuoteのメソッドのうち、全サブクラスで有益と思われるコードをSubPartsMessageに移植してみました。
ご意見いただければと思います。

・show_header?()、show_icon?()、show_edge?()
 現状のとあるメソッドでnilを返すのに変えて、直感的な名前のメソッドをオーバーライドするのはどうでしょうか。

・render_outline()、render_badge()
 SubPartsQuote、ReplyViewerでそれぞれ持っている枠タイプを選ぶ処理を汎用化して、edge_type()と言うメソッドで枠タイプを切り替えるようにしてみました。

#11

Updated by toshi_a 初音 over 3 years ago

  • Assignee changed from Satoshi Okuno to toshi_a 初音

ごめんな。気づいてあげられなくて。
(すんません。今気づきました。)
休日以降に見ます

#12

Updated by toshi_a 初音 over 1 year ago

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

Also available in: Atom PDF