Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

レスポンシブサイトにした #77

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

AreaEffectCloud
Copy link
Collaborator

@AreaEffectCloud AreaEffectCloud commented Sep 3, 2024

概要

  • PC等で見てもスタイルが崩れないようにした
    • 最大の幅を決め、それを越えたら両端に余白が追加されるようにした
  • カードの移動もそれに対応させた

その他

  • HTML/CSSの一部コードのリファクタリング
  • 画像サイズの変更による、読み込み速度の上昇

関連

  • 背景については、白だと寂しいので何か画像を入れた方がいい
  • 分流作成ボタンのデザインは未着手

テスト方法

@AreaEffectCloud AreaEffectCloud requested review from 2i-7 and 3BLABLAB and removed request for 2i-7 September 3, 2024 11:33
@AreaEffectCloud AreaEffectCloud self-assigned this Sep 3, 2024
@taisukef
Copy link

taisukef commented Sep 4, 2024

なるほど、縦長維持する感じだね!
image

こちらの対応では、横幅いっぱい使う感じ
#71
image

ターゲットとビジョンと照らし合わせて対応を決めるといいよー

@3BLABLAB
Copy link
Collaborator

3BLABLAB commented Sep 7, 2024

lgtm:D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants