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
Weird zoom in/out! #2761
Comments
The zoom is relative to your cursor position, try zooming out / in with your cursor centered in the graph |
@carbonrobot first of all, why can I zoom off the chart? |
Your cursor appears to be still over the chart. Remember, the chart has padding all the way around and fills the entire column.
Obviously, it is unclear what your mouse is doing, because we can't see what buttons are being pressed via the video. I am unable to replicate that behaviour using Edge/Chrome/Firefox/Safari. |
Sure you cannot see what I do with my mouse buttons :D But I'm telling you what I'm doing. Steps to reproduce:
|
what you have done is correct, but zoom out more than that, don't stop zooming out. |
I'm still unclear what you are asking. Are you talking about the numbers repeating due to the datum using |
It's generally about both axises, but in the video you can see x-axis. Peek.2024-01-31.14-03.mp4 |
I used this script to show my mouse events :D Steps:
In the video below, there is ONLY ONE zoom in, and after that, all others are zoom out! Peek.2024-01-31.16-19.mp4 |
Mouse wheel events have a
|
Don't debug my code snippet, debug the library :D |
yes, I see that :) when you did 26 scroll-ups, it means ZOOM OUT... but actually the chart zooming in. To see the bug better, never do scroll-down after the first one. do scroll-down JUST ONCE |
I'm not seeing a bug, I'm just seeing the designed behavior. Can you describe the actual VS expected behavior? |
Perhaps there is some confusion in this example because it uses an initial zoom domain (in other words, the charts starts pre-zoomed-in). You can paste the following code into the "Editable Source" box to see if that changes anything for you.
I think we are more than happy to change something, we just need to know what you would like changed. |
same :/ Peek.2024-01-31.18-16.mp4 |
@carbonrobot can we meet somewhere? google meet or discord or wherever you can? |
Describe the bug
You can zoom out out of the chart, and the zoom out is weird then!! I don't know how to describe it. watch the screencast below!
Victory version
Your version at: https://formidable.com/open-source/victory/docs/victory-zoom-container/
Code Sandbox link
Your online example: https://formidable.com/open-source/victory/docs/victory-zoom-container/
To Reproduce
First, zoom in, and then zoom out many times, out of the chart. Like the screencast here.
Peek.2024-01-31.10-30.mp4
Desktop (please complete the following information):
The text was updated successfully, but these errors were encountered: