バグ #1516
完了
機能 #887: gtk3
gtk3: worldshifter の world選択肢のアイコンが表示されない
説明
題名のとおりですが
アイコンが出ない理由がよくわかっていませんが、そもそも gtk3 では
https://docs.gtk.org/gtk3/class.ImageMenuItem.html#description
Gtk.ImageMenuItemhas been deprecated since GTK+ 3.10. If you want to display an icon in a menu item, you should useGtk.MenuItemand pack aGtk.Boxwith aGtk.Imageand aGtk.Labelinstead.
ということのようです。（が、試しに書き換えようとして挫折しました）
ファイル
Izumi Tsutsui さんが3年以上前に更新
アイコンが出ない理由がよくわかっていませんが、
かなりしょうもない話で、 Gtk::ImageMenuItem.always_show_image のデフォルトが変わった? ということのようです。
Izumi Tsutsui さんが3年以上前に更新
- ファイル mikutter_gtk3-contextmenu-icon.png mikutter_gtk3-contextmenu-icon.png を追加
- ファイル 0001-contextmenu-refs-1516.patch 0001-contextmenu-refs-1516.patch を追加
かなりしょうもない話で、
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 使われていますが、
どこの表示なのかがわかりませんでした……
Izumi Tsutsui さんが3年以上前に更新
Izumi Tsutsui さんは #note-2 で書きました:
source:plugin/gtk3/account_box.rb@0b99c704#L48 でも
Gtk::ImageMenuItem使われていますが、
どこの表示なのかがわかりませんでした……
8d1192c5 で source:core/plugin/gtk/account_box.rb@3e3c6722 → source:core/plugin/gtk/world_shifter.rb@8d1192c5 にリネームされてた、
というオチのようなので、たぶんこれは不要なファイルですね……
Izumi Tsutsui さんが3年以上前に更新
- ステータス を 分類待ち から パッチ適用待ち に変更
手元では動いていてステータス的に区別したいので先走って変更してしまいます
#note-3 の account_box.rb も削除で問題無さそうならそれでパッチ投げます。
Shibafu Midorino さんが3年以上前に更新
- ステータス を パッチ適用待ち から レビュー待ち に変更
- 担当者 を Izumi Tsutsui にセット
- ブランチ を topic/1516-show-menu-icon にセット
良さげなのでここまでのパッチを適用してpushしました。
個人的には、Boxでパッキングしろってのは実際できるだろうけど何だかなぁという感じがします。納得できない……
自分の環境ではgsettingsでアイコン出しちゃってて再現できなかったので、Manjaro LinuxのGNOMEエディションで表示を確認しました。
account_box.rbも要らならそうですね……
Izumi Tsutsui さんが3年以上前に更新
- ファイル mikutter_gtk3-worldshifter-MenuItem.png mikutter_gtk3-worldshifter-MenuItem.png を追加
- ステータス を レビュー待ち から マージ待ち に変更
なぜかブランチを git pull できないのですが
添付したパッチをそのままマージしてもらっているのであればOKと思います。
（手元ソースと差分無しなので）
Shibafu Midorino さんは #note-5 で書きました:
良さげなのでここまでのパッチを適用してpushしました。
個人的には、Boxでパッキングしろってのは実際できるだろうけど何だかなぁという感じがします。納得できない……
これ、やってみたんですが、もともとのアイコンを表示する用のスペース(?)が残るんですよね……。
設定次第で詰められるのかもしれませんが。
account_box.rbも要らならそうですね……
これはわざわざパッチなくても任せてしまっていいでしょうか。
Shibafu Midorino さんが3年以上前に更新
Izumi Tsutsui さんは #note-6 で書きました:
個人的には、Boxでパッキングしろってのは実際できるだろうけど何だかなぁという感じがします。納得できない……
これ、やってみたんですが、もともとのアイコンを表示する用のスペース(?)が残るんですよね……。
設定次第で詰められるのかもしれませんが。
うわぁ、思ったより微妙ですね！
account_box.rbも要らならそうですね……
これはわざわざパッチなくても任せてしまっていいでしょうか。
消しましたー