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

[#785] [3.0] Horizontal Bar Chart의 Tip 위치 버그 #804

Merged
merged 4 commits into from
Apr 28, 2021
Merged

[#785] [3.0] Horizontal Bar Chart의 Tip 위치 버그 #804

merged 4 commits into from
Apr 28, 2021

Conversation

jhee564
Copy link
Collaborator

@jhee564 jhee564 commented Apr 27, 2021

이슈 요약

image

  1. Indicator 가 막대 상단에 붙어있음
  2. Tip이 막대와 관련 없는 곳에 붙어있음
  3. Tip의 화살표 방향이 세로 막대형 차트 전용으로 개발되어있음

작업내용

  1. Horizontal bar char 막대 높이 중앙값을 계산하는 로직 수정
  2. Tip의 위치를 계산하는 로직은 유지 (막대 높이값이 null 로 저장되어 발생한 문제)
  3. Tip의 화살표 방향 - 왼쪽을 바라보도록 추가
    image

추가 작업내용

  1. chartData의 값이 객체 타입이면서 value가 0일 경우 에러 발생하는 문제 해결
  2. Mouse over시 data label이 겹쳐 보이는 현상 해결

@jhee564
[#785][3.0] Horizontal Chart MaxTip 버그 수정
5745489 
##################################
- 코드리뷰 피드백 반영
  - maxTip의 값을 셋팅하는 과정에서 columnChart인 케이스만 고려되었던 로직 수정
  - maxTip, selectedTip의 위치를 계산하였으나 그리는 과정에서 Horizontal여부가 고려되지 않았던 로직 수정
@jhee564
Merge branch '3.0' of https://github.com/ex-em/EVUI into 785_3.0_char…
ad72aad 
…t_horizontal_issue
@jhee564
[#785][3.0] Horizontal Bar Chart의 Tip 위치 버그
b44f68f 
- Indicator, tip 위치 수정
- horizontal의 경우 tip 방향 왼쪽으로 전환
- 값이 0이 들어올 경우 null로 판단하는 로직 수정
@jhee564
Merge branches '3.0' and '785_3.0_chart_horizontal_issue' of https://…
7640eb1 
…github.com/ex-em/EVUI into 785_3.0_chart_horizontal_issue
@kdeun1 kdeun1 merged commit 15ca3d6 into 3.0 Apr 28, 2021
@kdeun1 kdeun1 deleted the 785_3.0_chart_horizontal_issue branch April 28, 2021 05:57
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