Conversation
Leeyoonji23
requested review from
jisooooooooooo,
kwonsaebom and
shinjigu
as code owners
빌드 결과빌드 성공 🎉 |
빌드 결과빌드 성공 🎉 |
🎨 스토리북 배포 완료!변경된 컴포넌트의 디자인을 확인해주세요. |
빌드 결과빌드 성공 🎉 |
🎨 스토리북 배포 완료!변경된 컴포넌트의 디자인을 확인해주세요. |
빌드 결과빌드 성공 🎉 |
🎨 스토리북 배포 완료!변경된 컴포넌트의 디자인을 확인해주세요. |
| transition: 'all 0.3s ease', | ||
| ':hover': { | ||
| width: '17.5rem', | ||
| padding: '0 0 0 2.4rem', |
There was a problem hiding this comment.
p5) paddingLeft: '2.4rem' 저는 이런 거 좋아해요 ㅎㅎ
그냥 저스트 취향공유🐧~
There was a problem hiding this comment.
반영하고 싶었으나... 스타일 수정하면서 없앴읍니다...ㅎㅎ
나중에 이런 상황이 생기면 그렇게 할게요!
빌드 결과빌드 성공 🎉 |
🎨 스토리북 배포 완료!변경된 컴포넌트의 디자인을 확인해주세요. |
shinjigu
left a comment
shinjigu
left a comment
There was a problem hiding this comment.
수정하기 버튼 수고하셨습니다 윤지언니 짱 🤩🤩🤩
| transition: 'clip-path 0.3s ease', | ||
| ':hover': { | ||
| backgroundImage: colors.gradient05, | ||
| clipPath: 'circle(100% at center)', |
| justifyContent: 'flex-end', | ||
| alignItems: 'center', | ||
| gap: '1rem', |
There was a problem hiding this comment.
p4) gap 속성 실제로는 justifyContent: 'flex-end'로 인해 사용되지 않는 거 아닌가요 ? 아닐 수도 있는데 그냥 궁금해서 물어봅니다 💭
| return ( | ||
| <button type="button" className={styles.editBtnWrapper} onClick={handleClick}> | ||
| <span className={styles.editText}>수정하기</span> | ||
| <div className={styles.editIcon}> | ||
| <IcPencil className={styles.iconSvg} /> | ||
| </div> | ||
| </button> | ||
| ); |
There was a problem hiding this comment.
p4) 접근성을 위해 aria-label이나 title 속성을 추가하는 건 어떤가요 ??
| borderRadius: '50%', | ||
| backgroundColor: colors.grey3, | ||
| color: colors.white01, | ||
| zIndex: 1, |
There was a problem hiding this comment.
p2) zIndex 는 토큰화해서 분리해놨는데 그 파일에 추가하시면 더 좋을 것 같습니다 😊
There was a problem hiding this comment.
이 부분을 고민해봤는데… 새봄님이 작성해주신 zIndex 토큰화는 주로 헤더, 모달, 툴팁 등 전역적으로 우선순위가 중요한 컴포넌트에 사용되는 패턴이라, 지금처럼 저만 사용하는 (...?) 수정하기 버튼의 z-index: 1은 굳이 토큰화하지 않고 인라인으로 두는 게 더 적절하다고 판단했습니다!
다른 분들의 의견은 어떠신지 궁금해욥
4 participants
💡 Summary
✅ Tasks
👀 To Reviewer
디쌤들 의견 반영해서 수정했습니닷
📸 Screenshot
2025-07-10.9.45.52.mov