Project

General

Profile

Actions

バグ #1516

closed

機能 #887: gtk3

gtk3: worldshifter の world選択肢のアイコンが表示されない

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

Status:
終了
Priority:
通常
Assignee:
Target version:
プラグイン名:
gtk3
ブランチ:
topic/1516-show-menu-icon
クラッシュする:
No

Description

題名のとおりですが

アイコンが出ない理由がよくわかっていませんが、そもそも gtk3 では
https://docs.gtk.org/gtk3/class.ImageMenuItem.html#description

Gtk.ImageMenuItem has been deprecated since GTK+ 3.10. If you want to display an icon in a menu item, you should use Gtk.MenuItem and pack a Gtk.Box with a Gtk.Image and a Gtk.Label instead.

ということのようです。(が、試しに書き換えようとして挫折しました)


Files

Actions #1

Updated by Izumi Tsutsui 11 months ago

アイコンが出ない理由がよくわかっていませんが、

かなりしょうもない話で、 Gtk::ImageMenuItem.always_show_image のデフォルトが変わった? ということのようです。

Actions #2

Updated by Izumi Tsutsui 11 months ago

かなりしょうもない話で、 Gtk::ImageMenuItem.always_show_image のデフォルトが変わった? ということのようです。

同様に Gtk::ImageMenuItem を使っている source:core/mui/gtk_contextmenu.rb@0b99c704#L81 も同様に修正すると
コンテキストメニューのアイコンが出るようになりました。

source:plugin/gtk3/account_box.rb@0b99c704#L48 でも Gtk::ImageMenuItem 使われていますが、
どこの表示なのかがわかりませんでした……

Actions #3

Updated by Izumi Tsutsui 11 months ago

Izumi Tsutsui さんは #note-2 で書きました:

source:plugin/gtk3/account_box.rb@0b99c704#L48 でも Gtk::ImageMenuItem 使われていますが、
どこの表示なのかがわかりませんでした……

8d1192c5source:core/plugin/gtk/account_box.rb@3e3c6722source:core/plugin/gtk/world_shifter.rb@8d1192c5 にリネームされてた、
というオチのようなので、たぶんこれは不要なファイルですね……

Actions #4

Updated by Izumi Tsutsui 11 months ago

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

手元では動いていてステータス的に区別したいので先走って変更してしまいます

#note-3 の account_box.rb も削除で問題無さそうならそれでパッチ投げます。

Actions #5

Updated by Shibafu Midorino 11 months ago

  • Status changed from パッチ適用待ち to レビュー待ち
  • Assignee set to Izumi Tsutsui
  • ブランチ set to topic/1516-show-menu-icon

良さげなのでここまでのパッチを適用してpushしました。
個人的には、Boxでパッキングしろってのは実際できるだろうけど何だかなぁという感じがします。納得できない……

自分の環境ではgsettingsでアイコン出しちゃってて再現できなかったので、Manjaro LinuxのGNOMEエディションで表示を確認しました。

account_box.rbも要らならそうですね……

Actions #6

Updated by Izumi Tsutsui 11 months ago

なぜかブランチを git pull できないのですが
添付したパッチをそのままマージしてもらっているのであればOKと思います。
(手元ソースと差分無しなので)

Shibafu Midorino さんは #note-5 で書きました:

良さげなのでここまでのパッチを適用してpushしました。
個人的には、Boxでパッキングしろってのは実際できるだろうけど何だかなぁという感じがします。納得できない……

これ、やってみたんですが、もともとのアイコンを表示する用のスペース(?)が残るんですよね……。
設定次第で詰められるのかもしれませんが。

account_box.rbも要らならそうですね……

これはわざわざパッチなくても任せてしまっていいでしょうか。

Actions #7

Updated by Shibafu Midorino 11 months ago

Izumi Tsutsui さんは #note-6 で書きました:

個人的には、Boxでパッキングしろってのは実際できるだろうけど何だかなぁという感じがします。納得できない……

これ、やってみたんですが、もともとのアイコンを表示する用のスペース(?)が残るんですよね……。
設定次第で詰められるのかもしれませんが。

うわぁ、思ったより微妙ですね!

account_box.rbも要らならそうですね……

これはわざわざパッチなくても任せてしまっていいでしょうか。

消しましたー

Actions #8

Updated by Shibafu Midorino 11 months ago

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

merged

Actions

Also available in: Atom PDF