プロジェクト

全般

プロフィール

バグ #1554

完了

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

Izumi Tsutsui さんが約3年前に追加. 約3年前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
プラグイン名:
set_view
クラッシュする:
いいえ

説明

"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版も同様。


ファイル

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

toshi_a 初音 さんが約3年前に更新

  • ステータス分類待ち から 実装待ち に変更

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

Izumi Tsutsui さんが約3年前に更新

とりあえず消してみましたが
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, _('ヘッダを表示する'))

toshi_a 初音 さんが約3年前に更新

  • ステータスパッチ適用待ち から レビュー待ち に変更
  • ブランチtopic/1554-remove-bgcolor-share-with-comment にセット

pushしました

toshi_a 初音 さんが約3年前に更新

  • 担当者Izumi Tsutsui にセット

Izumi Tsutsui さんが約3年前に更新

  • ステータスレビュー待ち から マージ待ち に変更
  • プラグイン名set_view にセット

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

toshi_a 初音 さんが約3年前に更新

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

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