砧Dojoのロゴ画像を追加#1741
Merged
Merged
Conversation
rakuda-san-desu
left a comment
rakuda-san-desu
left a comment
Contributor
There was a problem hiding this comment.
対応ありがとうございます！表示を確認しました🙆♀️
画像について、以下の対応をお願いできると助かります🙏
- 画像を正方形に調整
- ピクセルサイズを400×400程度に調整
- データサイズを圧縮
- 可能なら
webp形式に変換
logoのロゴ画像は TinyPNG で圧縮してから public/img/dojos に置いてください。
- サイズが正方形以外の場合、表示が崩れる場合があるので、Macのプレビューで画像に余白を付け足す方法を参考に正方形にすると良さそうです。
Contributor
Author
Member
|
ロゴ画像の対応ありがとうございます!! マージしますね...!! 🚀✨ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CoderDOjo 砧のロゴを、メールでいただいた画像に差し替えました！