issues
search
depromeet
/
ggogeet-client
GgoGeet - 꼬깃 접어 전하는 이야기💌
13
stars
2
forks
source link
✨ Button 공통 컴포넌트 추가
#27
Closed
numeru
closed
1 year ago
numeru
commented
1 year ago
📎 이슈번호
13
📃 변경사항
Button
공통 컴포넌트를 추가했습니다
BottomButton
의 스타일에서 prop 에 따른 패딩 지정이 반대로 되어 있어서 수정했습니다
📌 중점적으로 볼 부분
가장 무난한 기본 버튼인 것 같아서 이름을
Button
으로 했는데 다른 의견있으시면 수정하겠습니다
버튼 내에서도 유사한 것들끼리 공통 컴포넌트를 만들고 있는데 규격에 일관된 규칙이 없어서 조금씩 다른 부분을 처리하다보니 prop 도 늘고 괜찮은 방법인가 싶네요 괜찮은 방법 있으시면 말씀 부탁드립니다!
🎇 스크린샷
✔ 체크리스트
[x] 브랜치 전략에 맞는 브랜치에 PR을 올리고 있습니다. (master/main이 아닙니다.)
[x] 커밋 메세지를 컨벤션에 맞추었습니다.
[x] 변경 후 코드는 컴파일러/브라우저 warning/error 가 발생시키지 않습니다.
[x] 변경 후 코드는 기존의 테스트를 통과합니다.
[x] 테스트 추가가 필요한지 검토해보았고, 필요한 경우 테스트를 추가했습니다.
[x] docs 수정이 필요한지 검토해보았고, 필요한 경우 docs를 수정했습니다.
📎 이슈번호
13
📃 변경사항
Button
공통 컴포넌트를 추가했습니다BottomButton
의 스타일에서 prop 에 따른 패딩 지정이 반대로 되어 있어서 수정했습니다📌 중점적으로 볼 부분
Button
으로 했는데 다른 의견있으시면 수정하겠습니다🎇 스크린샷
✔ 체크리스트