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

[#1038] chart select item 옵션 관련 로직 개선 #1060

Merged
merged 6 commits into from
Feb 8, 2022

Conversation

jhee564
Copy link
Collaborator

@jhee564 jhee564 commented Feb 7, 2022

이슈 내용

  1. chartOption > selectItem > use: true로 되어 있을 경우, 차트에서 데이터를 클릭한 곳으로 selectItem옵션에서 설정한 Tip 혹은 Indicator 가 옮겨지는 기능 존재
  2. 기본값의 위치가 최대값으로 고정되어 있어 maxTip과 헷갈림

주요 수정 내용

  • v-model:selectedItem로 바인딩 된 값에 해당하는 아이템을 선택
  • line, bar chart에만 해당됨 (pie, scatter 제외)
  • image
  • image
  • image

기타 수정 내용

  1. md파일에 설명 추가
  2. v-model:selectedItem로 받은 데이터를 기준으로 tip을 표시할 수 있도록 차트 생성 인자에 defaultSelectInfo 추가
  3. 불필요한 함수 제거 및 코드 정리
  4. Version Update (3.3.3 -> 3.3.4)

###################################
1. v-model 바인딩 추가
2. 관련 md파일 수정 (bar, line Chart)
3. 관련 example code 수정( bar Chart > Evnet, line chart > Event)
@jhee564 jhee564 changed the title 1038 chart select item [#1038] chart select item 옵션 관련 로직 개선 Feb 7, 2022
###################################
1. v-model 명칭 정정 (selectedItem)
2. data가 변하면 tip 제거
3. 관련 문서 수정
###################################
1. v-model 명칭 정정 (selectedItem)
2. data가 변하면 tip 제거
3. 관련 문서 수정
###################################
1. v-model 명칭 정정 (selectedItem)
2. data가 변하면 tip 제거
3. 관련 문서 수정
@kdeun1 kdeun1 merged commit 86504e2 into 3.0 Feb 8, 2022
@kdeun1 kdeun1 deleted the 1038_chart_select_item branch February 8, 2022 06:40
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

Successfully merging this pull request may close these issues.

2 participants