-
Notifications
You must be signed in to change notification settings - Fork 31
[이현석] sprint1 #145
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
[이현석] sprint1 #145
The head ref may contain hidden characters: "Basic-\uC774\uD604\uC11D-sprint1"
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.
현석님~! 첫 번째 스프린트 미션 진행 하시느냐고 고생 많으셨습니다 :)
스프린트 미션 1에서는 보기 드물게 css가 기능별로 잘 정리 됐네요~! 🤣 다만, 대부분의 요소가 이미지로 구현되어 있어서 리뷰 드릴 수 있는 부분이 많이 없습니다 😢
피그마를 다시 확인해 주시고 이미지와 html 요소를 분리해 주세요~!

혹시 피그마 모드가 잘못 되어있어서 이미지로 보셨나 싶기도 한데, dev모드가 잘 되어있는 지 확인해 보세요!

html 요소가 이미지로 대체 된다면 SEO에 좋지 않습니다! 검색 엔진에서 볼 수 있는 내용이 없는거죠..! 또한 반응형이나 사용자 인터렉션 구현에도 많은 어려움이 생깁니다 :)
5, 6번째 기본 요구사항은 잘 이해가 안 돼서 제대로 했는지 모르겠어서 체크를 못 했습니다 😢
-> 5번의 경우 위에 말씀 드린 거 처럼 통째로 이미지로 넣는 것이 아니라 내부에 텍스트, 구경하러가기 버튼, 판다 이미지 모두 html 요소로 구현하셔야 합니다 :) 화면 크기가 커지면 그 요소들의
위치는 고정되고 파란색 배경색만 커지는거죠!
6번은 화면이 줄어들 때 로고와 로그인 버튼 좌우 여백이 줄어들 게 됩니다. 이 때 줄어드는 여백은 200px로 유지해 주시고, 200px 보다 더 줄어들어야 하면 로고와 로그인 버튼 사이 간격을 줄여주시면 됩니다 :)


"구경하러 가기"는 버튼을 따로 만들어야 하나요? 이미지에 다 박혀 있어서 따로 분리할 수가 없어서 하지 못했습니다 😭
-> 맞습니다..! 위에 말씀드린대로 이미지를 통째로 넣는 것이 아니라 각각 구현하셔야 합니다 :)
심화 또한 잘 이해가 안되서 제대로 했는지 안했는지 몰라 체크를 못했습니다 ㅠ 😭
-> 의도는 rem, % 등 유연한 단위를 사용하는 것입니다! 이미 잘 사용하고 계신 거 같아서 크게 신경쓰지 않으셔도 될 거 같아요 :)
| @@ -0,0 +1,73 @@ | |||
| <!DOCTYPE html> | |||
| <html lang="ko"> | |||
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.
lang을 잘 챙겨주셨네요! 👍
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.
reset 을 적용해 주셨군요! 👍
요구사항
기본
심화
주요 변경사항
스크린샷
nav

main

footer

멘토에게