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