Project

General

Profile

バグ #1183

「mikutter について」のウインドウのみくったーさんアイコンが1回目は表示されない

Added by Izumi Tsutsui over 1 year ago. Updated over 1 year ago.

Status:
終了
Priority:
通常
Target version:
プラグイン名:
ブランチ:
topic/1183-photo-model-load-issue
クラッシュする:
No

Description

「設定」→「基本設定」→一番下の「mikutter について」のボタンを押すと
mikutter のバージョン表示のウインドウが開きますが、
1回目にバージョン表示を開いた場合にみくったーさんのアイコンが
loading のアイコンのままで表示されません。

一度ウインドウを閉じてもう一度「mikutter について」を押すと
2回目以降はアイコンが表示されます。

「ロード完了前にウインドウを表示するけれどロード完了後にアイコンを再表示する処理がない」
という感じがします。

このウインドウ表記自体を見直すという課題もありますが……


Files

mikutter-about.png (37.3 KB) mikutter-about.png Izumi Tsutsui, 2018-03-03 09:38
mikutter-1183.png (66.8 KB) mikutter-1183.png Izumi Tsutsui, 2018-03-03 12:55
mikutter-1183b.png (40.8 KB) mikutter-1183b.png Izumi Tsutsui, 2018-03-03 13:21

Associated revisions

Revision 637d75b7 (diff)
Added by toshi_a 初音 over 1 year ago

Form DSLのaboutダイアログウィジェットのロゴ画像が表示されないことがあっcた refs #1183

Revision 1fe87047 (diff)
Added by toshi_a 初音 over 1 year ago

aboutダイアログのロゴサイズを元の仕様に合わせた refs #1183

History

#1

Updated by toshi_a 初音 over 1 year ago

  • Status changed from 分類待ち to 実装待ち

多分そんな感じですね。
他にも似たような現象起こっている場所があった気がするのでもう少し奥深いところで発生しているかもしれません。

#2

Updated by toshi_a 初音 over 1 year ago

  • Status changed from 実装待ち to レビュー待ち
  • Assignee set to Izumi Tsutsui
  • ブランチ changed from master to topic/1183-photo-model-load-issue

修正しました。確認してもらって大丈夫そうなら「マージ待ち」にしといてください

#3

Updated by Izumi Tsutsui over 1 year ago

topic/1183-photo-model-load-issue で確認しました。

「みくったーさんアイコンが1回目は表示されない」という問題は解決して、
初回からみくったーさんが表示されます。

が、そのみくったーさん画像がアイコンサイズ (64x64?) ではなく、
オリジナルのサイズ (256x256) でもなく、128x128 で表示されるようです。

637d75b7 の差分で明示的に 128x128 が指定されているように見えますが
これは意図した仕様変更なのでしょうか。

#4

Updated by toshi_a 初音 over 1 year ago

  • Assignee changed from Izumi Tsutsui to toshi_a 初音

何も考えてなかったですね。確かもともとは64px四方でやってたと思うので、軽く調べてから元の仕様に戻します。

  1. いやー、4Kディスプレイだとこれくらいでもちょうどよくて辛いわー
#5

Updated by Izumi Tsutsui over 1 year ago

  • Assignee changed from toshi_a 初音 to Izumi Tsutsui

画像サイズのピクセル数が即値で埋め込まれているところが結構あって DPI対応つらそうだと思ったことはありますが
それはまた別の課題ですね……

#6

Updated by Izumi Tsutsui over 1 year ago

  • Assignee changed from Izumi Tsutsui to toshi_a 初音

firefox でチケットリロードするとステータスその他のプルダウン選択がリセットされない問題(´・ω・`)

#7

Updated by toshi_a 初音 over 1 year ago

  • Status changed from まだダメ to レビュー待ち
  • Assignee changed from toshi_a 初音 to Izumi Tsutsui

そうですねえ。
調査したところ、もともとは Gtk::WebIcon というクラスを使っていて、サイズはこれのデフォルト値である48x48を使っていました。
Gtk::WebIcon を使わなくなったのに、 Gtk::WebIcon::DEFAULT_RECTANGLE だけ参照するのもおかしいので、48pxと指定しています。

#8

Updated by Izumi Tsutsui over 1 year ago

git pull して確認しました。問題ないと思います。

#9

Updated by toshi_a 初音 over 1 year ago

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

:tada:

Also available in: Atom PDF