Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[FIX] 버튼 컴포넌트 사용성 개선 #70

Merged
merged 3 commits into from
Aug 28, 2024
Merged

[FIX] 버튼 컴포넌트 사용성 개선 #70

merged 3 commits into from
Aug 28, 2024

Conversation

sera2222
Copy link
Contributor

작성자: @sera2002

체크 리스트

  • 적절한 제목으로 수정했나요?
  • 상단에 이슈 번호를 기입했나요?
  • Target Branch를 올바르게 설정했나요?
  • Label을 알맞게 설정했나요?

작업 내역

  • 기존에 만들어두었던 버튼 컴포넌트를 개선하는 작업을 진행하였습니다.
  • React.ComponentProps<'button'>을 사용하여 button HTML 요소의 속성을 상속받도록 수정했습니다.
  • 기존 버튼 컴포넌트에 className 전달 시 해당 속성이 overwrite되는 문제가 있어서, classnames 라이브러리를 사용해 버튼 컴포넌트의 스타일과 새롭게 전달한 스타일이 모두 적용될 수 있도록 하였습니다.
  • 기존에는 버튼 내용으로 string 타입의 label만 받아오도록 되어있었는데, children을 받아오는 것으로 변경했습니다.

비고

  • 참고 사항을 적어주세요. 코드 리뷰하는 사람이 참고해야 하는 내용을 자유로운 형식으로 적을 수 있습니다.

close/resolve/fix #69

@sera2222 sera2222 added the fix label Aug 18, 2024
@sera2222 sera2222 self-assigned this Aug 18, 2024
@github-actions GitHub Actions
Copy link

🚀storybook: https://669667110ecf1f847fa21749-hhgcmvxpbq.chromatic.com/

@sera2222 sera2222 merged commit 8d4f17c into develop Aug 28, 2024
1 check passed
@github-actions GitHub Actions
Copy link

🚀storybook: https://669667110ecf1f847fa21749-ffsxsrxabg.chromatic.com/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hynseok hynseok hynseok approved these changes

Assignees

@sera2222 sera2222

Labels
fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

🚀 [FIX] 버튼 컴포넌트 사용성 개선
2 participants
@sera2222 @hynseok