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] ButtonBar 컴포넌트 퍼블리싱 #79

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from

Conversation

seong-hui
Copy link
Collaborator

🛰️ 관련 이슈

해결한 이슈 번호를 작성해주세요
close #77

🧑‍💻 작업 내용

작업한 내용을 간략히 작성해주세요

  • ButtonBar 컴포넌트 퍼블리싱을 하였습니다.
  • flowerBtn 컴포넌트를 구현하였습니다.
  • 기존의 TextBtn이 size에 따라서 색상이 다르게 설정되어서 해당 컴포넌트 스타일을 수정하였습니다.

🗯️ PR 포인트

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

  • BottomBar 버튼이 reset인 경우와 wish인 경우 2가지로 나누었는데 확장성이 떨어진다고 생각이 들긴하지만 일단 이렇게 만들었습니다. 더 좋은 방식이 있다고 생각하시면 말씀해주세요!

🚀 알게된 점

기록하며 개발하기!

📖 참고 자료 (선택)

참고했던 문서들 공유하기!

📸 스크린샷 (선택)

  • bottomBtn
image
  • flowerBtn
image

Copy link

github-actions bot commented Jan 15, 2025

🪷 Storybook 확인 🪷
🔗 https://677fa9af1269762b7858c29f-bxkzxaqpqd.chromatic.com/

Copy link
Collaborator

Choose a reason for hiding this comment

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

해당 파일에 초기화 버튼 클릭시에 색이 어두워지도록 적용하면 좋을 것 같아요!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

reset버튼일 경우에는 &:active 해당 효과 추가해서 클릭할때마다 텍스트 색상이 변경될 수 있도록 수정하였습니다! 해당 부분에 대해서 생각하지 못했었는데 좋은 리뷰 감사합니다

Copy link
Collaborator

@Taew00k Taew00k left a comment

Choose a reason for hiding this comment

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

깔끔하게 잘 작성해주신 것 같아요 고생하셨습니다!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] ButtonBar 컴포넌트 퍼블리싱
2 participants