버튼 컴포넌트 수정 (Issue #179) #286
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Parkhanyoung
approved these changes
Aug 13, 2024
| } | ||
|
|
||
| const buttonSize = (size: ButtonSize) => { | ||
| switch (size) { | ||
| case 'half': |
There was a problem hiding this comment.
[A]
하드코딩하면 철자 실수가 발생할 수도 있으니 variant들을 상수화해서 사용해도 좋을 것 같아요!
| @@ -38,30 +24,6 @@ export const getMissionById = async (id: number): Promise<MissionWithDescription | |||
| return mission; | |||
| }; | |||
|
|
|||
There was a problem hiding this comment.
[C]
아래 친구들은 삭제된 건가요~? 😯
There was a problem hiding this comment.
네~ 기획 변경 전 미션 현황 페이지의 배너에서 사용되었던 '리뷰 완료 알림' '진행 중인 미션' '완료한 미션' API 입니다! submission 관련 API가 이제 쓰이지 않아서 제거했어요!
There was a problem hiding this comment.
아아 안써서 삭제된 거였군요! 설명해주셔서 감사합니다
| > | ||
| {type === 'icon' && children} | ||
| {content} | ||
| <S.CommonButton $size={size} $variant={variant} {...rest}> |
There was a problem hiding this comment.
[A]
사소한데, rest라는 이름 대신 attributes는 어떨까요?
rest라는 네이밍이 파라미터를 받는 위치에서는 의미 상 적절한 것 같은데, 여기 사용처에서는 조금 어색하게 느껴지는 것 같기도 해서요!
There was a problem hiding this comment.
라이언 말씀처럼 rest보다 다른 이름을 사용하는 것이 좋을 것 같아요! attribute도 좋은데 혹시 props는 어떨까요? props를 내려받는 형태라 '표시된 것 이외의 다른 props' 느낌으로 ...props는 어떨까 제안드려봅니다 ㅎㅎ
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
구현 요약
defaultprimarydefaultdefault는 회색조,primary는 컬러가 있는 버튼입니다.defaulthalffulldefaultdefault는 글자에 따라 크기가 달라지며,half의 경우 미션 상세 페이지의 미션 시작하기 버튼을 기준으로 크기를27.1rem으로 지정했습니다.full의 경우width: 100%입니다.버튼 컴포넌트를 사용하는 곳을 모두 변경했습니다!
하나하나 바꿔주면서 전부 확인했는데 혹시나 변경이 안된 부분이 있을지도 몰라요.....흑미션 현황 페이지였던 MissionSubmission 관련한 코드를 모두 제거했습니다!
연관 이슈
close #179
참고
코드 리뷰에
RCA 룰을 적용할 시 참고해주세요.