-
Notifications
You must be signed in to change notification settings - Fork 31
[고서영] Sprint 3 #120
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
[고서영] Sprint 3 #120
The head ref may contain hidden characters: "Basic-\uACE0\uC11C\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.
서영님, 이번 미션도 깔끔하게 잘 마무리해 주셨네요! 👍
자바스크립트도 기대하겠습니다! :)
코멘트 주신 부분은 리뷰 참고해 주세요!
| <link rel="stylesheet" href="./styles/components/footer.css" /> | ||
| <link rel="stylesheet" href="./styles/components/gnb.css" /> | ||
| <!-- Open Graph (OG) 메타 태그 --> | ||
| <meta property="og:title" 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를 챙겨주셨군요! 👍
| /> | ||
| <meta | ||
| property="og:image" | ||
| content="https://www.google.com/imgres?q=%ED%8C%90%EB%8B%A4&imgurl=https%3A%2F%2Fi.namu.wiki%2Fi%2FN7XtnLu7mENPBU1wMlAoOo5_w13roksendvuswR8gkFw_8SilcxCpT3kTTdzaP42jSpZAQ2-R4x3aNxaj6A3JA.webp&imgrefurl=https%3A%2F%2Fnamu.wiki%2Fw%2F%25ED%258C%2590%25EB%258B%25A4&docid=5YXeSdnyXQW-OM&tbnid=L95TdOcl4sN4TM&vet=12ahUKEwiptZXLvcyMAxU7sVYBHUTEITMQM3oECG4QAA..i&w=578&h=385&hcb=2&ved=2ahUKEwiptZXLvcyMAxU7sVYBHUTEITMQM3oECG4QAA" |
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.
음.. 지금은 사실 크게 중요하지는 않지만, 이미지가 아니라 나무위키 이미지 페이지를 불러오는 거 같네요! 🤣
| name="password" | ||
| placeholder="비밀번호를 입력해 주세요" | ||
| /> | ||
| <button> |
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.
form 안에 있는 버튼은 type이 submit으로 됩니다 :) submit이 아니라 다른 기능이라면 type을 명시해 주셔야 해요!
| width: 100%; | ||
| padding: 16px 71px; | ||
| font: var(--text-2lg-semibold); | ||
| transition: 0.5s; |
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.
깨알 같은 transition.. 🤣 👍
| .landing-text h2 { | ||
| text-align: center; | ||
| } | ||
| .landing-text.top { |
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.
경우에 따라 width로 줄바꿈을 조절하는 것도 가능하지만, br 과 display를 활용해 보시면 조금 더 쉽게 원하는 UI를 만들 수 있습니다!
| padding: 32px 32px 0 32px; | ||
| } | ||
|
|
||
| .footer-info { |
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 css 부분에 대한 자세한 피드백 부탁드립니다.
-> 특별히 리뷰 드릴 부분 없이 너무 잘 하셨습니다 👍 absolute의 경우 기준이 되는 요소의 크기가 변하면 깨질 가능성이 크지만, 현재 footer의 경우 크기가 고정 되어있으니 상관 없을 거 같습니다 :)
다른 방법을 쓰고 싶으시다면 flex와 order, grid와 grid-template-areas 등을 사용해 보실 수 있습니다!
| .footer-links { | ||
| justify-content: space-between; | ||
| width: 50%; | ||
| margin-right: auto; |
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-wrapper가 이미 flex를 가지고 있으므로 footer-wrapper에 space-beteen을 추가해줘도 되겠네요 :)
| right: 24px; | ||
| top: 50%; | ||
| transform: translateY(-50%); | ||
| transform: translateY(-80%); |
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.
| alt="home-card02" | ||
| /> | ||
| <div class="section-home-text reverse"> | ||
| <h1>Search</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가 참 성가시죠.. 😭
h1 tag는 페이지에 하나만 사용됩니다~! 🤣
https://developer.mozilla.org/ko/docs/Web/HTML/Element/Heading_Elements#사용_일람
|
|
||
| } | ||
|
|
||
| .section-home-text 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.
모바일 폰트 크기를 확인해 주세요 :)

요구사항
기본
공통
1200px이상768px이상 ~1199px이하375px이상 ~767px이하375px미만 사이즈의 디자인은 고려하지 않습니다랜딩 페이지
24px, “로그인” 버튼 오른쪽 여백24px을 유지할 수 있도록 “판다마켓” 로고와 “로그인" 버튼의 간격이 가까워집니다.16px, “로그인” 버튼 오른쪽 여백16px을 유지할 수 있도록 “판다마켓” 로고와 “로그인" 버튼의 간격이 가까워집니다.로그인, 회원가입 페이지 공통
16px제외하고 내부 요소들이 너비를 모두 차지합니다.400px을 넘지 않습니다.심화
주요 변경사항
스크린샷
멘토에게