-
Notifications
You must be signed in to change notification settings - Fork 33
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 #52
The head ref may contain hidden characters: "Basic-\uAE40\uC218\uC5F0"
[김수연] Sprint3 #52
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.
수연님 이번 미션도 고생 많으셨어요!!
대체로 잘 구현해주셨습니다 :)
미디어쿼리에 브레이크 포인트, 중복 스타일 들어간 부분을 전반적으로 다시 살펴주세요!
Sprint2 미션 때 피드백 주신 "이미지가 지그재그로 배치되어야 합니다!" 이건 어떤 것을 의미하는지 모르겠습니다... 올바르게 잘 한 것 같은데 혹시 자세히 알려주실 수 있을까요?
-> 죄송합니다..! 🙇♂️ 제가 이전에 사용됐던 피그마를 기준으로 말씀 드렸네요 😢
숨김 아이콘으로 인해 화면을 줄여도 제대로 작동하지 않습니다.. 어떻게 해야하나요..? 어떤 이유 때문에 이런건가요..
-> position과 관련된 내용을 다시 살펴보셔야 합니다 :)
https://developer.mozilla.org/ko/docs/Web/CSS/position
relative는 본래 자신의 위치를 기준으로 움직이죠! 따라서 password-container가 줄어든다고 해서 같이 움직이지 않습니다. password-container를 기준으로 움직이게 하기 위해 absolute를 활용해 보세요!
netilfy를 써서 미리보기를 테스트 해봤지만 이미지가 제대로 보이지 않습니다. 짤림 현상이 일어나느 것 같은데 어떻게 해야하나요,,
-> 이미지가 너무 큽니다~! 현재 2664 × 1383인데 권장사항은 1200 x 630입니다!
https://developers.facebook.com/docs/sharing/webmasters/images/
@@ -31,26 +40,26 @@ <h1 class="header-title-font">일상의 모든 물건을 거래해 보세요</h1 | |||
<img src="./img/Img_home_01.png" class="main-popular-sell-image" alt="판다마켓 인기 상품 사진"> | |||
<div class="main-theme-basic"> | |||
<div> | |||
<h1 class="main-popular-sell-font-top">Hot item</h1> | |||
<h2 class="main-popular-sell-font-top">Hot item</h2> |
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.
h tag 계층 구조를 다시 확인해주세요! (여기서 꼭 h tag를 쓰지 않아도 괜찮습니다!)
https://developer.mozilla.org/ko/docs/Web/HTML/Element/Heading_Elements#사용_일람
@@ -13,21 +13,23 @@ | |||
</div> | |||
<!-- form --> | |||
<form class="container-box"> | |||
<h1 class="email-password-font">이메일</h1> | |||
<div class="email-password-container"> | |||
<label class="email-password-font"> |
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.
label을 의미뿐 아니라 기능도 가지고 있는 태그입니다! 해당 기능을 위해서는 for을 사용하셔야 합니다 :) 참고해주세요~!
https://developer.mozilla.org/ko/docs/Web/HTML/Element/label
<button class="login-button"> | ||
<p class="login-button-font">로그인</p> | ||
<h1 class="login-button-font">로그인</h1> |
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.
h tag는 해당 페이지의 제목을 표현합니다! 로그인 기능을 하는 버튼에 들어가는 것은 부적절해 보여요!
.login-header-logo{ /**/ | ||
height: 66px; | ||
width: 12.38rem; | ||
height: 4.13rem; |
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이 중복이군요 🤣
} | ||
|
||
/** ipad **/ | ||
@media (max-width: 768px){ |
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.
미디어 쿼리를 다시 확인해주세요~!
태블릿은 768~1199 입니다!
} | ||
} | ||
|
||
@media (max-width: 375px){ |
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.
여기도 다시 확인해주세요!
margin-top: 14.385rem; | ||
} | ||
|
||
.login-header-logo{ /**/ |
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.
반응형 이미지를 쓰실 때 picture를 써보셔도 좋아요!
지금은 큰 이미지를 다운 받고 css로 줄이고 있는데, 이미지는 페이지 성능에 큰 영향을 끼치는 요소중 하나입니다. 모바일 화면이라면 애초에 큰 이미지를 가져올 필요가 없는거죠 :)
} | ||
|
||
|
||
.email-box{ |
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.
미디어쿼리 내부에 중복 스타일이 상당히 많이 들어가 있는 거 같습니다!
위에 정의한 스타일 외에 반응형에 추가로 필요한 요소만 넣어주시면 됩니다 :)
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.
요구사항
기본
심화
주소와 이미지는 자유롭게 설정하세요.
주요 변경사항
스크린샷
멘토에게