Closed ryamakuchi closed 5 years ago
ryamakuchi
commented
5 years ago https://github.com/kazupon/vuefes-2019/pull/192/commits/afe8bbd05d757ec366e9d3a56c1b88e5bda1e7d6
https://twitter.com/masaakikunsan/status/1179326440317313027
こんなツイート見つけたので直しました 😅（ちょっと関係ないコミットなのですがそんなに大したことないと思うので、混ぜちゃいました。もし別にした方が良いよってときは別にするので言ってください 🙏 ）
yotaszk
commented
5 years ago @ryamakuchi デザイン、バッジの微調整、ありがとうございます。大丈夫だと思います！
一点気になったのですが、画像が荒いのは僕がupした画像が荒いからですかね🤔🤔
inouetakuya
commented
5 years ago @yotaszk @ryamakuchi CC: @448jp
上が Figma から 2x でダウンロードした PNG で、下がプレビュー環境の PNG です（圧縮されています）
たしかに一番左上の画像が圧縮前に較べて少しボヤけているように感じますね。
が、この画像、ファイルサイズが大きくて、
となっています。
まだ計測していませんが、おそらくダウンロード時間に影響が及ぶと思います。
ちょっと日にちが日にちなので、アプリへの導線を早く出してしまいたいのと、公開に足る基準は満たしていると判断したという理由で、この PR は現状のものでマージしたいです。
で、画像の品質調整（ダウンロード時間とのバランスを見る）は別 PR 扱いにさせてください。
@ryamakuchi
@yotaszk と相談しつつ、
あたりを参考にして画像の品質調整をお願いします〜 :pray:
（分からない点があったら、遠慮なく呼び出してください〜）
ryamakuchi
commented
5 years ago @inouetakuya @yotaszk CC: @448jp
レビューありがとうございます！
画像の荒さ、私の画像書き出し方法が悪かったのかな、と思いもう一度 Figma から 2x を取得して上げ直してみました 🤔 （まだ荒い気がしますが...）
画像の調整については別 PR でとのこと承知しました！よろしくお願いします。
inouetakuya
commented
5 years ago @ryamakuchi
はて。ちょっと劣化してしまったような...（左が こちら で、右下がプレビュー環境です）:thinking:
「2018年は Vue.js にとって飛躍の年でした」あたりのボヤけが進んでしまったようなので、一旦 https://github.com/kazupon/vuefes-2019/pull/192/commits/6a126fec8f4b31ba0a433e3c3faced7a5c14efed を Revert してもらってよいでしょうか？:pray:
上記 Revert が終わってテストが通ったら、マージしちゃって OK です！引き続き別 PR で画質の調整をお願いします 🙏
yotaszk
commented
5 years ago リリースは賛成ですー🙋♂️
@ryamakuchi 僕の方で元のソフトから画像書き出して送ります。別issue立てるのでそちらで対応引き続きよろしくお願いします🙆♂️
@inouetakuya @448jp ありがとうございます
ryamakuchi
commented
5 years ago @inouetakuya 確かに 💦 Revert しました。
レビューありがとうございます！マージします。
resolve https://github.com/kazupon/vuefes-2019/issues/182
TODO
レビューポイント
参考
Figma https://www.figma.com/file/BpIgcZdusbS3CgPHQhhK79T7/Vue-Fes-Japan-2019-Web?node-id=0%3A1