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

SP-537 사용자 평점 그래프 컴포넌트 #175

Merged
merged 18 commits into from
May 14, 2024
Merged

Conversation

abiriadev
Copy link
Member

@abiriadev abiriadev commented May 7, 2024

image

배포된 storybook preview 링크: https://6639b92a06ad1d3b783f16fd-ndmldekdvb.chromatic.com/?path=/docs/components-circularrate--docs

Closes #191

💡 다음 이슈를 해결했어요.

  • 사용자 평점을 원형(circular)으로 보여주는 그래프 컴포넌트를 외부 종속성을 가져오지 않고 구현하였어요!
  • 원 안에 숫자, 문자 등의 원하는 컴포넌트를 렌더링 할 수 있게 되었어요!
  • 사용자 마음대로의 그래디언트를 사용할 수 있어요!
  • 그래프의 진행도가 전부 같게 보이는 이슈를 수정했어요.
  • 0%일때 미세한 gap이 표시되는 것을 해결했어요.


💡 이슈를 처리하면서 추가된 코드가 있어요.

💡 필요한 후속작업이 있어요.

  • 그래디언드 stop에 대한 논의
  • 마운트 시 자연스러운 애니메이션 보여주기


💡 다음 자료를 참고하면 좋아요.

✅ 셀프 체크리스트

  • 브랜치 전략에 맞는 브랜치에 PR을 올리고 있습니다. (master/main이 아닙니다.)
  • 커밋 메세지를 컨벤션에 맞추었습니다.
  • 변경 후 코드는 컴파일러/브라우저 warning/error 가 발생시키지 않습니다.
  • 변경 후 코드는 기존의 테스트를 통과합니다.
  • 테스트 추가가 필요한지 검토해보았고, 필요한 경우 테스트를 추가했습니다.
  • docs 수정이 필요한지 검토해보았고, 필요한 경우 docs를 수정했습니다.

@abiriadev abiriadev self-assigned this May 14, 2024
@abiriadev abiriadev changed the title 사용자 평점 그래프 컴포넌트 초안 SP-537 사용자 평점 그래프 컴포넌트 초안 May 14, 2024
@abiriadev abiriadev changed the title SP-537 사용자 평점 그래프 컴포넌트 초안 SP-537 사용자 평점 그래프 컴포넌트 May 14, 2024
Copy link
Collaborator

@SungHyun627 SungHyun627 left a comment

Choose a reason for hiding this comment

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

css 라이브러리 없이 사용자 평점 그래프를 구현하시느라 고생 많으셨습니다. svg를 활용하여 다양한 방식으로 색상 gradient를 표현하는 방법 배우고 갑니다!! 수고하셨습니다.

src/Components/CircularRate/index.tsx Show resolved Hide resolved
src/Components/CircularRate/index.tsx Show resolved Hide resolved
Copy link
Collaborator

@hyosin-Jang hyosin-Jang left a comment

Choose a reason for hiding this comment

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

라이브러리 없이 직접 구현하신거 정말 대단하십니다 👍
코드가 정말 깔끔하고 직관적으로 작성되어 있어서 읽기 편했습니다.
고생하셨습니다 !!

@abiriadev abiriadev merged commit e0ff579 into dev May 14, 2024
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SP-537 원형 진행도 컴포넌트
3 participants