-
Notifications
You must be signed in to change notification settings - Fork 39
[김지현] sprint5 #178
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
[김지현] sprint5 #178
The head ref may contain hidden characters: "React-\uAE40\uC9C0\uD604-sprint5"
Conversation
main.html -> home.html main.css -> home.css
GANGYIKIM
left a comment
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.
지현님 5주차 미션 작업 고생하셨습니다~
제가 반복되는 내용에 대해서는 한번만 코멘트를 남기려고 노력했는데,
자세히 보다보니 코멘트가 좀 많네요 😅
코멘트 반영은 필수가 아니니 보시고 마음에 드시는 것만 반영해보세요~
다음 미션도 화이팅입니다~
- 기존 미션을 합치려고 하시는 중인 것 같아요. 중복되는 파일들이 보이니 추후 신경써주시면 좋겠습니다.
- 시멘틱태그 작성을 더 신경써주시면 좋겠어요. 우선 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.
💬 여담
React Router를 사용하고 계신데 향후 라우트가 많아지면 Route들을 별도의 파일로 분리하면 App 컴포넌트가 더 깔끔해지고, 라우트 관리도 쉬워집니다.
요구사항
기본
📋 중고마켓
📋 중고마켓 반응형
Desktop : 4개 보이기
Tablet : 2개 보이기
Mobile : 1개 보이기
Desktop : 12개 보이기
Tablet : 6개 보이기
Mobile : 4개 보이기
심화
주요 변경사항
스크린샷
멘토에게
https://p-anda-market.netlify.app/