-
Notifications
You must be signed in to change notification settings - Fork 3
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
refactor : card 공용 카드컴포넌트 리팩토링 #59
base: main
Are you sure you want to change the base?
Conversation
051581c
to
c1e73df
Compare
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.
- "✅ LGTM"
고생하셨습니다
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.
- "✅ LGTM"
2a9b886
to
0fe18dc
Compare
🔥 Issues🎯 작업 내용⚡️: Trading 반응형 css 수정⚡️: 공용 카드 컴포넌트 cva 형식으로 수정
|
🔥 Issues
🎯 작업 내용
+) clsx로 만들었다가 버튼에서 cva 사용하였기 때문에 버튼과 같은 cva로 바꿨습니다 추가 코멘트 참고해주세요
⚡️ : 공용 카드컴포넌트 리팩토링
줄바꿈 적용
폭이 좁아지면 다음 줄로 넘어가게 수정
수정 전 커밋
공용 카드 리팩토링 ⚡️ : 공용 카드컴포넌트 리팩토링
카드컴포넌트 리팩토링 해봤습니다
카드, 카드헤더, 카드 컨텐츠로 구분되어 있고 미니카드는 영록님이 분리해주신거 거의 그대로입니다
줄바꿈 적용
폭이 좁아지면 다음 줄로 넘어가게 수정
clsx 적용 ➕ : clsx 라이브러리 설치
mini card 부분에 clsx 적용해서 배경색 적용하기 편하게 했습니다
아래 코드처럼 map으로 데이터 출력할 때 인덱스 값 줘서 배경 색 편하게 지정할 수 있도록 숫자로도 이름 붙였습니다
✅ 체크 리스트
🙏 꼭 리뷰 남겨주세요.!!