-
Notifications
You must be signed in to change notification settings - Fork 4
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
[토너먼트] 토너먼트 라운딩 카드 제작 완료 #59
Conversation
border: ${({ isClicked }) => | ||
isClicked ? '3px solid var(--Pink-P-06, #FF2176)' : '3px solid var(--Gray-G-02, #ebe9ea)'}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍👍👍👍👍👍👍
position: relative; | ||
border-radius: 12px; | ||
border: ${({ isClicked }) => | ||
isClicked ? '3px solid var(--Pink-P-06, #FF2176)' : '3px solid var(--Gray-G-02, #ebe9ea)'}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
theme 적용 시켜 놓겠습니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
좋아요오👍
|
||
export const Title = styled.p` | ||
height: 4.2rem; | ||
word-break: keep-all; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오 이거 찾아보니 단어의 분리를 어떻게 할 것인지 결정하는 것이군요!! 새롭게 알아갑니다
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
확인 했습니다 !
<S.Price>{formatPrice(8130000)}원</S.Price> | ||
{/* 보러가기 컴포넌트 분리할까 말까 링크있는 | ||
아이템이면 생성 조건 렌더링 */} | ||
</S.ItemInfo> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ㅎㅎ 흠.. 이 부분을 보고 figma 를 한번 보고 왔는데요, 해당 페이지가 아닌 다른 페이지에서도 반복적으로 쓰이는 재사용가능한 친구이기 때문에 컴포넌트로 분리 하는게 좋지 않을까 생각했어요 !!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
컴포넌트 분리도 깔끔하고 구조 미리 잡아놓으신 게 보여서 좋았습니다👍 고생하셨어요!!
position: relative; | ||
border-radius: 12px; | ||
border: ${({ isClicked }) => | ||
isClicked ? '3px solid var(--Pink-P-06, #FF2176)' : '3px solid var(--Gray-G-02, #ebe9ea)'}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
좋아요오👍
이슈 넘버
구현 사항
Need Review
📸 스크린샷
2024-01-09.10.02.30.mov
Reference