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

[토너먼트] 토너먼트 라운딩 카드 제작 완료 #59

Merged
merged 10 commits into from
Jan 10, 2024

Conversation

imeureka
Copy link
Member

이슈 넘버

구현 사항

  • 토너먼트 카드 퍼블리싱 완료
  • 토너먼트 카드 선택 시 카테고리 커스텀

Need Review

  • 선택 했을 때 카테고리 테두리 변경 state 변경했는데 적절하게 쓴건지 ,, 리팩토링 해야될 것 같습니다!
  • css 순서 다시 리팩토링 해보겟습니다!

📸 스크린샷

2024-01-09.10.02.30.mov

Reference

Sorry, something went wrong.

@imeureka imeureka added style🎨 작은 사항 수정 및 style 코드 작성 토너먼트 🏆 토너먼트 페이지 labels Jan 10, 2024
@imeureka imeureka self-assigned this Jan 10, 2024
Comment on lines +24 to +25
border: ${({ isClicked }) =>
isClicked ? '3px solid var(--Pink-P-06, #FF2176)' : '3px solid var(--Gray-G-02, #ebe9ea)'};
Copy link
Member

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)'};
Copy link
Member Author

Choose a reason for hiding this comment

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

theme 적용 시켜 놓겠습니다!

Copy link
Contributor

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;
Copy link
Member

Choose a reason for hiding this comment

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

오 이거 찾아보니 단어의 분리를 어떻게 할 것인지 결정하는 것이군요!! 새롭게 알아갑니다

Copy link
Contributor

@hoeun0723 hoeun0723 left a comment

Choose a reason for hiding this comment

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

확인 했습니다 !

Comment on lines +29 to +32
<S.Price>{formatPrice(8130000)}원</S.Price>
{/* 보러가기 컴포넌트 분리할까 말까 링크있는
아이템이면 생성 조건 렌더링 */}
</S.ItemInfo>
Copy link
Contributor

Choose a reason for hiding this comment

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

ㅎㅎ 흠.. 이 부분을 보고 figma 를 한번 보고 왔는데요, 해당 페이지가 아닌 다른 페이지에서도 반복적으로 쓰이는 재사용가능한 친구이기 때문에 컴포넌트로 분리 하는게 좋지 않을까 생각했어요 !!

Copy link
Contributor

@urjimyu urjimyu left a 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)'};
Copy link
Contributor

Choose a reason for hiding this comment

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

좋아요오👍

@imeureka imeureka merged commit 8c49a72 into develop Jan 10, 2024
1 check failed
@imeureka imeureka deleted the feat/#55-tournament-rounding branch February 2, 2024 05:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
style🎨 작은 사항 수정 및 style 코드 작성 토너먼트 🏆 토너먼트 페이지
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[토너먼트] 토너먼트 라운드 중 카드 퍼블리싱
4 participants