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.

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

[#805] [3.0] Bar Chart 그라데이션 값 rgba 지원 #811

Merged
merged 2 commits into from
May 10, 2021
Merged

[#805] [3.0] Bar Chart 그라데이션 값 rgba 지원 #811

merged 2 commits into from
May 10, 2021

Conversation

jhee564
Copy link
Collaborator

@jhee564 jhee564 commented May 3, 2021

작업내용

  • 사용자가 입력한 색상값(hex, rgb, rgba)을 rgba로 변환하는 util 추가
  • 사용자가 입력한 색상값이 rgba일 경우, opacity값을 추출하는 util 추가
  • 그라데이션 생성로직을 비롯하여 series 그리는 부분의 fill style 로직 모두, 앞서 추가한 util을 호출하는 로직으로 변경
  • Bar Chart > Gradient 예제 코드 수정
  • 관련 .md 파일에서 color관련 설명 수정 및 누락되었던 부분 추가

작업 결과

image

@jhee564
[#805][3.0] Bar Chart 그라데이션 값 rgba 지원
7f15f02 
- 사용자가 입력한 색상값(hex, rgb, rgba) 을 rgba로 변환하는 util 메소드 추가
- 사용자가 입력한 색상값이 rgba일 경우, opacity값을 추출하는 util 메소드 추가
- 그라데이션 생성로직을 비롯하여 series 그리는 부분의 fill style 로직 모두 앞서 추가한 util을 호출하는 로직으로 변경
- Barchart > Gradient 예제 코드 수정
- 관련 md 파일 수정
@jhee564
[#805][3.0] Bar Chart 그라데이션 값 rgba 지원
5ec6776 
- 코드리뷰 피드백 반영
  1. else 문을 생략 하지 않도록 하여 의도를 명확하게 전달
  2. getOpacity 메소드에 인자가 반드시 rgba로 들어오지 않을 경우를 대비한 로직 추가
  3. 공백 제거 로직 변경
@kdeun1 kdeun1 merged commit a5c48b8 into 3.0 May 10, 2021
@kdeun1 kdeun1 deleted the 805_3.0_chart_gradient_rgba branch May 10, 2021 02:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kdeun1 kdeun1 kdeun1 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jhee564 @kdeun1