バグ #1183
closed「mikutter について」のウインドウのみくったーさんアイコンが1回目は表示されない
Description
「設定」→「基本設定」→一番下の「mikutter について」のボタンを押すと
mikutter のバージョン表示のウインドウが開きますが、
1回目にバージョン表示を開いた場合にみくったーさんのアイコンが
loading のアイコンのままで表示されません。
一度ウインドウを閉じてもう一度「mikutter について」を押すと
2回目以降はアイコンが表示されます。
「ロード完了前にウインドウを表示するけれどロード完了後にアイコンを再表示する処理がない」
という感じがします。
このウインドウ表記自体を見直すという課題もありますが……
Files
Updated by toshi_a 初音 almost 7 years ago
- Status changed from 分類待ち to 実装待ち
多分そんな感じですね。
他にも似たような現象起こっている場所があった気がするのでもう少し奥深いところで発生しているかもしれません。
Updated by toshi_a 初音 almost 7 years ago
- Status changed from 実装待ち to レビュー待ち
- Assignee set to Izumi Tsutsui
- ブランチ changed from master to topic/1183-photo-model-load-issue
修正しました。確認してもらって大丈夫そうなら「マージ待ち」にしといてください
Updated by Izumi Tsutsui almost 7 years ago
- File mikutter-1183.png mikutter-1183.png added
- Status changed from レビュー待ち to まだダメ
topic/1183-photo-model-load-issue で確認しました。
「みくったーさんアイコンが1回目は表示されない」という問題は解決して、
初回からみくったーさんが表示されます。
が、そのみくったーさん画像がアイコンサイズ (64x64?) ではなく、
オリジナルのサイズ (256x256) でもなく、128x128 で表示されるようです。
637d75b7 の差分で明示的に 128x128 が指定されているように見えますが
これは意図した仕様変更なのでしょうか。
Updated by toshi_a 初音 almost 7 years ago
- Assignee changed from Izumi Tsutsui to toshi_a 初音
何も考えてなかったですね。確かもともとは64px四方でやってたと思うので、軽く調べてから元の仕様に戻します。
- いやー、4Kディスプレイだとこれくらいでもちょうどよくて辛いわー
Updated by Izumi Tsutsui almost 7 years ago
- Assignee changed from toshi_a 初音 to Izumi Tsutsui
画像サイズのピクセル数が即値で埋め込まれているところが結構あって DPI対応つらそうだと思ったことはありますが
それはまた別の課題ですね……
Updated by Izumi Tsutsui almost 7 years ago
- Assignee changed from Izumi Tsutsui to toshi_a 初音
firefox でチケットリロードするとステータスその他のプルダウン選択がリセットされない問題(´・ω・`)
Updated by toshi_a 初音 almost 7 years ago
- Status changed from まだダメ to レビュー待ち
- Assignee changed from toshi_a 初音 to Izumi Tsutsui
そうですねえ。
調査したところ、もともとは Gtk::WebIcon
というクラスを使っていて、サイズはこれのデフォルト値である48x48を使っていました。Gtk::WebIcon
を使わなくなったのに、 Gtk::WebIcon::DEFAULT_RECTANGLE
だけ参照するのもおかしいので、48pxと指定しています。
Updated by Izumi Tsutsui almost 7 years ago
- File mikutter-1183b.png mikutter-1183b.png added
- Status changed from レビュー待ち to マージ待ち
- Assignee changed from Izumi Tsutsui to toshi_a 初音
git pull して確認しました。問題ないと思います。