Project

General

Profile

Actions

バグ #966

closed

settingsのaboutのnameが使われていない

Added by あひる 家鴨 over 7 years ago. Updated over 7 years ago.

Status:
終了
Priority:
通常
Target version:
プラグイン名:
core/plugin/settings/basic_settings.rb
ブランチ:
クラッシュする:

Description

settingsabout のインスタンス生成時のオプションに :name を指定していますが、この値は使われておらずアプリケーション名( "mikutter.rb" )が常に渡されてしまっています。( before.png を参照してください )
正しくは、aboutdialog.rb#L35 のように program_name を指定する必要があるようです。
program_name を指定した結果 after.png のように ENVIRONMENT::NAME で指定している "mikutter" が使用されているのを確認しました。
また、 :name を指定してアプリケーション名が表示されていることに関しては、 Gtk::AboutDialog の仕様のようです。(AboutDialog#program_name

修正パッチを添付します。( about.patch


Files

after.png (77.2 KB) after.png 修正後 あひる 家鴨, 2017-01-09 16:22
before.png (80.3 KB) before.png 修正前 あひる 家鴨, 2017-01-09 16:22
about.patch (1.65 KB) about.patch あひる 家鴨, 2017-01-09 16:29
Actions #1

Updated by あひる 家鴨 over 7 years ago

  • Description updated (diff)
Actions #2

Updated by あひる 家鴨 over 7 years ago

  • Status changed from 新規 to パッチ適用待ち
Actions #3

Updated by toshi_a 初音 over 7 years ago

  • Status changed from パッチ適用待ち to レビュー待ち
  • Assignee set to あひる 家鴨

このパッチはaboutの仕様を変更してますが、それだと今までとは違う形式でパラメータを渡さなければならなくなるため、非互換な変更となってしまいます。
name属性に与えられた値をアプリケーション名として利用するようにしましたが、どうでしょう。

Actions #4

Updated by あひる 家鴨 over 7 years ago

  • Status changed from レビュー待ち to 解決

確認しました
いいと思います。
確かに互換性を破壊してしまう変更でした。

Actions #5

Updated by あひる 家鴨 over 7 years ago

  • Assignee changed from あひる 家鴨 to toshi_a 初音
Actions #6

Updated by toshi_a 初音 over 7 years ago

  • Status changed from 解決 to 終了
Actions

Also available in: Atom PDF