-
Notifications
You must be signed in to change notification settings - Fork 14
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
Basic 김형준 sprint1,2 #16
base: main
Are you sure you want to change the base?
Conversation
gnb/Group [email protected]
Outdated
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.
전체적으로 이미지 파일 이름들이 피그마에서 바로 받은 파일이름이더라고요 ㅠ
그리고 3x.png 이런것까지는 지금은 넣을 필요는 없답니다. :)
웹에서는 특별히 따로 쓰이진 않고, 앱개발시 해상도마다 다르게 적용되게 사용되기 위해서 주로 쓰입니다!
웹에서는 png, jpg 도 쓰이지만 주로 svg, 요즘은 webp도 많이 쓰이는 것 같아요 ㅎㅎ
정리하면 1x, 2x, 3x 이런것은 웹에서는 안쓰셔도 되고, 파일이름을 이쁘게 지어주시면 더 좋을듯 합니다~
Rectangle_2993/판다마켓산책.jpg
Outdated
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.
한글을 파일이름으로 하는것은 지양해주셔야합니다. 영어로 파일이름 바꿔주세요!
형준님! 고생 많으셨습니다! 디렉토리 구조를 신경써주시면 더 좋을것 같습니다. prettier 라는 것을 한번 확인해보셔도 좋을것 같습니다 ㅎㅎ svg 로 이미지 받아서 적용해보시는것도 추천드려요! 리뷰 관련해서 질문이 있으시면 말씀해주세요~ |
|
<div id="section1_box"> | ||
<div id="left"> | ||
<span>일상의 모든 물건을 거래해 보세요.</span> | ||
<a href="Item.html" target="_self"><button id="cta_button">구경하러 가기</button></a> |
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.
여기를 items.html로 변경해야 요구사항에 맞게끔 /items 로 가게 됩니다. 전체적으로 html파일들의 이름을 소문자 + 요구사항에 맞게끔 변경해주세요!
@Kimhyeog 님 이거 PR에서 base 확인해주세요! codeit-sprint-fullstack:main 이 아니라 codeit-sprint-fullstack:basic-김형준 이렇게 해야할듯 합니다~ |
This reverts commit 23c1d79.
요구사항
기본
랜딩 페이지
cursor: pointer
로 설정합니다.로그인, 회원가입 페이지 공통
#3692FF
입니다.로그인 페이지
회원가입 페이지
심화