-
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
feat: add gdsc-grass component #27
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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 🐰🐰
잔디 구현 꽤 빡셌을 것 같은데 수고가 많으셨읍니다,,
랭킹 컴포넌트 작업도 후다닥 피알 날릴게요!
+요일/월 정보 글자 크기가 사알짝쿵 커보이는 것 같긴 해서... 나중에 바꿔달라고 요청하면 한번만 봐줘요... from 디자인한 사람
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.
<rect | ||
data-level={level} | ||
className={styles.grassGraphItemStyle[level]} | ||
x={x} | ||
y={y} | ||
rx="2" | ||
ry="2" | ||
width="13" | ||
height="13" | ||
onMouseEnter={handleMouseEnter} | ||
onMouseLeave={onMouseLeave} | ||
> | ||
{infoText} | ||
</rect> |
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.
오 div
말고 rect
를 쓰신 이유는 많은 요소들을 렌더링할때 svg가 더 빨라서인가요??
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.
ㅋㅋㅋ와 이거 그래프 구현하기 꽤 어렵겠다 생각했는데,, 역시,, 👍 이강현이 이강현했네요
Issue
Resolves #9
Description
외부 서비스 팀의 Github Contest 파트의 "GDSC 잔디" 컴포넌트 개발 관련 PR입니다.
아직 스타일 관련(색상, 폰트 등) constants 작업이 이루어지지 않아 Figma의 디자인에 따라 작업하였으며, 추후에 constants 작업이 완성되면 main 브랜치를 rebase하고 추가 작업하여 업로드하겠습니다.
(반응형 디자인도 일부 작업하였으나, 폰트 크기 같은 부분은 constatns 작업 이후 위의 추가 작업 때 반영하겠습니다.)
위 사항들을 제외한 나머지 요소들에 대해 리뷰 부탁드리겠습니다:)
피드백 언제든지 환영이요!! (얘기하고 싶은거 다 얘기해주세요ㅎ)
Check List
main
브랜치의 최신 상태를 반영하고 있는지 확인