Skip to content

Conversation

@juheehasaeyo
Copy link
Member

🔥 Related Issues

✅ 작업 내용

Button 컴포넌트에 outlined props 추가했습니다. 판매리스트의 모달창에도 동일한 디자인으로 적용하면 될 것 같습니다!
디자이너님께 버튼 텍스트 관련해서 문의 남겨둔 상태라 답변오면 공유드리겠습니당😉

before after

@juheehasaeyo juheehasaeyo self-assigned this Sep 16, 2025
@juheehasaeyo juheehasaeyo added the ✨ feat 기능 구현 작업 label Sep 16, 2025
@juheehasaeyo juheehasaeyo marked this pull request as ready for review September 16, 2025 15:45
Copy link
Member

@jungwoo3490 jungwoo3490 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

요것도 좋은데, 더욱 범용성 있게 설계해보는 것도 좋을 것 같아요. 배리에이션이 다양할 수도 있고, outlined의 테두리 색이 변경될 수도 있기 때문이에요.
이런 부분들을 고려해서 보통 디자인 시스템 버튼 컴포넌트는 theme (mint | red | grey | ...), variant (fill | outlined | ...) 과 같은 prop을 받습니다. 저희도 계속 색상이나 배리에이션이 추가되고 있는데, 추후 새로운 유형의 버튼이나 색상이 추가되면 수정이 쉽도록 이렇게 재설계해보는 것도 좋을 것 같아요 😃

@jungwoo3490 jungwoo3490 self-requested a review September 18, 2025 11:27
@juheehasaeyo
Copy link
Member Author

theme (mint | red | grey | ...), variant (fill | outlined | ...) 과 같은 prop을 받습니다. 저희도 계속 색상이나 배리에이션이 추가되고 있는데, 추후 새로운 유형의 버튼이나 색상이 추가되면 수정이 쉽도록 이렇게 재설계해보는 것도 좋을 것 같아요 😃

확인했습니다!! 현재 코드도 저희 프로젝트에서는 충분해서 추후 배리에이션이 확장된다면 그때 리팩토링하는 걸로 하겠습니다~!
(구두 논의 완료)

@juheehasaeyo juheehasaeyo merged commit cdf230d into main Sep 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ feat 기능 구현 작업 size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: Button 컴포넌트 수정

3 participants