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

[Feat/#9] 버튼 컴포넌트 제작 #10

Merged
merged 8 commits into from
Feb 1, 2025
Merged

[Feat/#9] 버튼 컴포넌트 제작 #10

merged 8 commits into from
Feb 1, 2025

Conversation

sikkzz
Copy link
Collaborator

@sikkzz sikkzz commented Jan 29, 2025

💡 변경사항 & 이슈

버튼 컴포넌트 제작

✍️ 관련 설명

Button, IconButton 컴포넌트 제작 완료
BaseButton은 위 버튼에 기본 틀로 생각하면 될 거 같고 직접적인 UI 컴포넌트는 아니야 (확장성 고려)

구성은 radix ui 오픈 소스랑 비슷해서 좀 더 프로페셔널(?)한 코드를 원한다면 오픈 소스 코드 한번 보면 좀 더 이해하기 쉬울 거 같아

storybook은 좀 더 확인하기 편하라고 미리 추가했는데 storybook docs나 test code 작성은 좀 이후에 작업할 예정이야

버튼은 종류별로 primary, secondary, disabled
아이콘 버튼은 사이즈별로 md, sm으로 나눠봤는데 더 좋은 의견 있으면 얘기 부탁해!

참고로 버튼 width들은 따로 지정 안했는데 버튼의 width를 지정하기보다 버튼 부모 컴포넌트의 width를 지정해서 작업하는게 나을 거 같아서 버튼 width들은 다 100% 기준이야. 어차피 버튼 크기가 고정이 아니라 반응형으로 달라지는게 더 좋을 거 같아서!

추가적으로 global.scss도 추가해줬어. 원래 reset.scss에 있다가 이후 설정 확장성을 고려해서 global.scss로 따로 뺏어
global 스타일들은 내 임의대로 지정했는데 혹시 의견 있으면 얘기해줘!

⭐️ Review point

  • 누락된 버튼이나 추가해야할 버튼 있는지
  • storybook에 잘 보이는지
  • global 스타일 수정 or 개선점
  • 이외 추가 의견

📷 Demo

스크린샷 2025-01-30 오전 1 22 53

@sikkzz sikkzz added the Feature new feature label Jan 29, 2025
@sikkzz sikkzz self-assigned this Jan 29, 2025
@sikkzz sikkzz requested a review from lgrin-byte as a code owner January 29, 2025 16:25
@sikkzz sikkzz linked an issue Jan 29, 2025 that may be closed by this pull request
3 tasks
Copy link

github-actions bot commented Jan 29, 2025

🧷 Storybook: https://6798a432c75b75bf6b81bc26-numjvcfnvh.chromatic.com/

⏰ Update: 2025년 01월 31일 14시 26분

@sikkzz sikkzz merged commit ba61d37 into develop Feb 1, 2025
3 checks passed
@sikkzz sikkzz deleted the feat/#9 branch February 1, 2025 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] 공통 버튼 추가
2 participants