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

[Fix] 기본 배율 지정, 과도한 확대/축소 제한 #412

Merged
merged 2 commits into from
Dec 5, 2023

Conversation

yoopark
Copy link
Contributor

@yoopark yoopark commented Dec 5, 2023

Summary

차트 zoom-inout을 되살렸지만(#364), 무한정 확대/축소가 되어 UX에 좋지 않은 문제가 있었습니다.

Describe your changes

  • 일간 지표 : 365일 불러옴, 최근 18일만 초기 표시, 축소 한계는 diff 2일, 확대 한계는 diff가 xrange 안일 때까지만 가능합니다.
  • 월간 지표 : 전체 지표의 경우 42 시작달, 유저의 경우 본과정 시작달로부터의 정보 불러옴, 최근 12개월만 초기 표시, 축소 한계는 diff 2개월, 확대 한계는 diff가 xrange 안일 때까지만 가능합니다.
  • pan 기능을 추가로 되살렸는데, 이건 차트 배율을 조정하지 않고 옆으로 슬라이드할 때 쓰는 겁니다. (하단 영상 참고)

Issue number and link

@yoopark yoopark added the bug Something isn't working label Dec 5, 2023
@yoopark yoopark requested a review from 42sungwook December 5, 2023 10:12
@yoopark yoopark self-assigned this Dec 5, 2023
@yoopark
Copy link
Contributor Author

yoopark commented Dec 5, 2023

참고 영상 올립니다!

2023-12-05.7.20.28.1.mov

@42sungwook
Copy link
Contributor

지금은 홈 아이콘을 누르면 초기값이 아니라 전체를 보여주고 있는데
초기값으로 가게 설정하는건 어떤가요?
이런식으로 홈 아이콘 누를 때 기본값을 설정해줄 수 있어요

        beforeResetZoom: (ctx, opt) => {
          return {
            xaxis: {
              min: subDays(new Date(), 18).getTime(),
              max: ctx.maxX,
            },
          };
        },

Ref: apexcharts/apexcharts.js#1138

@yoopark
Copy link
Contributor Author

yoopark commented Dec 5, 2023

아 영상 찍으면서도 그걸 못봤네요. 추가하겠습니다!

Copy link
Contributor

@42sungwook 42sungwook left a comment

Choose a reason for hiding this comment

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

생각보다 어렵지 않게 바꼈네요 고생하셨습니당👍

@yoopark yoopark merged commit 0d392a7 into develop Dec 5, 2023
1 check passed
@yoopark yoopark deleted the fix/too-much-zoom-inout branch December 5, 2023 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants