Skip to content

Conversation

@grapefruit13
Copy link
Contributor

유형

  • 기능 구현
  • UI 구현
  • 리팩토링
  • 버그 해결
  • 문서 업데이트
  • 기타( )

작업 내용

  • Button 컴포넌트 리디자인 및 테스트 페이지 구현

설명 (선택)

  • middleware 주석 처리하시고 /button-test 가시면 버튼 UI 확인 가능합니다.

스크린샷

image

리뷰 요구사항

  • size prop 이름은 피그마 시안 그대로 가지않고 xs, sm, md, lg 이렇게 넣었는데 괜찮으신가요? 시안에서는 xs, s, ... 이런 식으로 되어있었어요.
  • circular(동그라미)는 variant랑 다른 프롭으로 분류하는게 맞는 것 같아서 width에 넣었는데, width라는 prop명이 어울리는지는 잘 모르겠어요. 좋은 아이디어 있으시면 알려주세요!

@grapefruit13 grapefruit13 self-assigned this Jan 8, 2025
@grapefruit13 grapefruit13 requested a review from plla2 January 8, 2025 13:58
Copy link
Member

@plla2 plla2 left a comment

Choose a reason for hiding this comment

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

circular에서 너비와 높이를 고정으로 주어야하니까 width에 넣는게 좋아보입니다! 고생하셔써여
Input 빨리해야겠네여...

@grapefruit13 grapefruit13 merged commit 37ea2c4 into dev Jan 9, 2025
1 check passed
@grapefruit13 grapefruit13 deleted the THKV-108 branch January 9, 2025 04:43
plla2 added a commit to plla2/NNplanner-FE that referenced this pull request Jan 9, 2025
Feat[THKV-108] : Button 컴포넌트 리디자인 및 테스트 페이지 구현
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants