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.

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

fix: 버튼의 아이콘 props type을 reactNode로 수정했습니다. #182

Merged
merged 1 commit into from
Aug 21, 2024

Conversation

summermong
Copy link
Collaborator

🤔 어떤 문제가 발생했나요?

  • 버튼 컴포넌트 사용 시, 아이콘은 next/image를 사용하여 src가 string으로 들어가야 하지만 현재 우리 팀은 아이콘을 컴포넌트로 만들고 있어 타입이 일치하지 않습니다.

🎉 어떻게 해결했나요?

  • 아이콘의 type을 string이 아닌 ReactNode로 수정하고, Button 컴포넌트 내에서도 Image가 아닌 div 태그로 랩핑했습니다.
  • 아이콘 사이즈에 대한 스타일링도 기존 스타일링과 유사하게 Map으로 수정하였습니다.

📷 이미지 첨부 (Option)

[페이지에서 사용할 때]
image

⚠️ 유의할 점! (Option)

  • 스토리북에서는 컴포넌트를 아이콘으로 넣다 보니 테스트할 수 없습니다. (하는 방법을 찾지 못했는데 아신다면 알려주시면 감사하겠습니다!)
  • 그래서 스토리를 3가지 따로 만들어두어서 여기서 다른 props로 테스트하실 수 있습니다.

image

@summermong summermong added the fix # Fix bug label Aug 21, 2024
@king2dwellsang king2dwellsang added the Review Plz🙏 # Review is not yet complete label Aug 21, 2024
Copy link

Copy link
Collaborator

@hjy0951 hjy0951 left a comment

Choose a reason for hiding this comment

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

👍

@king2dwellsang king2dwellsang added Approved 🆗 # Review is completed and removed Review Plz🙏 # Review is not yet complete labels Aug 21, 2024
@summermong summermong merged commit 3f77074 into main Aug 21, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved 🆗 # Review is completed fix # Fix bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants