codeforjapan / decidim-cfj

Code for Japan Decidim
GNU Affero General Public License v3.0
54 stars 36 forks source link

バナー画像の画像の貼り付けサイズの表記がおかしい #59

Closed isaozz closed 2 years ago

isaozz commented 3 years ago

対象ドメイン/ The domain

起こっている問題 / The Problem

スクリーンショット / Screenshot

スクリーンショット 2020-10-25 22 30 01

期待する見せ方・挙動 / Expected Behavior

起こっている問題の再現手段 / Steps to Reproduce

  1. xxx
  2. xxx
  3. xxx

動作環境・ブラウザ / Environment

mcyph commented 3 years ago

そのようですね、uploaders/decidim/image_uploader.rbuploaders/decidim/attachment_uploader.rbvalidate_dimensionsでは広さや高さが検証されてI18n.t("carrierwave.errors.image_too_big")メッセージが使われているのに、uploaders/decidim/image_uploader.rbvalidate_sizeでは容量が過ぎる場合にも同じimage_too_bigのメッセージ使われているそうです。

ここではmonkey patchしかできないみたいなので、upstreamのdecidimにバグリポートを作るのがもっとも良い解決になると思います。

halsk commented 3 years ago

@isaozz とりあえず、サイトの設定で100Mにしてみましたが、いかがでしょうか

isaozz commented 3 years ago

@halsk 相変わらず、貼ることはできないですね。

halsk commented 3 years ago

@isaozz どこにどの画像をアップロードしたいかどうか教えてもらって良いですか?

isaozz commented 3 years ago

@halsk 加古川市版であれば、参加型プロセスの「加古川市スマートシティ構想2」の「バナー画像」のところです。 画像はヒーロー画像でも構わないです。小さい画像なら大丈夫なんですが。

halsk commented 3 years ago

そうですね。バナー画像なので、縦横サイズ制限があるようです。 画像のバイト数の問題でのエラーでは無いようです。 今、加古川市の風景を試しに横1000ピクセルに変えたらアップロードできました。 [システムが画像を切り抜きます。]と書いてあるのが嘘で、実際には縦横比制限があるようなので、その辺を明示したほうが良いですね。 縦横比が大きすぎる画像は載せないほうが良いとは思うので、ここの制限はなくさないほうが良いと思うのですが、どうでしょうか。

isaozz commented 3 years ago

@halsk ありがとうございます。 大きい画像を載せる必要はありませんので、制限はなくさなくて大丈夫です。 注意書きを修正いただければ、問題ないかと思います。