Skip to content

Bug/298/차트색상조정 #299

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.

Sign up for GitHub

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

Jump to bottom
Merged
cksrlcks merged 1 commit into from
Apr 9, 2025
Merged

Bug/298/차트색상조정 #299

cksrlcks merged 1 commit into from
Apr 9, 2025

Conversation

@cksrlcks
Copy link
Collaborator

@cksrlcks cksrlcks commented Apr 9, 2025

❓이슈

📝 Description

작업내용

차트 색상 순서를 조정했습니다.

  • 현재 월별 누적 데이터셋을 만들때 형성된 구조의 순서에 맞춰서 색상이 차트컴포넌트에 들어가게됩니다.
  • 차트 라이브러리에 일괄된 색상 순서를 위해서는 고정된 순서의 칼라 배열이 필요하여 수정했습니다.

수정전
image

수정후
image

🌀 PR Type

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항 (오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

PR

  • Branch Convention 확인

    feat/* 기능 구현, fix/* 버그 수정, refactor/* 개선

  • Base Branch 확인
  • 적절한 Label 지정
  • Assignee 및 Reviewer 지정

@cksrlcks cksrlcks added Priority: Medium 🔖 우선순위 보통 Type: Bug 🐛 버그 labels Apr 9, 2025
@cksrlcks cksrlcks requested a review from LMS10 April 9, 2025 05:22
@cksrlcks cksrlcks self-assigned this Apr 9, 2025
@cksrlcks cksrlcks linked an issue Apr 9, 2025 that may be closed by this pull request
Bug: 차트 색상 순서 조정 #298
Closed
1 task
@vercel
Copy link

vercel bot commented Apr 9, 2025

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

Name Status Preview Comments Updated (UTC)
epigram-gilv ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 9, 2025 5:22am

@github-actions
Copy link

github-actions bot commented Apr 9, 2025

Storybook URL: https://storybook-static-27phtmv1h-cksrlcks-projects.vercel.app

LMS10
LMS10 approved these changes Apr 9, 2025
View reviewed changes
Copy link
Collaborator

@LMS10 LMS10 left a comment

Choose a reason for hiding this comment

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

👍🏻훨씬 보기 좋아요!!

@cksrlcks cksrlcks merged commit 712252c into main Apr 9, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@LMS10 LMS10 LMS10 approved these changes

Assignees

@cksrlcks cksrlcks

Labels

Priority: Medium 🔖 우선순위 보통 Type: Bug 🐛 버그

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Bug: 차트 색상 순서 조정

3 participants

@cksrlcks @LMS10