nims-dpfc / rdetoolkit

MIT License
2 stars 0 forks source link

サムネイルのサイズ #2

Closed sonoh5n closed 6 days ago

sonoh5n commented 3 months ago

rdetoolkit==0.2.0 から、サムネイルをRDEシステムのv5バージョンアップに合わせてメインイメージをサムネイルフォルダにコピーするように変更になっているようなのですが、 v5ではサムネイルフォルダの画像が一覧画面のみでの表示となったという認識です。

サイズについて一覧画面の画像サイズに合わせてもよいかと思いました。

将来の拡張に備えて、多少余裕を持たせて少し大きめでもよいかもしれませんが、フルサイズは必要ないかも、と思いました。

ご検討ください。

sonoh5n commented 3 months ago

NAGAO Hiroko commented:

@sonokawa-hayato さま

現状、一覧画面の画像サイズは250px*250pxです。

ただ、この画面でも、画像はできるだけ大きく表示したい、という要望もあり、今後もこのままなのか、統合UIなども関係してくると、一概に上記サイズがいいとはいえません。

このissueを立てた意図は、代表画像に非常に大きい(例えば2048px以上など)の画像が登録された際でも、現状ではサムネイルのサイズも同じサイズになってしまうため、表示に負荷がかかることを懸念したためです。

藤間さんともお話しましたが、仕様を決める時点で大きい画像が想定される場合は、上記のことも考慮し、構造化側で適切な大きさに自分でリサイズしてサムネイルフォルダに書き込む、がよいのでは、という意見をいただきました。

その時の指針(=推奨画像サイズ)(例えば縦横で最大を300px程度にしておく、など?)を出しておく、というのが資材提供側の役割になるのかと思いました。 

なお、補足ですが、現在の代表画像の推奨サイズは figsizeにある通り、640px*480px です。

sonoh5n commented 3 months ago

SONOKAWA Hayato commented:

@nagao-hiroko

issueありがとうございます。一覧画面の画像の推奨画像サイズなど仕様があれば実装にしたいと思います。そのため、RDE側で定める推奨画像サイズについて教えていただきたいです。もし一覧画像の推奨サイズが明確に定義されていない場合、江波さんのご指摘のとおり、Configにてコピーを抑止し、自分でリサイズしてサムネイルフォルダに書き込む。がいいと思いました。

sonoh5n commented 3 months ago

ENAMI Hitoshi commented:

サイズの変更は、画像をゆがませたりいろいろ問題が発生する可能性があるので、そのままのコピーでいいんじゃないでしょうか?

それが嫌なら、Configにてコピーを抑止し、自分でリサイズしてサムネイルフォルダに書き込む。その方がシンプルじゃないですか?

とはいえ、長尾さんのいうように適切なサイズにリサイズしてからコピーする処理がデフォでもいいとは思います。で、だめだったら、抑止して自分で作ると。