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

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

Closed
jhee564 opened this issue Apr 27, 2021 · 0 comments
Closed

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

jhee564 opened this issue Apr 27, 2021 · 0 comments

Comments

@jhee564
Copy link
Collaborator

jhee564 commented Apr 27, 2021

현재 색상값으로 hex 만 지원하는데 rgba 형태로도 지정할 수 있도록 로직 개선

jhee564 added a commit that referenced this issue May 3, 2021
- 사용자가 입력한 색상값(hex, rgb, rgba) 을 rgba로 변환하는 util 메소드 추가
- 사용자가 입력한 색상값이 rgba일 경우, opacity값을 추출하는 util 메소드 추가
- 그라데이션 생성로직을 비롯하여 series 그리는 부분의 fill style 로직 모두 앞서 추가한 util을 호출하는 로직으로 변경
- Barchart > Gradient 예제 코드 수정
- 관련 md 파일 수정
jhee564 added a commit that referenced this issue May 4, 2021
- 코드리뷰 피드백 반영
  1. else 문을 생략 하지 않도록 하여 의도를 명확하게 전달
  2. getOpacity 메소드에 인자가 반드시 rgba로 들어오지 않을 경우를 대비한 로직 추가
  3. 공백 제거 로직 변경
kdeun1 pushed a commit that referenced this issue May 10, 2021
* [#805][3.0] Bar Chart 그라데이션 값 rgba 지원
- 사용자가 입력한 색상값(hex, rgb, rgba) 을 rgba로 변환하는 util 메소드 추가
- 사용자가 입력한 색상값이 rgba일 경우, opacity값을 추출하는 util 메소드 추가
- 그라데이션 생성로직을 비롯하여 series 그리는 부분의 fill style 로직 모두 앞서 추가한 util을 호출하는 로직으로 변경
- Barchart > Gradient 예제 코드 수정
- 관련 md 파일 수정

* [#805][3.0] Bar Chart 그라데이션 값 rgba 지원
- 코드리뷰 피드백 반영
  1. else 문을 생략 하지 않도록 하여 의도를 명확하게 전달
  2. getOpacity 메소드에 인자가 반드시 rgba로 들어오지 않을 경우를 대비한 로직 추가
  3. 공백 제거 로직 변경

Co-authored-by: jinhee park <jinhee@ex-em.com>
@jhee564 jhee564 closed this as completed May 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant