Project

General

Profile

バグ #986

抽出タブの条件設定画面で、条件を削除するボタンのアイコンが表示されない。

Added by Satoshi Okuno over 3 years ago. Updated over 3 years ago.

Status:
終了
Priority:
通常
Assignee:
Target version:
プラグイン名:
extract
ブランチ:
クラッシュする:

Description

まぁ、僕しいたけ食べられないから別にいいんですけどね。

現状WebIconにSkin["close.png"]が渡されていますが、 load errorアイコンも出ずのっぺらぼうのボタンになります。

/Users/moguno/mikutter//core/mui/gtk_postbox.rbを参考に、WebIconにSkin["close.png"].get_pathを渡すようにしました。
でもWebIconって、モデルが渡されたらload_pixbufするはずだなぁ。とか思っています。


Files

0001-.patch (1.17 KB) 0001-.patch Satoshi Okuno, 2017-02-25 10:16
#1

Updated by Satoshi Okuno over 3 years ago

X:現状WebIconにSkin["close.png"]が渡されていますが、
○:現状Skin["close.png"] .pixbuf()が渡されていますが、

#2

Updated by toshi_a 初音 over 3 years ago

  • Status changed from 新規 to 実装待ち
  • Assignee set to toshi_a 初音

確かにボタン表示されてませんね。頂いたパッチ試してみます。

#3

Updated by toshi_a 初音 over 3 years ago

  • Status changed from 実装待ち to レビュー待ち
  • Assignee changed from toshi_a 初音 to Satoshi Okuno
  • % Done changed from 0 to 100

Gtk::Button#add は、Photo Modelを受け付けないので、Pixbufを表示したい場合はGtk::Imageを経由しないといけないということですね。3.5では Gtk::WebIcon は第一引数にファイル名やURLだけでなくPhotoModelも受け付けるようになり、PhotoModelを与えるのが一番高速に動作します。よって、SkinはPhoto Modelの一種なので、これを渡すようにすこし改変しました。

topic/986-forever-shiitake ブランチです。意図通りになっているか確認してもらえますか。

#4

Updated by Satoshi Okuno over 3 years ago

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

対応ありがとうございます。
ばっちりしいたけが生えていました。

#5

Updated by toshi_a 初音 over 3 years ago

  • Status changed from 解決 to 終了

merged

Also available in: Atom PDF