-
Notifications
You must be signed in to change notification settings - Fork 26
[김단이] Sprint5 #128
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
base: react-김단이
Are you sure you want to change the base?
[김단이] Sprint5 #128
The head ref may contain hidden characters: "react-\uAE40\uB2E8\uC774-sprint5"
Conversation
dokdo2013
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.
- memo 폴더 올리신거 지우셔도 될 것 같습니다 ㅎㅎ
- 아직 스프린트 미션 파트는 구현 안 하신 걸로 이해했고, 이거는 다음 리뷰때라도 작업해서 올려주시면 리뷰해드리겠습니다!
package.json
Outdated
| }, | ||
| "dependencies": { | ||
| "axios": "^1.8.2", | ||
| "express": "^4.21.2", |
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.
오잉 express도 여기서 설치하셨네요? 미사용이면 지우셔도 될 듯 합니다.
| <div className="nav"> | ||
| <Link to="/" className="homepage-logo-link"> | ||
| <img className="homepage-logo" src={logo} alt="판다마켓 로고" /> | ||
| </Link> | ||
| <Link className="homepage-login" to="/login">로그인</Link> | ||
| </div> |
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.
큰 이슈는 아니고, 이왕 nav로 className 지정하셨으니... div 태그 대신 nav 태그를 쓰는게 HTML 웹 접근성 측면에서 더 좋아보입니다
🏁 스프린트 미션 5 - 시안 확인
🔗 참고 링크
배포링크
스프린트 미션 5 Figma 디자인 확인
✅ 요구사항 체크리스트
기본 요구사항
공통
중고마켓 페이지
🔥 심화 요구사항
공통
중고마켓 페이지
중고 마켓의 카드 컴포넌트 반응형 기준은 다음과 같습니다.
베스트 상품
전체 상품
반응형에 따른 페이지 네이션 기능을 구현합니다.
🛠 주요 변경사항
📸 스크린샷
📝 멘토님에게 남길 메시지