-
Notifications
You must be signed in to change notification settings - Fork 0
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
[퍼블리싱] 디테일 페이지 #18
base: dev
Are you sure you want to change the base?
[퍼블리싱] 디테일 페이지 #18
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.
리뷰 모두 남겼습니다~! 고생하셨어요! 수정 완료되면 노티 부탁드립니다.
src/components/Detail/MusicAbout/MusicCommentList/MusicCommentList.tsx
Outdated
Show resolved
Hide resolved
희윤님께 음악 카테고리 부분 최대 개수 3개로 전달 받아서 따로 레이아웃은 처리해야할 부분 없을꺼 같고, |
999 오버시 K,M 사용하도록 3자리까지 디자인 수정하였다고 하셨습니다 |
999 오버시 K,M 변환 작업은 따로 빼야할꺼 같은데 어디에 빼는게 좋을까요? |
utils 폴더하나 만들구 getCounts 같은 함수 하나 만들어서 카운트 개수에 따라서 다르게 보여지게끔 처리하면 될 것 같습니다 |
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.
몇 가지 리뷰 사항이 있습니다! 반영해서 작업해주세요
디테일 페이지 퍼블리싱입니다.
Profile부분 PR 닫아주시면 반응형까지 작업해서 말씀드리겠습니다 :)