-
Notifications
You must be signed in to change notification settings - Fork 0
RINGUS-19 feat: 멘토 목록 페이지 #24
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
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.
현재 아이콘을 lucide-react라는 곳에서 가져와서 사용하고 있어요! signup 부분에서 CreatePassword.tsx 파일을 보시면 lucide-react 라이브러리에서 check 아이콘을 가져와서 사용한 부분이 있는데 그거 참고해서 최대한 figma에서 보이는 디자인과 같은 아이콘이 lucide icons에 있다면 사진 보다는 아이콘을 가져와서 사용하는 것이 유지보수나 성능적으로 더 좋을 것 같아요!
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.
현재 MentorshipList가 하나의 컴포넌트에서 여러 역할(필터링, 공통 버튼(FilterButton), 모달(FilterModal), 상태관리 등) 을 담당하고 있어서 코드 가독성이 떨어질 것 같습니다. 중복되는 코드를 따로 components로 분류하여 사용하는 것은 어떨까요?
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.
넵 반영하였습니다
SubJeeLee
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.
현재 어느 정도의 단위로 커밋을 해야 하는지와 커밋 주기에 대해서 논의 해봐야 할 것 같습니다. 제가 생각하기에는 다른 팀원들이 커밋을 보고 쉽게 이해할 수 있도록 작은 단위로 쪼개서 커밋 해주는 것이 좋을 것 같은데 다른 팀원분들 의견도 궁금합니다. 예를 들면 사진 같은 것들 따로 모아 커밋하고 tailwind 부분 추가된 것들 있으면 분류해주고.. 어떤가요?
|
필터 버튼 눌렀을때 색상 변해도 좋을 것 같아요! |
✨ 구현한 기능
📢 논의하고 싶은 내용
🎸 기타