Project

General

Profile

Actions

バグ #1548

closed

gtk3: Gtk::ImageMenuItem.new の引数が適切でない

Added by Izumi Tsutsui 6 months ago. Updated 5 months ago.

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

Description

題名のとおりですが
source:plugin/gtk3/widget/worldshifter.rb@f6025344 にある
ImageMenuItem.new の引数が意図通りになっていないようです。

Ruby-Gnome のマニュアル によれば
Gtk::ImageMenuItem.new(label = nil, use_underline = true)
もしくは
Gtk::ImageMenuItem.new(stock_id, accel_group = nil)
という引数ですが、現状では
Gtk::ImageMenuItem.new(label: world.title, accel_group: false)
および
Gtk::ImageMenuItem.new(label: Plugin[:gtk3]._('Worldを追加'), accel_group: false)
と両者が混ざった記述になっています。(これが ruby 的にどう解釈されるのかよくわかってませんが)

なお https://noanoa07.livedoor.blog/archives/2136674.html によれば
use_underline は機能していない(?)らしいので実害はないようです。


Files

0001-Gtk-ImageMenuItem.new.patch (1.73 KB) 0001-Gtk-ImageMenuItem.new.patch Izumi Tsutsui, 2021-11-29 21:15
Actions #1

Updated by Izumi Tsutsui 6 months ago

とりあえず修正パッチです。

source:core/mui/gtk_contextmenu.rb@f6025344 にも
Gtk::ImageMenuItem.new があってこちらは 2つ目の引数がない(≒デフォルトで true になっている?)のですが、
チケット本文記載の通り use_underline は機能していないので見た目としても問題になっていないようです。

Actions #2

Updated by Shibafu Midorino 6 months ago

https://github.com/ruby-gnome/ruby-gnome/blob/3.4.9/gtk3/lib/gtk3/image-menu-item.rbhttps://valadoc.org/gtk+-3.0/Gtk.ImageMenuItem.html を見る限り、label: を渡す場合は他のキーワード引数には一切意味が無いようなのでこれ1つで良さそうです。

Actions #3

Updated by Shibafu Midorino 6 months ago

いや意味がないことはないか、new_with_mnemonic を使うかどうかの分岐になってますね。ろくに読めていませんでした。

Actions #4

Updated by Izumi Tsutsui 6 months ago

new_with_mnemonic ってどこにあるんやと思ったら
https://valadoc.org/gtk+-3.0/Gtk.Label.Label.with_mnemonic.html とかなんですかね。

If characters in str are preceded by an underscore, they are underlined.

というのも ruby-gnome のドキュメント説明と合ってませんが、
アンダースコアつけても結局アンダーラインにならないような……。

Actions #5

Updated by toshi_a 初音 6 months ago

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

確認お願いします

Actions #6

Updated by Izumi Tsutsui 6 months ago

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

git merge topic/1548-gtk-menu-item して起動して
worldshifter 動作は特に問題なしです。

Actions #7

Updated by toshi_a 初音 5 months ago

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

merged

Actions

Also available in: Atom PDF