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

feat: add gdsc-grass component #27

Draft
wants to merge 12 commits into
base: main
Choose a base branch
from
Draft

Conversation

kanghyun98
Copy link
Contributor

Issue

Resolves #9

Description

외부 서비스 팀의 Github Contest 파트의 "GDSC 잔디" 컴포넌트 개발 관련 PR입니다.

아직 스타일 관련(색상, 폰트 등) constants 작업이 이루어지지 않아 Figma의 디자인에 따라 작업하였으며, 추후에 constants 작업이 완성되면 main 브랜치를 rebase하고 추가 작업하여 업로드하겠습니다.
(반응형 디자인도 일부 작업하였으나, 폰트 크기 같은 부분은 constatns 작업 이후 위의 추가 작업 때 반영하겠습니다.)

위 사항들을 제외한 나머지 요소들에 대해 리뷰 부탁드리겠습니다:)
피드백 언제든지 환영이요!! (얘기하고 싶은거 다 얘기해주세요ㅎ)

Check List

  • PR 제목을 커밋 규칙에 맞게 작성
  • 적절한 라벨 설정
  • GitHub Projects에 연결
  • 작업한 사람 모두를 Assign
  • Code Review 요청
  • 올라온 PR 중 DB Schema를 바꾸는 PR이 없는지 확인
  • DB Schema를 바꾼 경우, 다른 PR이 없는지 확인
  • main 브랜치의 최신 상태를 반영하고 있는지 확인

@kanghyun98 kanghyun98 added the task 한 스프린트 안에 끝나는 작업(스토리) label Mar 3, 2023
@kanghyun98 kanghyun98 self-assigned this Mar 3, 2023
@vercel
Copy link

vercel bot commented Mar 3, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
web ✅ Ready (Inspect) Visit Preview Mar 3, 2023 at 8:14AM (UTC)

@kanghyun98 kanghyun98 temporarily deployed to Preview March 3, 2023 08:14 — with GitHub Actions Inactive
Copy link

@ballsona ballsona left a comment

Choose a reason for hiding this comment

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

LGTM 🐰🐰
잔디 구현 꽤 빡셌을 것 같은데 수고가 많으셨읍니다,,
랭킹 컴포넌트 작업도 후다닥 피알 날릴게요!

+요일/월 정보 글자 크기가 사알짝쿵 커보이는 것 같긴 해서... 나중에 바꿔달라고 요청하면 한번만 봐줘요... from 디자인한 사람

Choose a reason for hiding this comment

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

스크린샷 2023-03-16 15 04 08

요거 컴포넌트가 안보이길래 x,y 좌표값을 수정해보니까 보이네요! 뭔가 스토리북에서는 수정이 필요할 거 같슴돠

Comment on lines +33 to +46
<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>

Choose a reason for hiding this comment

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

div 말고 rect 를 쓰신 이유는 많은 요소들을 렌더링할때 svg가 더 빨라서인가요??

Choose a reason for hiding this comment

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

ㅋㅋㅋ와 이거 그래프 구현하기 꽤 어렵겠다 생각했는데,, 역시,, 👍 이강현이 이강현했네요

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
task 한 스프린트 안에 끝나는 작업(스토리)
Projects
Status: 🏗 In progress
Development

Successfully merging this pull request may close these issues.

GDSC 잔디 프론트 작업
2 participants