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

【残り いしかわさん担当】 レスポンシブ処理の調整 #763

Closed
kurudrive opened this issue Sep 22, 2021 · 3 comments · Fixed by #1000
Closed

【残り いしかわさん担当】 レスポンシブ処理の調整 #763

kurudrive opened this issue Sep 22, 2021 · 3 comments · Fixed by #1000
Assignees

Comments

@kurudrive
Copy link
Member

モバイル / タブレット / PC で画像の出し分けなど、画面サイズ毎の処理をしているブロックをリストアップ
各ブロックについて、
モバイル : max-width
タブレット : min-width / max-width
PC : min-width
を調べてください。

@shimotmk
Copy link
Contributor

ブロック名 SP タブレット PC それ以上
・balloon @media only screen and (max-width: 480px)
・border-box @media (min-width: 992px)
・faq @media (min-width: 768px)
・icon @media screen and (max-width: 992px)
・pr-blocks @media screen and (max-width: 992px)
・pr-content @media (max-width: 575.98px) @media (min-width: 576px) @media (min-width: 992px)
・spacer @media (max-width: 576px) @media (min-width: 577px) and (max-width: 768px) @media (min-width: 769px)
・staff @media (min-width: 992px) @media (min-width: 1200px)
PRO
・card @media (max-width: 576px) @media (min-width: 577px) and (max-width: 768px) @media (min-width: 769px)
・icon-card-item @media screen and (max-width: 992px)
・outer @media (min-width: 577px) @media (min-width: 769px)
・slider @media (max-width: 575.98px)
@media (max-width: 576px)
@media (min-width: 577px) and (max-width: 768px) @media (min-width: 769px)
・table-of-contents-new @media (min-width: 576px)

@kurudrive
vk-blocksのcssで使用しているメディアクエリは、このような形でした。

  • スマホ
    @media (max-width: 576px)
  • タブレット
    @media (min-width: 577px) and (max-width: 992px)
    max-width: 768pxが5個
    max-width: 992pxが6個
  • PC
    @media (min-width: 992px)

で統一していくのが妥当かなと思います。

@kurudrive
Copy link
Member Author

kurudrive commented Sep 24, 2021

@shimotmk ありがとうございます!完璧デス!
1px 単位でかぶっている処理等があるので

スマホ
@media (max-width: 575.98px)

タブレット
@media (min-width: 576px) and (max-width: 991.98px)

PC
@media (min-width: 992px)

イレギュラー : 管理画面側での指定はなく、CSSでブレイクポイントが一つの場合(モバイル/PC など)以下の設定がありえる
@media (min-width: 768px)

で統一しましょう!

@kurudrive kurudrive changed the title レスポンシブ処理の確認 レスポンシブ処理の確認 → 再調整 Sep 24, 2021
@kurudrive
Copy link
Member Author

kurudrive commented Sep 24, 2021

@shimotmk @drill-lancer @miminari @sysbird @mthaichi @goutetsuguma

■ 修正が必要なブロック

  • max-width: 576px → max-width: 575.98px (min が 576px なので)
  • max-width: 992px → max-width: 991.98px (min が 992px なので)
  • 直接CSSを吐き出しているものは要deprecated
  • ブレイクポイントが一つで、設定パネルで モバイル / タブレット / PC という設定指定のないものは着手しなくてOK

こちらはパターンに置き換わるから物凄く優先度低くてOKかな...

@shimotmk shimotmk linked a pull request Feb 8, 2022 that will close this issue
2 tasks
@shimotmk shimotmk changed the title レスポンシブ処理の確認 → 再調整 【残り いしかわさん担当】 レスポンシブ処理の調整 Mar 8, 2022
kurudrive added a commit that referenced this issue Mar 11, 2022
[ スペーサーブロック ] ブレイクポイント修正 #763
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 a pull request may close this issue.

6 participants