Conversation
Leeyoonji23
requested review from
jisooooooooooo,
kwonsaebom and
shinjigu
as code owners
빌드 결과빌드 성공 🎉 |
🎨 스토리북 배포 완료!변경된 컴포넌트의 디자인을 확인해주세요. |
jisooooooooooo
left a comment
jisooooooooooo
left a comment
There was a problem hiding this comment.
확인했습니당 ~~
빡센 만다르트 박스들 기깔나게 만드셨네요🤩
머지할 때 테스트한 부분만 지우고 올려주심 될 거 같아욤!
|
|
||
| const squares = Array(9) | ||
| .fill(null) | ||
| .map((_, index) => { |
There was a problem hiding this comment.
p3) map 로직이 살짝 길다는 생각이 들어서 함수 분리하는 것도 좋아보여요~!
There was a problem hiding this comment.
좋아요! 길어서 가독성이 떨어진다고 생각은 하고 있었는데, 해당 내용 반영해서 renderSquare 함수로 분리했습니다!
더해서, Index===4 인게 맘에 안들어서 (...) 상수로 분리 했으니 참고 부탁드려요~!
| export const grid = style({ | ||
| display: 'grid', | ||
| gridTemplateColumns: 'repeat(3, 1fr)', | ||
| gap: '1.9rem', |
There was a problem hiding this comment.
p2) 피그마 상에서 gap이 16px인 것 같은데 제가 잘못 봤을 수도 있어서 확인 한 번 부탁드려욤
There was a problem hiding this comment.
공통컴포넌트하고 뷰 상 gap 이 달라서 그랬나봐요! 뷰 기준으로 수정 완료했습니다~!
shinjigu
left a comment
shinjigu
left a comment
There was a problem hiding this comment.
이번 만다라트 공통 컴포넌트 정말 수고하셨어요 !! 꼼꼼하게 잘 만들어주셔서 좋아요 🫶🏻
| return <div className={styles.squareContainer}>{children}</div>; | ||
| }; | ||
|
|
||
| export const Square = { |
There was a problem hiding this comment.
p4) Fast Refresh 경고가 Square를 객체로 export 해서 발생하는 것 같아요 각 컴포넌트를 개별 export 하고 index.ts에서 묶어서 export 하는건 어떻게 생각하시나요 !?
There was a problem hiding this comment.
굿굿 좋습니다. 반영완료했습니다~~!
🎨 스토리북 배포 완료!변경된 컴포넌트의 디자인을 확인해주세요. |
빌드 결과빌드 성공 🎉 |
🎨 스토리북 배포 완료!변경된 컴포넌트의 디자인을 확인해주세요. |
빌드 결과빌드 성공 🎉 |
🎨 스토리북 배포 완료!변경된 컴포넌트의 디자인을 확인해주세요. |
3 participants
💡 Summary
✅ Tasks
👀 To Reviewer
📸 Screenshot