-
Notifications
You must be signed in to change notification settings - Fork 31
[전수영] sprint3 #169
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
[전수영] sprint3 #169
The head ref may contain hidden characters: "Basic-\uC804\uC218\uC601-sprint3"
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수영님 이번 미션도 고생 많으셨습니다!
반응형을 점진적으로 잘 구현해 주셨네요! 👍 자바스크립트도 기대하겠습니다!!
메인 페이지, 태블릿이나 모바일에서 height 를 고정으로 할 때 어떤 수치를 사용하는 게 좋을지 확인 부탁드립니다.
모바일 기기에서는 vh 를 사용해도 좋을 것 같았는데, 기기 높이가 너무 길어지는 경우에는 화면이 너무 길어지는 것 같아서,
px 로 사용해야하는 건지 잘 모르겠어서 문의드립니다!
->
서비스 성격이나 디자인 의도에 따라 다를 수 있지만, 저는 웬만한 경우 max-height과 px로 합니다. 혹은 따로 높이를 정하지 않고 화면이 줄어들면서 줄어드는 내부 콘텐츠(이미지 등)의 크기에 맞게 자연스럽게 맞춰지도록 구현하는 편이에요.
%, vh를 사용하는 경우 결과를 예측 하기가 힘든 경우가 많습니다. 따라서 이런 상대 단위를 쓸 때는 정확한 비율이 주어질 때 쓰는 편입니다. 예를 들면 부모의 절반 정도의 크기일 때 50%를 주는 식으로요!
지금 padding-left: 8%로 이런 식으로 수치를 준 코드들이 보이는데, 저는 위의 이유로 그런 방식을 지양하는 편입니다. (그렇다고 틀린 거는 아니에요 🤣 )
추가로, 특히 vh, vw의 경우는 조심하셔야 합니다. vh, vw는 브라우저 창 크기를 나타내는 거라 스크롤이 있을 때와 없을 때 다른 요소에 영향을 줄 수 있고, 특히 모바일의 경우 주소창 까지 크기를 포함하기 때문에 주소창이 중간에 없어지면 UI가 틀어지는 경우도 있습니다.
어느정도로 유동적으로 구현 해야하는지가 피그마에 잘 안 나타날 때도 있습니다! 실무에서는 디자이너분과 의도에 대해 충분히 이야기할 수 있으니 너무 걱정하지 않으셔도 됩니다 :)
| <meta name="description" content="일상의 모든 물건을 거래해보세요" /> | ||
| <!-- 페이스북, 카카오톡 --> | ||
| <meta property="og:title" content="판다 마켓" /> | ||
| <meta property="og:description" content="일상의 모든 물건을 거래해보세요" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
og도 잘 넣어 주셨군요!! 👍
| line-height: 26px; | ||
| } | ||
| } | ||
| @media screen and (max-width: 767px) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
점진적으로 반응형을 잘 적용해 주셔서 중복이 많이 없는 거 같네요! 👍
| } | ||
| .footer { | ||
| position: relative; | ||
| padding-left: 8%; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
패딩, 갭 같은 부분들도 마찬가지로 %로 주었을 때 애매해질 수 있습니다 :)
| .header { | ||
| padding: 0 24px; | ||
| } | ||
| .footer { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.

요구사항
기본
심화
주요 변경사항
스크린샷
메인 (pc)

메인 (태블릿)

메인 (모바일)

멘토에게
모바일 기기에서는 vh 를 사용해도 좋을 것 같았는데, 기기 높이가 너무 길어지는 경우에는 화면이 너무 길어지는 것 같아서,
px 로 사용해야하는 건지 잘 모르겠어서 문의드립니다!