Project

General

Profile

Actions

バグ #1554

closed

リツイートの背景色の設定項目が重複している

Added by Izumi Tsutsui 7 months ago. Updated 7 months ago.

Status:
終了
Priority:
通常
Assignee:
Target version:
プラグイン名:
set_view
ブランチ:
topic/1554-remove-bgcolor-share-with-comment
クラッシュする:
No

Description

"UbuntuでTwitterクライアントMikutterを使う"
https://moerrari.hatenablog.com/entry/2021/12/10/113022
のエントリでの指摘。
https://moerrari.hatenablog.com/entry/2021/12/10/113022#%E8%A8%AD%E5%AE%9A

リツイートの背景色の設定項目が「背景色」→「コメント付きシェア」と「コメント付きシェア 背景色」で重複して存在しており、後から設定した方が有効になる。

確かに、指摘どおり重複しているようです。

GTK3版も同様。


Files

settings_gtk2.png (69.8 KB) settings_gtk2.png Izumi Tsutsui, 2021-12-12 03:17
settings_gtk3.png (81.9 KB) settings_gtk3.png Izumi Tsutsui, 2021-12-12 03:24
0001-remove-bgcolor-share-with-comment.patch (1.81 KB) 0001-remove-bgcolor-share-with-comment.patch Izumi Tsutsui, 2021-12-16 02:11
Actions #1

Updated by toshi_a 初音 7 months ago

  • Status changed from 分類待ち to 実装待ち

おそらく「背景色>コメント付きシェア」を削除すべきですね。
「コメント付きシェア>背景色」 のほうは、Message Modelに対して自動的に出現する設定項目だと思います。

Actions #2

Updated by Izumi Tsutsui 7 months ago

とりあえず消してみましたが
settings のラベルに set_tooltip_text できるようにしたほうがいいんですかね。

添付パッチと同じですがこれも develop に対して作ってしまいました。

diff --git a/plugin/set_view/set_view.rb b/plugin/set_view/set_view.rb
index d8b1c10c..384b7fa5 100644
--- a/plugin/set_view/set_view.rb
+++ b/plugin/set_view/set_view.rb
@@ -93,11 +93,6 @@ Plugin::create(:set_view) do
       end
     end

-    settings(_('背景色')) do
-      color(_('コメント付きシェア'), :quote_background_color).
-        tooltip(_('コメント付きシェアをすると、下に囲われて表示されるじゃないですか、あれです'))
-    end
-
     settings(_('リプライ先')) do
       fontcolor _('フォント'), :reply_text_font, :reply_text_color
       color(_('背景色'), :replyviewer_background_color)
@@ -121,7 +116,8 @@ Plugin::create(:set_view) do

     settings(_('コメント付きシェア')) do
       fontcolor _('フォント'), :quote_text_font, :quote_text_color
-      color(_('背景色'), :quote_background_color)
+      color(_('背景色'), :quote_background_color).
+        tooltip(_('コメント付きシェアをすると、下に囲われて表示されるじゃないですか、あれです'))

       multiselect _('表示項目'), :quote_present_policy do
         option(:header, _('ヘッダを表示する'))

Actions #3

Updated by toshi_a 初音 7 months ago

  • Status changed from パッチ適用待ち to レビュー待ち
  • ブランチ set to topic/1554-remove-bgcolor-share-with-comment

pushしました

Actions #4

Updated by toshi_a 初音 7 months ago

  • Assignee set to Izumi Tsutsui
Actions #5

Updated by Izumi Tsutsui 7 months ago

  • Status changed from レビュー待ち to マージ待ち
  • プラグイン名 set to set_view

master, develop相当、それぞれで git merge して
「背景色>コメント付きシェア」が無くなっているのを確認しました。
OKと思います。

Actions #6

Updated by toshi_a 初音 7 months ago

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

Also available in: Atom PDF