Project

General

Profile

提案 #1258

抽出タブのリストビューの改善案

Added by Yuto Tokunaga over 2 years ago. Updated 5 months ago.

Status:
終了
Priority:
通常
Target version:
プラグイン名:
extract
ブランチ:
topic/1258-pretty-extract-settings-for-4.1

Description

現状では,設定画面の抽出タブのリストビューは抽出タブの名前の列のみで,情報が少ないと感じます.各行がどのタブに対応するか,また抽出タブに通知音やポップアップ通知が設定されているかをアイコンによって識別できるようにする機能提案です.

TODO 通知音・ポップアップ通知の設定状況を識別するためのアイコンを標準スキンに入れる必要があります.

Before After

Files

1258-1.patch (5.96 KB) 1258-1.patch Yuto Tokunaga, 2018-06-02 01:41
1258-2.patch (2.67 KB) 1258-2.patch topic/1258-pretty-extract-settings(ccda3f8)に対するパッチ Yuto Tokunaga, 2018-09-19 00:46
Screenshot_20200531_022711.png (94.9 KB) Screenshot_20200531_022711.png Shibafu Midorino, 2020-05-31 02:28

Related issues

Related to バグ #1441: 抽出タブのリストビューに設定変更が反映されない終了Shibafu MidorinoActions
#1

Updated by toshi_a 初音 over 2 years ago

  • Status changed from 新規 to 実装待ち
  • Assignee set to Yuto Tokunaga

これめっちゃいいですね!採用したいと思います。 topic/1258-pretty-extract-settings にコミットしました。

マージするにあたって、ITER_系の定数がCOL_に変わっていたためか大量にコンフリクトしたため、こっちで適当に直しました。この変更自体には特に反対ではないのですが、適当に修正してしまったので一応確認をお願いします。

また、画像がパッチに含まれていないので、このチケットに添付してもらえますか。
そのさい、その画像はクリエイティブ・コモンズ 表示 - 継承 3.0 非移植ライセンス(http://creativecommons.org/licenses/by-sa/3.0/deed.ja)のもと再配布することを同意してもらう必要があります。
もしその画像が別のところから持ってきた素材なら、俺の方で作ります。

#2

Updated by Yuto Tokunaga over 2 years ago

topic/1258-pretty-extract-settings を確認しました.OKです.

添付画像のスキンは私とは別の方が作られたものなので( https://ray34g.deviantart.com/art/mikutter-icon-theme-miq-2-505894046 のcolorfulというテーマです),申し訳無いのですがそちらで作っていただけますか.

#3

Updated by Yuto Tokunaga about 2 years ago

スキンのアイコンを使うのではなくgtkテーマのアイコンを使うようにしてみました.

#4

Updated by toshi_a 初音 about 2 years ago

  • Tracker changed from 機能 to 提案
  • Status changed from 実装待ち to レビュー待ち
  • Start date deleted (2018-06-02)
  • ブランチ set to topic/1258-pretty-extract-settings

pushしました。レビューお願いします。

#5

Updated by Yuto Tokunaga about 2 years ago

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

OKです.

#6

Updated by cob odo about 2 years ago

  • Status changed from マージ待ち to まだダメ
  • Assignee changed from Yuto Tokunaga to toshi_a 初音

mergeしようとしたらconflictしたので差し戻します。

#7

Updated by toshi_a 初音 about 2 years ago

audio-volume-muted-symbolic などはうちの環境だと存在しないのと、他のところでは画像はいちいち作っているので、なんか書いてみますかねぇ

#8

Updated by Shibafu Midorino 5 months ago

  • File Screenshot_20200531_022711.png Screenshot_20200531_022711.png added
  • Status changed from まだダメ to レビュー待ち
  • Assignee changed from toshi_a 初音 to Yuto Tokunaga
  • Target version changed from 3.9 to 4.1
  • ブランチ changed from topic/1258-pretty-extract-settings to topic/1258-pretty-extract-settings-for-4.1

mikutter 4.1のdevelopブランチ向けに差分の当て直しを行い、アイコンを私が作成したもので置き換えました。採用いただけるようであれば、アイコンの権利については他のスキンファイルと同等でOKです。

差分の当て直しにあたって多少こちらでコードの手直しもしているので、改めてレビューをお願いします。

#9

Updated by toshi_a 初音 5 months ago

  • Assignee changed from Yuto Tokunaga to Shibafu Midorino

マージしてOKです。ただ、UIスケールが反映されていなかったので、そこだけ対応しました。一応見てもらおうかな

#10

Updated by Yuto Tokunaga 5 months ago

アイコンの濃度の差のおかげでとても分かりやすい表示になっていてとてもえらいっ!です.UIスケールが反映されていることも確認できました.

編集ダイアログで設定を変更してもリストビューに反映されない問題がありますが、これは別のチケットにしたほうが良さそうですね.

#11

Updated by Yuto Tokunaga 5 months ago

  • Related to バグ #1441: 抽出タブのリストビューに設定変更が反映されない added
#12

Updated by Shibafu Midorino 5 months ago

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

こちらでもUIスケールの反映を確認できましたので、マージしました。

Also available in: Atom PDF